[Xen-devel] [PATCH 0/2] libxl: try to support IGD passthrough for qemu upstream
When we're working to support IGD GFX passthrough with qemu upstream, instead of "-gfx_passthru" we'd like to make that a machine option, "-machine xxx,igd-passthru=on". This need to bring a change on tool side. After a discussion with Campbell, we'd like to construct a table to record all IGD devices we can support. If we hit that table, we should pass that option. And so we also introduce a new field of type, 'gfx_passthru_kind', to cooperate with 'gfx_passthru' to cover all scenarios like this, gfx_passthru = 0=> sets build_info.u.gfx_passthru to false gfx_passthru = 1=> sets build_info.u.gfx_passthru to false and build_info.u.gfx_passthru_kind to DEFAULT gfx_passthru = "igd"=> sets build_info.u.gfx_passthru to false and build_info.u.gfx_passthru_kind to IGD Tiejun Chen (2): libxl: introduce libxl__is_igd_vga_passthru libxl: introduce gfx_passthru_kind tools/libxl/libxl_dm.c | 13 + tools/libxl/libxl_internal.h | 2 ++ tools/libxl/libxl_pci.c | 124 tools/libxl/libxl_types.idl | 6 ++ tools/libxl/xl_cmdimpl.c | 19 +-- 5 files changed, 162 insertions(+), 2 deletions(-) Thanks Tiejun ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH 0/2] libxl: try to support IGD passthrough for qemu upstream
On Fri, Mar 06, 2015 at 05:08:21PM +0800, Tiejun Chen wrote: > When we're working to support IGD GFX passthrough with qemu > upstream, instead of "-gfx_passthru" we'd like to make that > a machine option, "-machine xxx,igd-passthru=on". This need > to bring a change on tool side. > > After a discussion with Campbell, we'd like to construct a table to record > all IGD devices we can support. If we hit that table, we should pass that > option. And so we also introduce a new field of type, 'gfx_passthru_kind', > to cooperate with 'gfx_passthru' to cover all scenarios like this, > > gfx_passthru = 0=> sets build_info.u.gfx_passthru to false > gfx_passthru = 1=> sets build_info.u.gfx_passthru to false and >build_info.u.gfx_passthru_kind to DEFAULT > gfx_passthru = "igd"=> sets build_info.u.gfx_passthru to false >and build_info.u.gfx_passthru_kind to IGD > > > Tiejun Chen (2): > libxl: introduce libxl__is_igd_vga_passthru > libxl: introduce gfx_passthru_kind > > tools/libxl/libxl_dm.c | 13 + > tools/libxl/libxl_internal.h | 2 ++ > tools/libxl/libxl_pci.c | 124 > It might be helpful to use --stat-width=72 or some other options to limit the width of your diffstat. :-) Wei. > tools/libxl/libxl_types.idl | 6 ++ > tools/libxl/xl_cmdimpl.c | 19 +-- > 5 files changed, 162 insertions(+), 2 deletions(-) > > Thanks > Tiejun ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH 0/2] libxl: try to support IGD passthrough for qemu upstream
On Fri, Mar 06, 2015 at 05:08:21PM +0800, Tiejun Chen wrote: > When we're working to support IGD GFX passthrough with qemu > upstream, instead of "-gfx_passthru" we'd like to make that Could you also include in the cover letter an URL link to the latest discussion on this? Thank you. > a machine option, "-machine xxx,igd-passthru=on". This need > to bring a change on tool side. > > After a discussion with Campbell, we'd like to construct a table to record > all IGD devices we can support. If we hit that table, we should pass that > option. And so we also introduce a new field of type, 'gfx_passthru_kind', > to cooperate with 'gfx_passthru' to cover all scenarios like this, > > gfx_passthru = 0=> sets build_info.u.gfx_passthru to false > gfx_passthru = 1=> sets build_info.u.gfx_passthru to false and >build_info.u.gfx_passthru_kind to DEFAULT > gfx_passthru = "igd"=> sets build_info.u.gfx_passthru to false >and build_info.u.gfx_passthru_kind to IGD > > > Tiejun Chen (2): > libxl: introduce libxl__is_igd_vga_passthru > libxl: introduce gfx_passthru_kind > > tools/libxl/libxl_dm.c | 13 + > tools/libxl/libxl_internal.h | 2 ++ > tools/libxl/libxl_pci.c | 124 > > tools/libxl/libxl_types.idl | 6 ++ > tools/libxl/xl_cmdimpl.c | 19 +-- > 5 files changed, 162 insertions(+), 2 deletions(-) > > Thanks > Tiejun > > ___ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH 0/2] libxl: try to support IGD passthrough for qemu upstream
On 2015/3/6 20:28, Wei Liu wrote: On Fri, Mar 06, 2015 at 05:08:21PM +0800, Tiejun Chen wrote: When we're working to support IGD GFX passthrough with qemu upstream, instead of "-gfx_passthru" we'd like to make that a machine option, "-machine xxx,igd-passthru=on". This need to bring a change on tool side. After a discussion with Campbell, we'd like to construct a table to record all IGD devices we can support. If we hit that table, we should pass that option. And so we also introduce a new field of type, 'gfx_passthru_kind', to cooperate with 'gfx_passthru' to cover all scenarios like this, gfx_passthru = 0=> sets build_info.u.gfx_passthru to false gfx_passthru = 1=> sets build_info.u.gfx_passthru to false and build_info.u.gfx_passthru_kind to DEFAULT gfx_passthru = "igd"=> sets build_info.u.gfx_passthru to false and build_info.u.gfx_passthru_kind to IGD Tiejun Chen (2): libxl: introduce libxl__is_igd_vga_passthru libxl: introduce gfx_passthru_kind tools/libxl/libxl_dm.c | 13 + tools/libxl/libxl_internal.h | 2 ++ tools/libxl/libxl_pci.c | 124 It might be helpful to use --stat-width=72 or some other options to limit the width of your diffstat. :-) Will do. Thanks Tiejun ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH 0/2] libxl: try to support IGD passthrough for qemu upstream
On 2015/3/6 23:53, Konrad Rzeszutek Wilk wrote: On Fri, Mar 06, 2015 at 05:08:21PM +0800, Tiejun Chen wrote: When we're working to support IGD GFX passthrough with qemu upstream, instead of "-gfx_passthru" we'd like to make that Could you also include in the cover letter an URL link to the latest discussion on this? Thanks for reminding. I really should do this. https://lists.nongnu.org/archive/html/qemu-devel/2015-01/msg02050.html Thanks Tiejun Thank you. a machine option, "-machine xxx,igd-passthru=on". This need to bring a change on tool side. After a discussion with Campbell, we'd like to construct a table to record all IGD devices we can support. If we hit that table, we should pass that option. And so we also introduce a new field of type, 'gfx_passthru_kind', to cooperate with 'gfx_passthru' to cover all scenarios like this, gfx_passthru = 0=> sets build_info.u.gfx_passthru to false gfx_passthru = 1=> sets build_info.u.gfx_passthru to false and build_info.u.gfx_passthru_kind to DEFAULT gfx_passthru = "igd"=> sets build_info.u.gfx_passthru to false and build_info.u.gfx_passthru_kind to IGD Tiejun Chen (2): libxl: introduce libxl__is_igd_vga_passthru libxl: introduce gfx_passthru_kind tools/libxl/libxl_dm.c | 13 + tools/libxl/libxl_internal.h | 2 ++ tools/libxl/libxl_pci.c | 124 tools/libxl/libxl_types.idl | 6 ++ tools/libxl/xl_cmdimpl.c | 19 +-- 5 files changed, 162 insertions(+), 2 deletions(-) Thanks Tiejun ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
