From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: Re: [PATCH v2] xen/gntdev: Use mempolicy instead of VM_IO flag to avoid NUMA balancing Date: Thu, 17 Nov 2016 11:01:15 -0500 Message-ID: References: <1479318378-595-1-git-send-email-boris.ostrovsky@oracle.com> <20161117112843.GA19761@aepfle.de> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============3129894516181673356==" Return-path: Received: from mail6.bemta6.messagelabs.com ([193.109.254.103]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1c7P5W-00071C-QE for xen-devel@lists.xenproject.org; Thu, 17 Nov 2016 15:58:58 +0000 In-Reply-To: <20161117112843.GA19761@aepfle.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Olaf Hering Cc: jgross@suse.com, xen-devel@lists.xenproject.org, david.vrabel@citrix.com, linux-kernel@vger.kernel.org List-Id: xen-devel@lists.xenproject.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --===============3129894516181673356== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="RfKDiS815O8b2FFB4iDXQwR5o4qUD5jkf" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --RfKDiS815O8b2FFB4iDXQwR5o4qUD5jkf Content-Type: multipart/mixed; boundary="pgFScQloRAQFWNO9g6GwA6A2nLfeIes9M"; protected-headers="v1" From: Boris Ostrovsky To: Olaf Hering Cc: david.vrabel@citrix.com, jgross@suse.com, xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org Message-ID: Subject: Re: [PATCH v2] xen/gntdev: Use mempolicy instead of VM_IO flag to avoid NUMA balancing References: <1479318378-595-1-git-send-email-boris.ostrovsky@oracle.com> <20161117112843.GA19761@aepfle.de> In-Reply-To: <20161117112843.GA19761@aepfle.de> --pgFScQloRAQFWNO9g6GwA6A2nLfeIes9M Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 11/17/2016 06:28 AM, Olaf Hering wrote: > On Wed, Nov 16, Boris Ostrovsky wrote: > >> Unfortunately I haven't been able to trigger NUMA balancing >> so while I tested this in general I am not sure I actually >> exercised the code path. > Thanks for the patch! > > Would be nice to actually test the code path which caused the initial > addition of VM_IO. I think I lack the hardware to excersise them. To trip the original bug is even more difficult than to trigger NUMA balancing. It was very sensitive to memory allocation. > > > In my 4.4 based sources I get the following unresolved symbols. If that= > happens to work in mainline the failures should at least be considered > during backporting of this proposed patch to the stable trees. > > ERROR: "__mpol_dup" [drivers/xen/xen-gntdev.ko] undefined! > ERROR: "get_task_policy" [drivers/xen/xen-gntdev.ko] undefined! > > Appearently these symbols lack just an EXPORT_SYMBOL_GPL. I just built 4.4.11 with this patch applied and haven't had any problems.= -boris --pgFScQloRAQFWNO9g6GwA6A2nLfeIes9M-- --RfKDiS815O8b2FFB4iDXQwR5o4qUD5jkf Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJYLdRLAAoJEIredpCGysGyTTkP/03Q6ruyNzifFm4jLpkgulZq GtIUPauIs5dBVtFbRNhPkzWZ1NMcxZfisb52vIKC1l0t22tT5PIdLv9oV9EAQlcO KXrr4mM76CXHx2LfbeZL1EvkUQwVmNy+O/iDP6dhG5OoIy7ldlAe/e0aRi7SAR5K cRKO+ePxRM6XaT/jtXp+M0/KWaf1e+7/qhoQxbXhcWRcq238IMUduqQ10TnaeBUZ eve3U5owG8v0KtwvtGuQad4fAMTjMpN1z+QjUbdoguqJ4KMiHjfnnzPBy58WxxEG 7u+hQyKgznPrU9AmVeRNg01Or89WAGYUtBibKpMZVCnWU9LKpYVLqvlBhW3eT+oO 5zU7KFrNGAsow7C4N3VIWE0915eLAcwKk+gGGvYAjtJ2kvDBMzJ4EbcocyleT/g6 wxRjbYjD2QuLHsTA+c8UAucKkvxe+7mvlzsce06j3G19uXUIjJbQUssnewKqg6E5 zsri2lHr9oNAznPgWqbVYKdNuI5E5+v/gyBCNhzHHPq00XAEGqO3mVF7WYm7pwQG 0IIXaR1gi3M6ybw2zx+WdldhoibWhfbZN49oHv8ZyXRR9z2p4d/pJiOh0YmPumpf XcDbERf1QsQtnwrHEcwMHbmKK1vcoO/3Y7xiQiuibvRo7QCr4VJ8nkfMdAA2rgxd sEk6T7VnY1L/jwTOXs1O =tGB6 -----END PGP SIGNATURE----- --RfKDiS815O8b2FFB4iDXQwR5o4qUD5jkf-- --===============3129894516181673356== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwczovL2xpc3RzLnhlbi5v cmcveGVuLWRldmVsCg== --===============3129894516181673356==--