On Dec 20, 2007 2:20 PM, FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote: > > On Thu, 20 Dec 2007 14:13:15 +1100 > "Julian Calaby" <julian.calaby@xxxxxxxxx> wrote: > > > On Dec 20, 2007 1:44 PM, FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote: > > > --- a/drivers/message/fusion/mptsas.c > > > +++ b/drivers/message/fusion/mptsas.c > > > @@ -1343,6 +1343,8 @@ static int mptsas_smp_handler(struct Scsi_Host *shost, struct sas_rphy *rphy, > > > smprep = (SmpPassthroughReply_t *)ioc->sas_mgmt.reply; > > > memcpy(req->sense, smprep, sizeof(*smprep)); > > > req->sense_len = sizeof(*smprep); > > > + req->data_len = 0; > > > + rsp->data_len -= smprep->ResponseDataLength; > > > > Would > > > > + rsp->data_len = -smprep->ResponseDataLength; > > > > not work? Oh <deity>! I'm totally blind! they're completely different variables! Argh! Sorry for the noise, -- Julian Calaby Email: julian.calaby@xxxxxxxxx - 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
