On Fri, Sep 21, 2012 at 1:02 AM, James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> wrote: > The data in status1 appears to get used a word at a time ... what about > the other three bytes you don't set; are they guaranteed to be zero? (in > which case this works, it just looks wrong from the way the thing is > used in the rest of the code). Yes, the structure comes from ctio = (struct ctio7_to_24xx *)qla2x00_alloc_iocbs(ha, NULL); a few lines earlier, and that does: /* Prep packet */ memset(pkt, 0, REQUEST_ENTRY_SIZE); before return pkt; -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html