[lkml]   [2006]   [Mar]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectRe: Slab corruption in 2.6.16-rc5-mm2

    On Mon, 6 Mar 2006, Linus Torvalds wrote:
    > So it's either an aic7xxx bug, or it's generic SCSI.
    > Considering that we've had other slab corruption issues (the reason I was
    > looking closely at yours), generic SCSI isn't out of the question.
    > If you were a git user, doing a bisection run would be useful since you
    > seem to be able to recreate it at will. Oh, well. Testign that one patch
    > would still help.

    Hmm.. This appended patch may or may not help.

    It overwrites the SCSI command "req" pointer when the request has been
    done. The request cannot be used afterwards, so anybody accessing it would
    be a bug. I think.

    HOWEVER. I noticed something else strange. Your slab corruption report

    Slab corruption: start=f72948a0, len=64
    Redzone: 0x5a2cf071/0x5a2cf071.
    Last user: [<c02934eb>](sr_do_ioctl+0x11b/0x270)

    and the scary thing is that "len=64".

    The thing is, SCSI uses "SCSI_SENSE_BUFFERSIZE" to determine the maximum
    sense size to copy, and what do we have, if not

    include/scsi/scsi_cmnd.h:#define SCSI_SENSE_BUFFERSIZE 96

    ie a 64-byte buffer is simply TOO DAMN SMALL!

    Now, the thing is, the 64 comes from "sizeof(struct request_sense)", which
    is what "struct packet_command *" uses. We can change that sizeof() to
    just use SCSI_SENSE_BUFFERSIZE, but that still makes me worry about
    somebody else having allocated a "packed_command->sense" using just the
    same 64-byte "struct request_sense".

    Can a SCSI sense-buffer really be 96? If so, why is "struct request_sense"
    the wrong size? This looks likea really nasty bug.


    diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
    index 701a328..296ac8e 100644
    --- a/drivers/scsi/scsi_lib.c
    +++ b/drivers/scsi/scsi_lib.c
    @@ -788,6 +788,9 @@ static struct scsi_cmnd *scsi_end_reques

    + /* Poison the request pointer: it is done and no longer exists after this */
    + cmd->request = (void *) 0x5a5a5a5a;

    spin_lock_irqsave(q->queue_lock, flags);
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at
     \ /
      Last update: 2009-11-18 23:46    [W:2.672 / U:0.052 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site