On Mon, Dec 03, 2018 at 12:26:10PM -0500, Doug Ledford wrote: > > > A per-char put_user() sounds pretty expensive since you are having to do > > > a __uaccess_begin();__uaccess_end() around every put_user(). I took it > > > as is, but it might be worth considering doing something like a 32 > > > byte > > > > This patch requries the last series of write() handlers before it will > > work properly, don't apply it just yet - will create a crazy security > > problem.. > > Yes, I thought I saw you say you applied it already? And I started from > your wip/jgg-for-next that's currently pushed. Was I incorrect in that? That is the right branch, but there are three series reworking write, only the first two are applied, the last one still in patchworks didn't get applied yet.. And this patch has to go after the prior three for all the bits to work properly, and as a uapi change it should ideally wait for the corresponding rdma-core patch to be posted.. Jason