Re: [Bug 14537] New: missing compat_ioctl on x86_64

From: Pavel Machek
Date: Sat Nov 07 2009 - 14:20:02 EST


Hi!

> > > > > - when trying to suspend:
> > > > > ioctl32(s2disk:8164): Unknown cmd fd(4) cmd(400c330d){t:'3';sz:12}
> > > > > arg(ffc190ec) on /dev/snapshot
> > > > > ioctl32(s2disk:8164): Unknown cmd fd(4) cmd(4004330a){t:'3';sz:4} arg(00000806)
> > > > > on /dev/snapshot
> > > >
> > > > snapshot_ioctl seems to lack compat support completely. Raphael ??
> > >
> > > It does, but that wouldn't work anyway. s2disk is one of the things that have
> > > to be native.
> >
> > Actually... it would be nice to fix that one day, if someone submitted
> > nice patch... :-).
>
> It must be fixed in s2disk first, but that would probably require us to drop
> support for older kernels. Which is not unthinkable, but might hurt some
> users.

It should be possible to check compat_task() and provide interface
userland expects.

But yes, improving interface to be same for 32 and 64bit would
probably be the way, too. (Or did you have something else in mind?)
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/