lkml.org 
[lkml]   [2003]   [Sep]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: CDROM_SEND_PACKET oddity
On Sat, 27 Sep 2003, Jens Axboe wrote:

> On Sat, Sep 27 2003, Jens Axboe wrote:
> > On Sat, Sep 27 2003, Jens Axboe wrote:
> > > On Fri, Sep 26 2003, Derek Foreman wrote:
> > > > The example code from
> > > > http://www.ussg.iu.edu/hypermail/linux/kernel/0202.0/att-0603/01-cd_poll.c
> > > >
> > > > Does not behave as expected on my 2.6.0-test5 system. While the command
> > > > seems to be successfully sent - 2 of my drives report it as an invalid
> > > > opcode - for the other 2 drives, the buffer comes back all zeros.
> > > > (actually, the buffer's contents will remain in whatever state they're in
> > > > before the ioctl is called)
> > > >
> > > > Sending the same command to those 2 drives with SG_IO results in the
> > > > expected behaviour.
> > >
> > > Can you try current -bk? It has some fixes for CDROM_SEND_PACKET.
> > >
> > > However, cd_poll should be rewritten to use SG_IO. Pretty trivial
> > > exercise.
> >
> > Actually, try this patch against current bk, it kills the
> > CDROM_SEND_PACKET setup and use SG_IO internally instead. Should be much
> > much better than what we have now. It's not tested here at all though,
> > I'd appreciate it if you could give it a go.
>
> This has a better chance of working. Changes:
>
> - Don't export sg_io() anyways (leftover)
> - Actually set ->cmdp and ->cmd_len
>
> still untested.

The old one worked after I fixed it up to set cmdp and the sense buffer.
I'll test this one out a little later today.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:48    [W:0.042 / U:0.572 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site