From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============1830055225474790928==" MIME-Version: 1.0 From: Walker, Benjamin Subject: [SPDK] Re: SPDK socket abstraction layer Date: Sun, 03 Nov 2019 16:56:01 +0000 Message-ID: List-ID: To: spdk@lists.01.org --===============1830055225474790928== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Nov 3, 2019 9:00 AM, Or Gerlitz wrote: On Thu, Oct 31, 2019 at 8:54 PM Walker, Benjamin wrote: > Here's the patch at the top of the series: > https://review.gerrithub.io/c/spdk/spdk/+/471752 > Zero copy is getting enabled on the socket and I do see completion > notifications, but it's always doing a deferred copy. If there was some > description somewhere of what causes the kernel to end up doing a deferre= d copy > instead of page pinning that would be really useful. the patch you pointed out doesn't change lib/nvmf/tcp.c only the sock code.. can you also refer to the patch that change nvmf The patches are all in a series with the one I linked at the end. They show= in the "relation chain" section of Gerrit. You can click the "Download" bu= tton in Gerrit for the patch I linked and it will give you a git-fetch comm= and that grabs the whole series. --===============1830055225474790928==--