From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46065) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WX52Q-0001dE-Om for qemu-devel@nongnu.org; Mon, 07 Apr 2014 04:36:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WX52L-0006s2-1X for qemu-devel@nongnu.org; Mon, 07 Apr 2014 04:36:18 -0400 Received: from smtp02.citrix.com ([66.165.176.63]:38920) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WX52K-0006rm-TS for qemu-devel@nongnu.org; Mon, 07 Apr 2014 04:36:12 -0400 Message-ID: <1396859768.22845.7.camel@kazak.uk.xensource.com> From: Ian Campbell Date: Mon, 7 Apr 2014 09:36:08 +0100 In-Reply-To: <20140404224605.GA10732@morn.localdomain> References: <1392965053-1069-1-git-send-email-yang.z.zhang@intel.com> <20140404224605.GA10732@morn.localdomain> Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [Xen-devel] [PATCH 0/5] xen: add Intel IGD passthrough support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin O'Connor Cc: peter.maydell@linaro.org, xen-devel@lists.xensource.com, stefano.stabellini@eu.citrix.com, weidong.han@intel.com, allen.m.kay@intel.com, qemu-devel@nongnu.org, jean.guyader@eu.citrix.com, anthony.perard@citrix.com, anthony@codemonkey.ws, Yang Zhang On Fri, 2014-04-04 at 18:46 -0400, Kevin O'Connor wrote: > On Fri, Feb 21, 2014 at 02:44:08PM +0800, Yang Zhang wrote: > > From: Yang Zhang > > > > The following patches are ported from Xen Qemu-traditional branch which are > > adding Intel IGD passthrough supporting to Qemu upstream. > > > > To pass through IGD to guest, user need to add following lines in Xen config > > file: > > gfx_passthru=1 > > pci=['00:02.0@2'] > > > > Besides, since Xen + Qemu upstream is requiring seabios, user also need to > > recompile seabios with CONFIG_OPTIONROMS_DEPLOYED=y to allow IGD pass through > > successfully: > > I'm not familiar with the Xen parts of your patch series. However, I > don't think one should be compiling SeaBIOS with > CONFIG_OPTIONROMS_DEPLOYED enabled. I agree, thanks for spotting this. I would nack any patch which tried to enable this option in the Xen build of SeaBIOS. > That option was for very old > versions of Bochs and QEMU that did not support fetching of optionroms > directly from PCI config space nor from fw_cfg. > > That compile time option is no longer well supported and it may be > removed in the future. I suggest looking at getting one of the other > mechanisms working instead of using CONFIG_OPTIONROMS_DEPLOYED. AFAIK the usual mechanisms already work for SeaBIOS under Xen, e.g. for the video BIOS of the emulated VGA it Just Works. I've no idea what makes Intel IGD passthrough special. Ian. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [Xen-devel] [PATCH 0/5] xen: add Intel IGD passthrough support Date: Mon, 7 Apr 2014 09:36:08 +0100 Message-ID: <1396859768.22845.7.camel@kazak.uk.xensource.com> References: <1392965053-1069-1-git-send-email-yang.z.zhang@intel.com> <20140404224605.GA10732@morn.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140404224605.GA10732@morn.localdomain> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org To: Kevin O'Connor Cc: peter.maydell@linaro.org, xen-devel@lists.xensource.com, stefano.stabellini@eu.citrix.com, weidong.han@intel.com, allen.m.kay@intel.com, qemu-devel@nongnu.org, jean.guyader@eu.citrix.com, anthony.perard@citrix.com, anthony@codemonkey.ws, Yang Zhang List-Id: xen-devel@lists.xenproject.org On Fri, 2014-04-04 at 18:46 -0400, Kevin O'Connor wrote: > On Fri, Feb 21, 2014 at 02:44:08PM +0800, Yang Zhang wrote: > > From: Yang Zhang > > > > The following patches are ported from Xen Qemu-traditional branch which are > > adding Intel IGD passthrough supporting to Qemu upstream. > > > > To pass through IGD to guest, user need to add following lines in Xen config > > file: > > gfx_passthru=1 > > pci=['00:02.0@2'] > > > > Besides, since Xen + Qemu upstream is requiring seabios, user also need to > > recompile seabios with CONFIG_OPTIONROMS_DEPLOYED=y to allow IGD pass through > > successfully: > > I'm not familiar with the Xen parts of your patch series. However, I > don't think one should be compiling SeaBIOS with > CONFIG_OPTIONROMS_DEPLOYED enabled. I agree, thanks for spotting this. I would nack any patch which tried to enable this option in the Xen build of SeaBIOS. > That option was for very old > versions of Bochs and QEMU that did not support fetching of optionroms > directly from PCI config space nor from fw_cfg. > > That compile time option is no longer well supported and it may be > removed in the future. I suggest looking at getting one of the other > mechanisms working instead of using CONFIG_OPTIONROMS_DEPLOYED. AFAIK the usual mechanisms already work for SeaBIOS under Xen, e.g. for the video BIOS of the emulated VGA it Just Works. I've no idea what makes Intel IGD passthrough special. Ian.