On Fri, 2006-06-09 at 22:25 +0200, Guennadi Liakhovetski wrote: > + > + if (cmd->use_sg) { > + struct scatterlist* sg = (struct scatterlist *)cmd->request_buffer; > + size_t offset = 0, len = sizeof(struct ScsiInqData); > + > + local_irq_save(flags); > + base = scsi_kmap_atomic_sg(sg, cmd->use_sg, &offset, &len); > + ptr = (struct ScsiInqData *)(base + offset); > + } else > + ptr = (struct ScsiInqData *)(cmd->request_buffer); hmm I wonder if the non-use-sg case can ever happen? (and if that irq_save isn't leaking here) - : 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