On 05/14/2015 02:12 AM, rajinikanth.pandurangan@xxxxxxxx wrote: > From: Rajinikanth Pandurangan <rajinikanth.pandurangan@xxxxxxxx> > > Description: > Driver sends the right size of the response buffer. > > Signed-off-by: Rajinikanth Pandurangan <rajinikanth.pandurangan@xxxxxxxx> > --- > drivers/scsi/aacraid/aachba.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/aacraid/aachba.c b/drivers/scsi/aacraid/aachba.c > index 9b3dd6e..fe59b00 100644 > --- a/drivers/scsi/aacraid/aachba.c > +++ b/drivers/scsi/aacraid/aachba.c > @@ -570,7 +570,7 @@ static int aac_get_container_name(struct scsi_cmnd * scsicmd) > > status = aac_fib_send(ContainerCommand, > cmd_fibcontext, > - sizeof (struct aac_get_name), > + sizeof(struct aac_get_name_resp), Hi Rajinikanth, in aac_fib_send the size parameter is used in just one place and a comment there says "Set the size of the Fib we want to send to the adapter" >From that^ it looks like it is the size of the command you sending, to the hw. Do I miss something? Cheers, Tomas > FsaNormal, > 0, 1, > (fib_callback)get_container_name_callback, > @@ -1052,7 +1052,7 @@ static int aac_get_container_serial(struct scsi_cmnd * scsicmd) > > status = aac_fib_send(ContainerCommand, > cmd_fibcontext, > - sizeof (struct aac_get_serial), > + sizeof(struct aac_get_serial_resp), > FsaNormal, > 0, 1, > (fib_callback) get_container_serial_callback, > -- 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