Re: [PATCH] Attempt to get eject failures back to ioctl(CDROMEJECT)

2007-11-15 Thread Rusty Russell
On Thursday 15 November 2007 19:47:05 Jens Axboe wrote: > On Thu, Nov 15 2007, Rusty Russell wrote: > > If blk_rq_map_sg returns more than was allocated, it's a bug, and > > something's already been overwritten. BUG_ON() is probably the right > > thing here. > > It really just means that it mapped

Re: [PATCH] Attempt to get eject failures back to ioctl(CDROMEJECT)

2007-11-15 Thread Jens Axboe
On Thu, Nov 15 2007, Rusty Russell wrote: > On Thursday 15 November 2007 19:47:05 Jens Axboe wrote: > > On Thu, Nov 15 2007, Rusty Russell wrote: > > > If blk_rq_map_sg returns more than was allocated, it's a bug, and > > > something's already been overwritten. BUG_ON() is probably the right > > >

Re: [PATCH] Attempt to get eject failures back to ioctl(CDROMEJECT)

2007-11-15 Thread Jens Axboe
On Thu, Nov 15 2007, Rusty Russell wrote: > On Wednesday 14 November 2007 23:39:31 Jens Axboe wrote: > > On Wed, Nov 14 2007, Rusty Russell wrote: > > > Hi Jens, > > > > > > As you asked for some time ago. Of course, it turns out that the > > > eject command ignores the error anyway, but it's

Re: [PATCH] Attempt to get eject failures back to ioctl(CDROMEJECT)

2007-11-14 Thread Rusty Russell
On Wednesday 14 November 2007 23:39:31 Jens Axboe wrote: > On Wed, Nov 14 2007, Rusty Russell wrote: > > Hi Jens, > > > > As you asked for some time ago. Of course, it turns out that the > > eject command ignores the error anyway, but it's nice that it now errors. > > > > Not entirely comf

Re: [PATCH] Attempt to get eject failures back to ioctl(CDROMEJECT)

2007-11-14 Thread Jens Axboe
On Wed, Nov 14 2007, Rusty Russell wrote: > Hi Jens, > > As you asked for some time ago. Of course, it turns out that the eject > command ignores the error anyway, but it's nice that it now errors. > > Not entirely comfortable with this patch: there's a req->errors but > that seems to ha

[PATCH] Attempt to get eject failures back to ioctl(CDROMEJECT)

2007-11-13 Thread Rusty Russell
Hi Jens, As you asked for some time ago. Of course, it turns out that the eject command ignores the error anyway, but it's nice that it now errors. Not entirely comfortable with this patch: there's a req->errors but that seems to have some existing semantics I'm not sure of, so I simply