Re: [PATCH] cifs: Fix problem with encrypted RDMA data read

From: David Howells
Date: Wed Nov 16 2022 - 07:20:22 EST


Stefan Metzmacher <metze@xxxxxxxxx> wrote:

> > Stefan Metzmacher <metze@xxxxxxxxx> wrote:
> >
> >> I'm not sure I understand why this would fix anything when encryption is
> >> enabled.
> >>
> >> Is the payload still be offloaded as plaintext? Otherwise we wouldn't have
> >> use_rdma_mr... So this rather looks like a fix for the non encrypted case.
> > The "inline"[*] PDUs are encrypted, but the direct RDMA data transmission is
> > not. I'm not sure if this is a bug in ksmbd.
>
> It's a bug in the client!

Well, if you can fix it the right way, I can test the patch. I'm not sure
what that would be if not what I suggested.

David