[lkml]   [2005]   [Jul]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [stable] Re: [05/11] SMP fix for 6pack driver
On Wed, Jul 13, 2005 at 03:13:11PM -0700, Greg KH wrote:
> On Thu, Jul 14, 2005 at 12:01:23AM +0200, Francois Romieu wrote:
> > Greg KH <> :
> > > -stable review patch. If anyone has any objections, please let us know.
> > >
> > > ------------------
> > >
> > >
> > > Drivers really only work well in SMP if they actually can be selected.
> > > This is a leftover from the time when the 6pack drive only used to be
> > > a bitrotten variant of the slip driver.
> >
> > Is the guideline above from 28/04/2005 obsoleted ?
> >
> > - It must fix a problem that causes a build error (but not for things
> > marked CONFIG_BROKEN), an oops, a hang, data corruption, a real
> > security issue, or some "oh, that's not good" issue. In short,
> > something critical.
> It lets the driver be built, when it previously could not be, unless the
> user used a config option that almost no one does...
> That's pretty critical if you ask me.

I do agree with Francois regarding this issue:

AFAIR, there has been not one 2.6 kernel where this driver was available
for SMP kernels. It's therefore untested which problems might arise with
this driver on SMP systems. I'm not arguing against including this
driver in 2.6.13, but isn't the right place.

What surprises me most is that you accepted this patch is neither in
2.6.13-rc3 nor in 2.6.13-rc3-mm1. There seems to be either an
(IMHO unfortunate) change in your policy of what patches to accept,
or there's a serious problem in your patch review process.

> thanks,
> greg k-h



"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-07-15 21:39    [W:0.178 / U:2.852 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site