Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S965781AbcKJRrs (ORCPT <rfc822;w@1wt.eu>);
        Thu, 10 Nov 2016 12:47:48 -0500
Received: from mo4-p00-ob.smtp.rzone.de ([81.169.146.221]:14130 "EHLO
        mo4-p00-ob.smtp.rzone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S934891AbcKJRrq (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 10 Nov 2016 12:47:46 -0500
X-RZG-AUTH: :P2EQZWCpfu+qG7CngxMFH1J+zrwiavkK6tmQaLfswM/RAW08C8KYHUlxfwrhu4f27QZY
X-RZG-CLASS-ID: mo00
Date: Thu, 10 Nov 2016 18:47:34 +0100
From: Olaf Hering <olaf@aepfle.de>
To: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: konrad.wilk@oracle.com, david.vrabel@citrix.com,
        linux-kernel@vger.kernel.org, stable@vger.kernel.org,
        xen-devel@lists.xen.org
Subject: Re: [Xen-devel] [PATCH RESEND] xen/gntdev: Grant maps should not be
 subject to NUMA balancing
Message-ID: <20161110174734.GB5937@aepfle.de>
References: <1447186233-3194-1-git-send-email-boris.ostrovsky@oracle.com>
 <20161110162633.GA5301@aepfle.de>
 <0c993a89-83a1-d8a8-4927-d36660179256@oracle.com>
 <20161110164226.GA5937@aepfle.de>
 <085299df-0d6e-2e1b-02b8-34f33c3d4fa6@oracle.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;
        protocol="application/pgp-signature"; boundary="5I6of5zJg18YgZEa"
Content-Disposition: inline
In-Reply-To: <085299df-0d6e-2e1b-02b8-34f33c3d4fa6@oracle.com>
User-Agent: Mutt/1.6.2 (6759)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1521
Lines: 58


--5I6of5zJg18YgZEa
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Nov 10, Boris Ostrovsky wrote:

> Are you sure it's this patch that causes the failure?
>=20
> I commented out '| VM_IO' and still unable to boot with this option.

Yes, this works for me, sles12sp2 dom0+domU, which is linux-4.4 based:

+++ b/drivers/xen/gntdev.c
@@ -804,7 +804,7 @@ static int gntdev_mmap(struct file *flip, struct vm_are=
a_struct *vma)
=20
        vma->vm_ops =3D &gntdev_vmops;
=20
-       vma->vm_flags |=3D VM_DONTEXPAND | VM_DONTDUMP | VM_IO;
+       vma->vm_flags |=3D VM_DONTEXPAND | VM_DONTDUMP /*| VM_IO*/;
=20
        if (use_ptemod)
                vma->vm_flags |=3D VM_DONTCOPY;

with this domU.cfg:

name=3D"x"
memory=3D1024
serial=3D"pty"
builder=3D"hvm"
disk=3D[ 'vdev=3Dxvda, direct-io-safe, backendtype=3Dqdisk, target=3Dx.raw'=
, ]
vif=3D[ 'bridge=3Dbr0' ]
keymap=3D"de"
cmdline=3D"linemode=3D1 console=3DttyS0,115200 ignore_loglevel install=3Dht=
tp://host/sles_dvd1/ start_shell"
kernel=3D "/sles_dvd1/boot/x86_64/vmlinuz-xen"
ramdisk=3D"/sles_dvd1/boot/x86_64/initrd-xen"


Without VM_IO 'fdisk -l /dev/xvda' works, with VM_IO 'fdisk -l
/dev/xvda' gives IO errors.

Olaf

--5I6of5zJg18YgZEa
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iEYEARECAAYFAlgksrMACgkQXUKg+qaYNn44yACePPBMJmJFki7jW3mezGP4AjQC
EMcAniZBaP6oKZTrcqzeUzxRwQam+xCY
=pCAA
-----END PGP SIGNATURE-----

--5I6of5zJg18YgZEa--