> -----Original Message----- > From: Hannes Reinecke [mailto:hare@xxxxxxx] > Sent: Monday, May 02, 2016 11:25 > To: Douglas Gilbert <dgilbert@xxxxxxxxxxxx>; linux-scsi@xxxxxxxxxxxxxxx > Cc: martin.petersen@xxxxxxxxxx; Winkler, Tomas > <tomas.winkler@xxxxxxxxx>; emilne@xxxxxxxxxx; > bart.vanassche@xxxxxxxxxxx > Subject: Re: [PATCH v2.5 2/6] scsi_debug: rework resp_report_luns > > On 05/01/2016 04:44 AM, Douglas Gilbert wrote: > > Based on "[PATH V2] scsi_debug: rework resp_report_luns" patch sent by > > Tomas Winkler on Thursday, 26 Feb 2015. His notes: > > 1. Remove duplicated boundary checks which simplify the fill-in > > loop > > 2. Use more of scsi generic API > > Replace fixed length response array a with heap allocation allowing up > > to 256 normal LUNs per target. > > > > Signed-off-by: Douglas Gilbert <dgilbert@xxxxxxxxxxxx> > > --- > > drivers/scsi/scsi_debug.c | 135 > > +++++++++++++++++++++++++++++----------------- > > 1 file changed, 87 insertions(+), 48 deletions(-) > > > Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Ack again -- 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