RE: [PATCH v1 4/5] xprtrdma: Reduce required number of send SGEs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> >> +	ia->ri_max_sgeno = max_sge - 3;
> >>
> >
> > I didn't noticed this new ri_max_sgeno variable being used in this patch set.
> Did I miss?
> 
> Yes, you snipped out the other hunk where it is used.

Right. I see it now.
> 
> > You also might want to rename it to ri_max_sge_num.
> > Regardless for some device with 3 SGEs, ri_max_sgeno will become zero. Is
> that fine?
> 
> Yes, that's OK, and tested. Zero means that NFS WRITE and SYMLINK
> payloads will always be sent in a Read chunk.
> 
Ok. Thanks.

> 
> > It would be good to have define for this minimum required 3 SGEs header
> file such as RPCRDMA_MIN_REQ_RECV_SGE.
> 
> OK.
��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux