Re: [PATCH] CPU hotplug: Slow down hotplug operations

From: Pavel Machek
Date: Mon May 12 2014 - 16:56:20 EST


On Sun 2014-05-11 20:48:08, Borislav Petkov wrote:
> On Sun, May 11, 2014 at 08:29:30PM +0200, Thomas Gleixner wrote:
> > On Sun, 11 May 2014, Pavel Machek wrote:
> > > Well. If you add the delay, you'll mask real problems and cause regressions when
> > > the delay is removed -- because fix-hotplug will probably take time to get right.
> > >
> > > Bad idea, AFAICT.
> >
> > Agreed, but Boris is right, that the current duct tape hackery needs
> > to stop. And the delay was the desperate attempt to wake up people to
> > focus on replacing the current mess instead of making it more
> > entangled.
>
> I think Pavel is missing the point: the delay will be removed with the
> rewrite of cpu hotplug, after it actually works reliably. Then we won't
> need delay anyway.

Well.. and if you don't get it right at the first try, you'll get
regression and bisect pointing at delay removal. Also... with delay in
place, you won't notice if it gets less reliable.

printk() or comment should deter people from filling bug reports....

Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/