ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* [Ksummit-discuss] Stable and delay backports
@ 2015-11-03 17:02 James Bottomley
  2015-11-03 17:08 ` Jens Axboe
  2015-11-03 17:30 ` Greg Kroah-Hartman
  0 siblings, 2 replies; 4+ messages in thread
From: James Bottomley @ 2015-11-03 17:02 UTC (permalink / raw)
  To: ksummit-discuss

I'm still not clear, even after all the discussion, whether there's any
value left to annotating the cc to stable with a delay backport.  I'm
getting ready to post a fix to our block size calculations which make
them completely accurate instead of within 5% like they were before.
Technically this is a bug fix because people get distressed even over
apparently losing 5% of their space, so it will have to be backported,
but the algorithm has increased in complexity, so it would be better to
incubate in main line for a while to make sure there are no further
complaints.

So the question: I think I heard Greg say you're automatically delaying
merge window backports anyway, so there's no real need to add a separate
delay tag, or is there?

James

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Ksummit-discuss] Stable and delay backports
  2015-11-03 17:02 [Ksummit-discuss] Stable and delay backports James Bottomley
@ 2015-11-03 17:08 ` Jens Axboe
  2015-11-03 18:09   ` James Bottomley
  2015-11-03 17:30 ` Greg Kroah-Hartman
  1 sibling, 1 reply; 4+ messages in thread
From: Jens Axboe @ 2015-11-03 17:08 UTC (permalink / raw)
  To: James Bottomley, ksummit-discuss

On 11/03/2015 10:02 AM, James Bottomley wrote:
> I'm still not clear, even after all the discussion, whether there's any
> value left to annotating the cc to stable with a delay backport.  I'm
> getting ready to post a fix to our block size calculations which make
> them completely accurate instead of within 5% like they were before.
> Technically this is a bug fix because people get distressed even over
> apparently losing 5% of their space, so it will have to be backported,
> but the algorithm has increased in complexity, so it would be better to
> incubate in main line for a while to make sure there are no further
> complaints.
>
> So the question: I think I heard Greg say you're automatically delaying
> merge window backports anyway, so there's no real need to add a separate
> delay tag, or is there?

Or you could just do it manually. Don't mark it stable, get it in 
mainline. Then send stable@ and email when you feel it's safe, asking 
them to pickup that commit.

-- 
Jens Axboe

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Ksummit-discuss] Stable and delay backports
  2015-11-03 17:02 [Ksummit-discuss] Stable and delay backports James Bottomley
  2015-11-03 17:08 ` Jens Axboe
@ 2015-11-03 17:30 ` Greg Kroah-Hartman
  1 sibling, 0 replies; 4+ messages in thread
From: Greg Kroah-Hartman @ 2015-11-03 17:30 UTC (permalink / raw)
  To: James Bottomley; +Cc: ksummit-discuss

On Tue, Nov 03, 2015 at 09:02:35AM -0800, James Bottomley wrote:
> I'm still not clear, even after all the discussion, whether there's any
> value left to annotating the cc to stable with a delay backport.  I'm
> getting ready to post a fix to our block size calculations which make
> them completely accurate instead of within 5% like they were before.
> Technically this is a bug fix because people get distressed even over
> apparently losing 5% of their space, so it will have to be backported,
> but the algorithm has increased in complexity, so it would be better to
> incubate in main line for a while to make sure there are no further
> complaints.
> 
> So the question: I think I heard Greg say you're automatically delaying
> merge window backports anyway, so there's no real need to add a separate
> delay tag, or is there?

If you want to ensure that I don't instantly pick it up, sure, put a
"add to stable after XXX" marking on the patch, or do like Jens
suggested, either will work.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Ksummit-discuss] Stable and delay backports
  2015-11-03 17:08 ` Jens Axboe
@ 2015-11-03 18:09   ` James Bottomley
  0 siblings, 0 replies; 4+ messages in thread
From: James Bottomley @ 2015-11-03 18:09 UTC (permalink / raw)
  To: Jens Axboe; +Cc: ksummit-discuss

On Tue, 2015-11-03 at 10:08 -0700, Jens Axboe wrote:
> On 11/03/2015 10:02 AM, James Bottomley wrote:
> > I'm still not clear, even after all the discussion, whether there's any
> > value left to annotating the cc to stable with a delay backport.  I'm
> > getting ready to post a fix to our block size calculations which make
> > them completely accurate instead of within 5% like they were before.
> > Technically this is a bug fix because people get distressed even over
> > apparently losing 5% of their space, so it will have to be backported,
> > but the algorithm has increased in complexity, so it would be better to
> > incubate in main line for a while to make sure there are no further
> > complaints.
> >
> > So the question: I think I heard Greg say you're automatically delaying
> > merge window backports anyway, so there's no real need to add a separate
> > delay tag, or is there?
> 
> Or you could just do it manually. Don't mark it stable, get it in 
> mainline. Then send stable@ and email when you feel it's safe, asking 
> them to pickup that commit.

Actually, that's the worst of both worlds because I'd have to remember
there's a patch to backport with no marker.  If I were going to do my
own stable patches, I'd follow the DaveM process because then at least
I'd be collecting all the stable patches into my trees so there'd be
external visibility and a marker for me not to forget.  However, I was
just after low overhead ...

Anyway, I have the answer: do it as before.

James

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-11-03 18:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-03 17:02 [Ksummit-discuss] Stable and delay backports James Bottomley
2015-11-03 17:08 ` Jens Axboe
2015-11-03 18:09   ` James Bottomley
2015-11-03 17:30 ` Greg Kroah-Hartman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox