Re: [PATCH v2 0/6] Reduce SMBDirect RDMA SGE counts and sizes

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

 



I can add the Acked-bys if you send them to me (for the cifs.ko ones)

The client for server (cifs vs ksmbd prefix) in the title is more of
an issue for email threads and patch review.

On Thu, Sep 29, 2022 at 10:15 AM Tom Talpey <tom@xxxxxxxxxx> wrote:
>
> I need to add the "cifs" and "ksmbd" prefixes, and a couple of
> Acked-by's. I'm still pretty ill so not getting much done just
> now though. I'll try to get on it later today.
>
> On 9/29/2022 1:02 AM, Steve French wrote:
> > merged patches 1, 3, 5, 6 of this series into cifs-2.6.git for-next
> > (will let Namjae test/try the server patches, 2 and 4) pending
> > additional testing.
> >
> > Let me know if any Reviewed-by to add
> >
> > On Fri, Sep 23, 2022 at 4:54 PM Tom Talpey <tom@xxxxxxxxxx> wrote:
> >>
> >> Allocate fewer SGEs and standard packet sizes in both kernel SMBDirect
> >> implementations.
> >>
> >> The current maximum values (16 SGEs and 8192 bytes) cause failures on the
> >> SoftiWARP provider, and are suboptimal on others. Reduce these to 6 and
> >> 1364. Additionally, recode smbd_send() to work with as few as 2 SGEs,
> >> and for debug sanity, reformat client-side logging to more clearly show
> >> addresses, lengths and flags in the appropriate base.
> >>
> >> Tested over SoftiWARP and SoftRoCE with shell, Connectathon basic and general.
> >>
> >> v2: correct an uninitialized value issue found by Coverity
> >>
> >> Tom Talpey (6):
> >>    Decrease the number of SMB3 smbdirect client SGEs
> >>    Decrease the number of SMB3 smbdirect server SGEs
> >>    Reduce client smbdirect max receive segment size
> >>    Reduce server smbdirect max send/receive segment sizes
> >>    Handle variable number of SGEs in client smbdirect send.
> >>    Fix formatting of client smbdirect RDMA logging
> >>
> >>   fs/cifs/smbdirect.c       | 227 ++++++++++++++++----------------------
> >>   fs/cifs/smbdirect.h       |  14 ++-
> >>   fs/ksmbd/transport_rdma.c |   6 +-
> >>   3 files changed, 109 insertions(+), 138 deletions(-)
> >>
> >> --
> >> 2.34.1
> >>
> >
> >



-- 
Thanks,

Steve



[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux