It is used in be_main.c (num_sge_per_io). The FW expects and sglist with 32 SGE's but 2 are used internally. -Jay -- _____ From: Mike Christie [mailto:michaelc@xxxxxxxxxxx] To: Jayamohan Kalickal [mailto:jayamohank@xxxxxxxxxxxxxxxxx] Cc: linux-scsi@xxxxxxxxxxxxxxx, James.Bottomley@xxxxxxx Sent: Wed, 06 Jan 2010 12:30:52 -0800 Subject: Re: [PATCH 2/3] be2iscsi: Fixing the number of SGE's On 01/06/2010 02:21 PM, Jayamohan Kallickal wrote: > The number of SGE's supported is fixed to what the > chip expects. Also, the max sectors set to tested values > > Signed-off-by: Jayamohan Kallickal<jayamohank@xxxxxxxxxxxxxxxxx> > --- > drivers/scsi/be2iscsi/be_main.h | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/be2iscsi/be_main.h b/drivers/scsi/be2iscsi/be_main.h > index 4cde8f6..13ff02c 100644 > --- a/drivers/scsi/be2iscsi/be_main.h > +++ b/drivers/scsi/be2iscsi/be_main.h > @@ -56,10 +56,10 @@ > #define BE2_DEFPDU_DATA_SZ 8192 > > #define MAX_CPUS 31 > -#define BEISCSI_SGLIST_ELEMENTS BE2_SGE > +#define BEISCSI_SGLIST_ELEMENTS 30 BE2_SGE is now not used, so you can kill it. -- 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 ___________________________________________________________________________________ This message, together with any attachment(s), contains confidential and proprietary information of ServerEngines Corporation and is intended only for the designated recipient(s) named above. Any unauthorized review, printing, retention, copying, disclosure or distribution is strictly prohibited. If you are not the intended recipient of this message, please immediately advise the sender by reply email message and delete all copies of this message and any attachment(s). Thank you. -- 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
