From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: GPU passthrough issue when VM is configured with 4G memory Date: Tue, 16 Apr 2013 13:37:03 +0100 Message-ID: <516D45EF.9040608@eu.citrix.com> References: <513739DD.8050507@eu.citrix.com> <513868F0.6020104@eu.citrix.com> <20130415212258.GG11427@reaktio.net> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1"; Format="flowed" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20130415212258.GG11427@reaktio.net> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: =?ISO-8859-1?Q?Pasi_K=E4rkk=E4inen?= Cc: Stefano Stabellini , "xudong.hao@intel.com" , Yanqiangjun , Luonengjun , Wangzhenguo , Yangxiaowei , "Gonglei (Arei)" , Anthony Perard , "xen-devel@lists.xen.org" , Hanweidong , "xiantao.zhang@intel.com" List-Id: xen-devel@lists.xenproject.org On 15/04/13 22:22, Pasi K=E4rkk=E4inen wrote: > On Tue, Mar 26, 2013 at 09:37:53AM +0000, Hanweidong wrote: >>>> HVM_BELOW_4G_RAM_END is f0000000. If we change HVM_BELOW_4G_RAM_END >>> to e0000000, >>>> Which it's consistent with hvmloader when assigning a GPU, and then >>> guest worked >>>> for us. So we wondering that xen_ram_init in QEMU should be >>> consistent with >>>> hvmloader. >>>> >>>> In addition, we found QEMU uses hardcode 0xe0000000 in pc_init1() as >>> below. >>>> Should keep these places handle the consistent mmio hole or not? >>>> >>>> if (ram_size >=3D 0xe0000000 ) { >>>> above_4g_mem_size =3D ram_size - 0xe0000000; >>>> below_4g_mem_size =3D 0xe0000000; >>>> } else { >>>> above_4g_mem_size =3D 0; >>>> below_4g_mem_size =3D ram_size; >>>> } >>> The guys at Intel sent a couple of patches recently to fix this issue: >>> >>> http://marc.info/?l=3Dxen-devel&m=3D136150317011027 >>> http://marc.info/?l=3Dqemu-devel&m=3D136177475215360&w=3D2 >>> >>> Do they solve your problem? >> These two patches didn't solve our problem. >> > Any updates on this? It'd be nice to get this fixed before Xen 4.3. It looks like the qemu guys didn't like the patch; they said there was = already an interface to get this information. I didn't see whether Xu = Dong followed that up or not. I'm going to list this as a bug, hopefully we can get it sorted out. -George > > Thanks, > > -- Pasi > = >> --weidong >> >>> Xudong and Xiantao, >>> are you going to send an update of the second patch to QEMU?