[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] tftp: roll-over block counter to prevent data packets timeou
|
From: |
Daniel Kiper |
|
Subject: |
Re: [PATCH] tftp: roll-over block counter to prevent data packets timeouts |
|
Date: |
Thu, 10 Sep 2020 15:02:55 +0200 |
|
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Thu, Sep 10, 2020 at 12:07:42PM +0200, Javier Martinez Canillas wrote:
> On 9/10/20 12:00 AM, Alexey Makhalov wrote:
>
> [snip]
>
> >>>>> Fixes: 781b3e5efc3 ("tftp: Do not use priority queue")
> >>>>
> >>>> Please drop this line.
> >>>>
> >>>
> >>> Same question here. I think is important information, specially for
> >>> downstream since they could allow people to decide whether they need
> >>> to backport this patch or not.
> >>
> >> You duplicate the information which is above. Additionally, IMO "Fixes:"
> >> should contain bug number, CVE number, link to the bug, etc. not the
> >> commit id.
> >
> > I think “Fixes: commit id” should remain in place. It provides direct
> > information
> > from what commit the bug existed in case of regression.
> >
>
> Yes, I think the same. For example I usually do git log --grep="Fixes:", but
> don't have a strong opinion and I'm OK with dropping it if Daniel prefer that.
If more people like it I am not going to object so strongly...
Daniel