[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] xen-netback stable backports request (regression fixes)



On Wed, 2013-06-05 at 13:25 -0700, Greg KH wrote: 
> > For the first we thought that allocating a variable size array on the
> > stack was bad practice which could lead to a stack overflow. The default
> > is safe but can be overridden by a module parameter.
> > 
> > The other renames the module parameter which we thought was useful for
> > consistency between newer kernels and the stable branches.
> 
> So you are doing a user/kernel api change in a stable branch?  Heck, you
> did it in 3.10?  Why?  What did you just break by doing that?  I can't
> do that to users of the stable kernels, sorry, and you shouldn't be
> doing that to users of the 3.10 kernel either.  You should rename this
> back so people's working systems don't break when they upgrade their
> kernel.

The module parameter was introduce after 3.9 and then renamed before
v3.10-rc1 happened, the original name was never part of an -rc let alone
an actual release.

I hadn't considered the consequences of the introduction and renaming
getting backported to stable in separate stable releases, sorry.

I think in practise it is highly unlikely that anyone is using this
option but equally I'd rather change the name before people do to start
to use it.

If this is a problem for stable perhaps a patch to make it accept either
name would be useful? Seems to me like that is just deferring things a
bit further though.

> No worries, I've applied just the 9ecd1a75 patch.

Thanks,

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.