On Fri, May 13, 2011 at 12:29:22PM +0200, Mark Kettenis wrote:
> > On Fri, May 13, 2011 at 11:39:01AM +0200, Mike Belopuhov wrote:
> >
> > > On Fri, May 13, 2011 at 11:26 AM, Mark Kettenis <mark.kette...@xs4all.nl>
> > > wrote:
> > > >> From: Vadim Zhukov <persg...@gmail.com>
> > > >> Date: Fri, 13 May 2011 13:10:10 +0400
> > > >>
> > > >> Hello all.
> > > >>
> > > >> Looks like there is a typo in ioctl number...
> > > >
> > > > What makes you think this is a typo?
> > > >
> > >
> > > there are two ioctls with the same command number:
> > >
> > > #define BIOCDISCIPLINE _IOWR('B', 40, struct bioc_discipline)
> > > #define BIOCINSTALLBOOT _IOWR('B', 40, struct bioc_installboot)
> >
> > Only if the two structs happen to have the same size they will clash.
>
> It's still a bad idea to use the same command number. I believe we
> encode the size of the structure in there as a sanity check, not to
> extend the ioctl "namespace".
I agree with that, so if the ABI change will not cause too many
troubles, it should be fixed.
-Otto
