> -----Original Message----- > From: Or Gerlitz [mailto:[email protected]] > Sent: Wednesday, July 01, 2015 3:27 PM > To: Steve Wise > Cc: Doug Ledford; Roi Dayan; [email protected]; Sagi Grimberg; Mike > Marciniszyn; [email protected]; Eli Cohen; Or > Gerlitz > Subject: Re: [PATCH V3 3/4] RDMA/iser: limit sg tablesize to device fastreg > max depth > > On Wed, Jul 1, 2015 at 7:30 PM, Steve Wise <[email protected]> > wrote: > > please no empty change-logs for this driver (and elsewhere), find > something that fills @ least one sentence, it's easy. >
sure. > Also, we are aiming to initiate (Sagi please make sure you follow on > this practice too) commit titles for the iser initiator with Capital > letter, so s/limit/Limit/g > I thought SOP was to _not_ capitalize in this case? > > Signed-off-by: Steve Wise <[email protected]> > > --- > > drivers/infiniband/ulp/iser/iscsi_iser.c | 7 +++++++ > > 1 files changed, 7 insertions(+), 0 deletions(-) > > > > diff --git a/drivers/infiniband/ulp/iser/iscsi_iser.c > > b/drivers/infiniband/ulp/iser/iscsi_iser.c > > index 6a594aa..ec692f7 100644 > > --- a/drivers/infiniband/ulp/iser/iscsi_iser.c > > +++ b/drivers/infiniband/ulp/iser/iscsi_iser.c > > @@ -640,6 +640,13 @@ iscsi_iser_session_create(struct iscsi_endpoint *ep, > > SHOST_DIX_GUARD_CRC); > > } > > > > + /* > > + * Limit the sg_tablesize based on the device max fastreg > > page > > + * list length. > > + */ > > + shost->sg_tablesize = min_t(u32, shost->sg_tablesize, > > + > > ib_conn->device->dev_attr.max_fast_reg_page_list_len); > > + > > I guess you should also somehow recap shost->max_sectors (which is > hard coded to 1024 (== 512KB IO) now) for your needs. > Ok. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
