Re: [patch] scsi: revert "[SCSI] Get rid of scsi_cmnd->done"


On Wed, 2008-01-02 at 12:40 -0700, Matthew Wilcox wrote:
On Wed, Jan 02, 2008 at 11:19:14AM -0800, Linus Torvalds wrote:
It's totally immaterial if we have one reporter or many. The fact is, that
thing has been outstanding for almost two months now. The root cause is
almost certainly known (and Matthew is apparently even aware of it), but

I really don't think the root cause is known.

By now, it needs to either have a patch, or to be reverted. It's too late
to make excuses.

I think reverting it is the correct thing to do.

OK ... I'll revert it. However, I still think it's the wrong course of
action, because as far as my analysis goes, this code is functionally
equivalent to what went before with the exception that we now rely on
the request->cmd_type information in the post processing (previously we
just relied on the cmnd->done pointer).

As far as I understand what's going on, the reporter is misusing the
interface. He sets up a packet mapping and then accesses the underlying
device, which is wrong (you're supposed to access now via the packet
device). The one thing the packet mapping does is to alter the
blocksize, which could lead to the errors reported (because the
underlying fs block size and the actual block size differ) ... however,
I think it should do this all the time, so what I'm unable to explain is
why it doesn't write past the end of the device with this commit
reverted.

But all of the above is the hallmark of an existing bug that this commit
exposes ... revert it, and we'll cover the underlying problem up again.

James


--
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/

Relevant Pages

  • Re: [PATCH] sched: revert load_balance_monitor()
    ... it's fairly lurchy with three root chew-max and one user chew-max ... Running the same with mainline + full revert, ... Group CPU distribution otoh is better than mainline git + full revert. ...
    (Linux-Kernel)
  • Re: backup solution for sbs?
    ... >> All trolls ultimately revert to personal attacks when they can't carry ... > root, if I may. ...
    (microsoft.public.windows.server.sbs)
  • Re: OSR5.0.7: hidden files at the root directory
    ... It's specifically -A that is initially inverted for root, ... see hidden files by giving an additional -A on the command line. ... command "ls -a" in vain in an attempt to revert "ls" behavior when invoked as root, and it obviously kept showing those hidden files. ...
    (comp.unix.sco.misc)
  • Re: chmod
    ... kde that the rights spring back to the original settings? ... I am root when I ... do it but they still revert back. ...
    (alt.os.linux.suse)
Loading