mbox

[PULL,0/19] xen-2015-09-08-tag

Message ID alpine.DEB.2.02.1509081818590.2672@kaball.uk.xensource.com
State New
Headers show

Pull-request

git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-2015-09-08-tag

Message

Stefano Stabellini Sept. 8, 2015, 5:21 p.m. UTC
The following changes since commit 8611280505119e296757a60711a881341603fa5a:

  target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)

are available in the git repository at:

  git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-2015-09-08-tag

for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:

  xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings. (2015-09-08 15:21:56 +0000)

----------------------------------------------------------------
Xen branch xen-2015-09-08

----------------------------------------------------------------
Don Slutz (1):
      xen-hvm: Add trace to ioreq

Jan Beulich (1):
      xen/HVM: atomically access pointers in bufioreq handling

Konrad Rzeszutek Wilk (7):
      xen-hvm: When using xc_domain_add_to_physmap also include errno when reporting
      xen/pt: Update comments with proper function name.
      xen/pt: Make xen_pt_msi_set_enable static
      xen/pt: xen_host_pci_config_read returns -errno, not -1 on failure
      xen: use errno instead of rc for xc_domain_add_to_physmap
      xen/pt/msi: Add the register value when printing logging and error messages
      xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.

Michael S. Tsirkin (1):
      i440fx: make types configurable at run-time

Tiejun Chen (9):
      pc_init1: pass parameters just with types
      piix: create host bridge to passthrough
      hw/pci-assign: split pci-assign.c
      xen, gfx passthrough: basic graphics passthrough support
      xen, gfx passthrough: retrieve VGA BIOS to work
      igd gfx passthrough: create a isa bridge
      xen, gfx passthrough: register a isa bridge
      xen, gfx passthrough: register host bridge specific to passthrough
      xen, gfx passthrough: add opregion mapping

 configure                     |   28 +++++
 hw/core/machine.c             |   20 +++
 hw/i386/Makefile.objs         |    1 +
 hw/i386/kvm/pci-assign.c      |   82 ++-----------
 hw/i386/pc_piix.c             |  139 ++++++++++++++++++++-
 hw/i386/pci-assign-load-rom.c |   93 ++++++++++++++
 hw/pci-host/piix.c            |   91 +++++++++++++-
 hw/xen/Makefile.objs          |    1 +
 hw/xen/xen-host-pci-device.c  |    5 +
 hw/xen/xen-host-pci-device.h  |    1 +
 hw/xen/xen_pt.c               |   42 ++++++-
 hw/xen/xen_pt.h               |   22 +++-
 hw/xen/xen_pt_config_init.c   |   59 ++++++++-
 hw/xen/xen_pt_graphics.c      |  272 +++++++++++++++++++++++++++++++++++++++++
 hw/xen/xen_pt_msi.c           |    2 +-
 include/hw/boards.h           |    1 +
 include/hw/i386/pc.h          |    9 +-
 include/hw/pci/pci-assign.h   |   27 ++++
 include/hw/xen/xen_common.h   |   34 +++++-
 qemu-options.hx               |    3 +
 trace-events                  |    7 ++
 vl.c                          |   10 ++
 xen-hvm.c                     |   55 +++++++--
 23 files changed, 891 insertions(+), 113 deletions(-)
 create mode 100644 hw/i386/pci-assign-load-rom.c
 create mode 100644 hw/xen/xen_pt_graphics.c
 create mode 100644 include/hw/pci/pci-assign.h

Comments

Peter Maydell Sept. 8, 2015, 7:12 p.m. UTC | #1
On 8 September 2015 at 18:21, Stefano Stabellini
<stefano.stabellini@eu.citrix.com> wrote:
> The following changes since commit 8611280505119e296757a60711a881341603fa5a:
>
>   target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
>
> are available in the git repository at:
>
>   git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-2015-09-08-tag
>
> for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
>
>   xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings. (2015-09-08 15:21:56 +0000)
>
> ----------------------------------------------------------------
> Xen branch xen-2015-09-08
>
> ----------------------------------------------------------------

Hi. I'm afraid this fails to build on OSX (and probably Windows too,
though that build hasn't run yet):

  CC    i386-softmmu/hw/i386/pci-assign-load-rom.o
/Users/pm215/src/qemu/hw/i386/pci-assign-load-rom.c:6:10: fatal error:
      'sys/io.h' file not found
#include <sys/io.h>
         ^
  CC    alpha-softmmu/hw/alpha/pci.o
1 error generated.

thanks
-- PMM
Stefano Stabellini Sept. 9, 2015, 1:06 p.m. UTC | #2
On Tue, 8 Sep 2015, Peter Maydell wrote:
> On 8 September 2015 at 18:21, Stefano Stabellini
> <stefano.stabellini@eu.citrix.com> wrote:
> > The following changes since commit 8611280505119e296757a60711a881341603fa5a:
> >
> >   target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
> >
> > are available in the git repository at:
> >
> >   git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-2015-09-08-tag
> >
> > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
> >
> >   xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings. (2015-09-08 15:21:56 +0000)
> >
> > ----------------------------------------------------------------
> > Xen branch xen-2015-09-08
> >
> > ----------------------------------------------------------------
> 
> Hi. I'm afraid this fails to build on OSX (and probably Windows too,
> though that build hasn't run yet):
> 
>   CC    i386-softmmu/hw/i386/pci-assign-load-rom.o
> /Users/pm215/src/qemu/hw/i386/pci-assign-load-rom.c:6:10: fatal error:
>       'sys/io.h' file not found
> #include <sys/io.h>
>          ^
>   CC    alpha-softmmu/hw/alpha/pci.o
> 1 error generated.

Tiejun,

this is caused by 33d33242b7d802e6c994f3d56ecba96a66465dc3,
"hw/pci-assign: split pci-assign.c". Could you please double-check
non-Linux builds?


I suspect that the fix would be quite small, but I don't have an OSX or
a Windows build environment to try it.

Speak about build environments, Peter, would you care to share your
scripts and setup so that I can run similar tests in the future on my
own?  I have no OSX machines so I tried to do a Windows
cross-compile, following http://wiki.qemu.org/Hosts/W32 on Debian 7, but
I failed very early with an "ERROR: zlib check failed".
Stefano Stabellini Sept. 9, 2015, 4:10 p.m. UTC | #3
On Wed, 9 Sep 2015, Stefano Stabellini wrote:
> On Tue, 8 Sep 2015, Peter Maydell wrote:
> > On 8 September 2015 at 18:21, Stefano Stabellini
> > <stefano.stabellini@eu.citrix.com> wrote:
> > > The following changes since commit 8611280505119e296757a60711a881341603fa5a:
> > >
> > >   target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
> > >
> > > are available in the git repository at:
> > >
> > >   git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-2015-09-08-tag
> > >
> > > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
> > >
> > >   xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings. (2015-09-08 15:21:56 +0000)
> > >
> > > ----------------------------------------------------------------
> > > Xen branch xen-2015-09-08
> > >
> > > ----------------------------------------------------------------
> > 
> > Hi. I'm afraid this fails to build on OSX (and probably Windows too,
> > though that build hasn't run yet):
> > 
> >   CC    i386-softmmu/hw/i386/pci-assign-load-rom.o
> > /Users/pm215/src/qemu/hw/i386/pci-assign-load-rom.c:6:10: fatal error:
> >       'sys/io.h' file not found
> > #include <sys/io.h>
> >          ^
> >   CC    alpha-softmmu/hw/alpha/pci.o
> > 1 error generated.
> 
> Tiejun,
> 
> this is caused by 33d33242b7d802e6c994f3d56ecba96a66465dc3,
> "hw/pci-assign: split pci-assign.c". Could you please double-check
> non-Linux builds?

I found another issue introduced by the gfx passthrough series on
Windows:

../hw/pci-host/piix.o: In function `host_pci_config_read':
/root/qemu/hw/pci-host/piix.c:778: undefined reference to `_pread'

It is introduced by:

commit fdb70721ba0496a767137e5505dd27627d19c4a8
Author: Tiejun Chen <tiejun.chen@intel.com>
Date:   Wed Jul 15 13:37:43 2015 +0800

    piix: create host bridge to passthrough


You might have to replace the pread call with lseek and read.
Peter Maydell Sept. 9, 2015, 5:21 p.m. UTC | #4
On 9 September 2015 at 14:06, Stefano Stabellini
<stefano.stabellini@eu.citrix.com> wrote:
> Speak about build environments, Peter, would you care to share your
> scripts and setup so that I can run similar tests in the future on my
> own?  I have no OSX machines so I tried to do a Windows
> cross-compile, following http://wiki.qemu.org/Hosts/W32 on Debian 7, but
> I failed very early with an "ERROR: zlib check failed".

My build scripts are at
https://git.linaro.org/people/peter.maydell/misc-scripts.git
(in particular pull-buildtest runs the test and remake-merge-builds
does the one-time configure-each-build-tree setup.) But they're really
just doing a make/make check. I don't have scripts for getting the
cross-compile environment set up in the first place, I'm afraid.

Your error message means you're missing the zlib cross library.
Fedora provides packaged versions of all the cross libs, but for
Debian you have to download them from the upstream websites and
stick them in a directory somewhere.

My (Ubuntu-based) instructions from last time somebody asked:
https://lists.gnu.org/archive/html/qemu-devel/2015-04/msg00688.html

thanks
-- PMM
Peter Maydell Sept. 9, 2015, 5:33 p.m. UTC | #5
On 9 September 2015 at 17:10, Stefano Stabellini
<stefano.stabellini@eu.citrix.com> wrote:
> I found another issue introduced by the gfx passthrough series on
> Windows:
>
> ../hw/pci-host/piix.o: In function `host_pci_config_read':
> /root/qemu/hw/pci-host/piix.c:778: undefined reference to `_pread'
>
> It is introduced by:
>
> commit fdb70721ba0496a767137e5505dd27627d19c4a8
> Author: Tiejun Chen <tiejun.chen@intel.com>
> Date:   Wed Jul 15 13:37:43 2015 +0800
>
>     piix: create host bridge to passthrough
>
>
> You might have to replace the pread call with lseek and read.

Will passthrough even work on Windows and OSX hosts?
Consider whether we should be building this code on those
hosts at all...

thanks
-- PMM
Tiejun Chen Sept. 10, 2015, 1:21 a.m. UTC | #6
On 9/10/2015 12:10 AM, Stefano Stabellini wrote:
> On Wed, 9 Sep 2015, Stefano Stabellini wrote:
>> On Tue, 8 Sep 2015, Peter Maydell wrote:
>> > On 8 September 2015 at 18:21, Stefano Stabellini
>> > <stefano.stabellini@eu.citrix.com> wrote:
>> > > The following changes since commit 8611280505119e296757a60711a881341603fa5a:
>> > >
>> > >   target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
>> > >
>> > > are available in the git repository at:
>> > >
>> > >   git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-2015-09-08-tag
>> > >
>> > > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
>> > >
>> > >   xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings. (2015-09-08 15:21:56 +0000)
>> > >
>> > > ----------------------------------------------------------------
>> > > Xen branch xen-2015-09-08
>> > >
>> > > ----------------------------------------------------------------
>> >
>> > Hi. I'm afraid this fails to build on OSX (and probably Windows too,
>> > though that build hasn't run yet):
>> >
>> >   CC    i386-softmmu/hw/i386/pci-assign-load-rom.o
>> > /Users/pm215/src/qemu/hw/i386/pci-assign-load-rom.c:6:10: fatal error:
>> >       'sys/io.h' file not found
>> > #include <sys/io.h>
>> >          ^
>> >   CC    alpha-softmmu/hw/alpha/pci.o
>> > 1 error generated.
>>
>> Tiejun,
>>
>> this is caused by 33d33242b7d802e6c994f3d56ecba96a66465dc3,
>> "hw/pci-assign: split pci-assign.c". Could you please double-check
>> non-Linux builds?
>
> I found another issue introduced by the gfx passthrough series on
> Windows:
>
> ../hw/pci-host/piix.o: In function `host_pci_config_read':
> /root/qemu/hw/pci-host/piix.c:778: undefined reference to `_pread'
>
> It is introduced by:
>
> commit fdb70721ba0496a767137e5505dd27627d19c4a8
> Author: Tiejun Chen <tiejun.chen@intel.com>
> Date:   Wed Jul 15 13:37:43 2015 +0800
>
>      piix: create host bridge to passthrough
>
>
> You might have to replace the pread call with lseek and read.
>

This is also surprising to me. Just see xen_host_pci_config_xxxx() 
inside /hw/xen/xen-host-pci-device.c, there are so many this pread 
usage(). So I really don't understand what's difference between these 
two files.	

Thanks
Tiejun
Peter Maydell Sept. 10, 2015, 8:57 a.m. UTC | #7
On 10 September 2015 at 02:21, Chen, Tiejun <tiejun.chen@intel.com> wrote:
> On 9/10/2015 12:10 AM, Stefano Stabellini wrote:
>> I found another issue introduced by the gfx passthrough series on
>> Windows:
>>
>> ../hw/pci-host/piix.o: In function `host_pci_config_read':
>> /root/qemu/hw/pci-host/piix.c:778: undefined reference to `_pread'
>>
>> It is introduced by:
>>
>> commit fdb70721ba0496a767137e5505dd27627d19c4a8
>> Author: Tiejun Chen <tiejun.chen@intel.com>
>> Date:   Wed Jul 15 13:37:43 2015 +0800
>>
>>      piix: create host bridge to passthrough
>>
>>
>> You might have to replace the pread call with lseek and read.
>>
>
> This is also surprising to me. Just see xen_host_pci_config_xxxx() inside
> /hw/xen/xen-host-pci-device.c, there are so many this pread usage(). So I
> really don't understand what's difference between these two files.

xen-host-pci-device.c is only compiled if CONFIG_XEN_PCI_PASSTHROUGH
was set by configure. That won't be the case on OSX or Windows, where
the Xen headers don't exist.

thanks
-- PMM
Tiejun Chen Sept. 21, 2015, 1:16 a.m. UTC | #8
Stefano,

I have two questions,

#1. Which qemu version is this igd stuff going into? 2.6?
#2. Is this igd stuff going into qemu-xen inside xen? Any plan to go 
into xen 4.6?

Thanks
Tiejun

On 9/9/2015 1:21 AM, Stefano Stabellini wrote:
> The following changes since commit 8611280505119e296757a60711a881341603fa5a:
>
>    target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
>
> are available in the git repository at:
>
>    git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-2015-09-08-tag
>
> for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
>
>    xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings. (2015-09-08 15:21:56 +0000)
>
> ----------------------------------------------------------------
> Xen branch xen-2015-09-08
>
> ----------------------------------------------------------------
> Don Slutz (1):
>        xen-hvm: Add trace to ioreq
>
> Jan Beulich (1):
>        xen/HVM: atomically access pointers in bufioreq handling
>
> Konrad Rzeszutek Wilk (7):
>        xen-hvm: When using xc_domain_add_to_physmap also include errno when reporting
>        xen/pt: Update comments with proper function name.
>        xen/pt: Make xen_pt_msi_set_enable static
>        xen/pt: xen_host_pci_config_read returns -errno, not -1 on failure
>        xen: use errno instead of rc for xc_domain_add_to_physmap
>        xen/pt/msi: Add the register value when printing logging and error messages
>        xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
>
> Michael S. Tsirkin (1):
>        i440fx: make types configurable at run-time
>
> Tiejun Chen (9):
>        pc_init1: pass parameters just with types
>        piix: create host bridge to passthrough
>        hw/pci-assign: split pci-assign.c
>        xen, gfx passthrough: basic graphics passthrough support
>        xen, gfx passthrough: retrieve VGA BIOS to work
>        igd gfx passthrough: create a isa bridge
>        xen, gfx passthrough: register a isa bridge
>        xen, gfx passthrough: register host bridge specific to passthrough
>        xen, gfx passthrough: add opregion mapping
>
>   configure                     |   28 +++++
>   hw/core/machine.c             |   20 +++
>   hw/i386/Makefile.objs         |    1 +
>   hw/i386/kvm/pci-assign.c      |   82 ++-----------
>   hw/i386/pc_piix.c             |  139 ++++++++++++++++++++-
>   hw/i386/pci-assign-load-rom.c |   93 ++++++++++++++
>   hw/pci-host/piix.c            |   91 +++++++++++++-
>   hw/xen/Makefile.objs          |    1 +
>   hw/xen/xen-host-pci-device.c  |    5 +
>   hw/xen/xen-host-pci-device.h  |    1 +
>   hw/xen/xen_pt.c               |   42 ++++++-
>   hw/xen/xen_pt.h               |   22 +++-
>   hw/xen/xen_pt_config_init.c   |   59 ++++++++-
>   hw/xen/xen_pt_graphics.c      |  272 +++++++++++++++++++++++++++++++++++++++++
>   hw/xen/xen_pt_msi.c           |    2 +-
>   include/hw/boards.h           |    1 +
>   include/hw/i386/pc.h          |    9 +-
>   include/hw/pci/pci-assign.h   |   27 ++++
>   include/hw/xen/xen_common.h   |   34 +++++-
>   qemu-options.hx               |    3 +
>   trace-events                  |    7 ++
>   vl.c                          |   10 ++
>   xen-hvm.c                     |   55 +++++++--
>   23 files changed, 891 insertions(+), 113 deletions(-)
>   create mode 100644 hw/i386/pci-assign-load-rom.c
>   create mode 100644 hw/xen/xen_pt_graphics.c
>   create mode 100644 include/hw/pci/pci-assign.h
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
>
Stefano Stabellini Sept. 21, 2015, 4:03 p.m. UTC | #9
It is going to be in QEMU 2.5 and qemu-xen 4.7.

On Mon, 21 Sep 2015, Chen, Tiejun wrote:
> Stefano,
> 
> I have two questions,
> 
> #1. Which qemu version is this igd stuff going into? 2.6?
> #2. Is this igd stuff going into qemu-xen inside xen? Any plan to go into xen
> 4.6?
> 
> Thanks
> Tiejun
> 
> On 9/9/2015 1:21 AM, Stefano Stabellini wrote:
> > The following changes since commit 8611280505119e296757a60711a881341603fa5a:
> > 
> >    target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
> > 
> > are available in the git repository at:
> > 
> >    git://xenbits.xen.org/people/sstabellini/qemu-dm.git
> > tags/xen-2015-09-08-tag
> > 
> > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
> > 
> >    xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
> > (2015-09-08 15:21:56 +0000)
> > 
> > ----------------------------------------------------------------
> > Xen branch xen-2015-09-08
> > 
> > ----------------------------------------------------------------
> > Don Slutz (1):
> >        xen-hvm: Add trace to ioreq
> > 
> > Jan Beulich (1):
> >        xen/HVM: atomically access pointers in bufioreq handling
> > 
> > Konrad Rzeszutek Wilk (7):
> >        xen-hvm: When using xc_domain_add_to_physmap also include errno when
> > reporting
> >        xen/pt: Update comments with proper function name.
> >        xen/pt: Make xen_pt_msi_set_enable static
> >        xen/pt: xen_host_pci_config_read returns -errno, not -1 on failure
> >        xen: use errno instead of rc for xc_domain_add_to_physmap
> >        xen/pt/msi: Add the register value when printing logging and error
> > messages
> >        xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
> > 
> > Michael S. Tsirkin (1):
> >        i440fx: make types configurable at run-time
> > 
> > Tiejun Chen (9):
> >        pc_init1: pass parameters just with types
> >        piix: create host bridge to passthrough
> >        hw/pci-assign: split pci-assign.c
> >        xen, gfx passthrough: basic graphics passthrough support
> >        xen, gfx passthrough: retrieve VGA BIOS to work
> >        igd gfx passthrough: create a isa bridge
> >        xen, gfx passthrough: register a isa bridge
> >        xen, gfx passthrough: register host bridge specific to passthrough
> >        xen, gfx passthrough: add opregion mapping
> > 
> >   configure                     |   28 +++++
> >   hw/core/machine.c             |   20 +++
> >   hw/i386/Makefile.objs         |    1 +
> >   hw/i386/kvm/pci-assign.c      |   82 ++-----------
> >   hw/i386/pc_piix.c             |  139 ++++++++++++++++++++-
> >   hw/i386/pci-assign-load-rom.c |   93 ++++++++++++++
> >   hw/pci-host/piix.c            |   91 +++++++++++++-
> >   hw/xen/Makefile.objs          |    1 +
> >   hw/xen/xen-host-pci-device.c  |    5 +
> >   hw/xen/xen-host-pci-device.h  |    1 +
> >   hw/xen/xen_pt.c               |   42 ++++++-
> >   hw/xen/xen_pt.h               |   22 +++-
> >   hw/xen/xen_pt_config_init.c   |   59 ++++++++-
> >   hw/xen/xen_pt_graphics.c      |  272
> > +++++++++++++++++++++++++++++++++++++++++
> >   hw/xen/xen_pt_msi.c           |    2 +-
> >   include/hw/boards.h           |    1 +
> >   include/hw/i386/pc.h          |    9 +-
> >   include/hw/pci/pci-assign.h   |   27 ++++
> >   include/hw/xen/xen_common.h   |   34 +++++-
> >   qemu-options.hx               |    3 +
> >   trace-events                  |    7 ++
> >   vl.c                          |   10 ++
> >   xen-hvm.c                     |   55 +++++++--
> >   23 files changed, 891 insertions(+), 113 deletions(-)
> >   create mode 100644 hw/i386/pci-assign-load-rom.c
> >   create mode 100644 hw/xen/xen_pt_graphics.c
> >   create mode 100644 include/hw/pci/pci-assign.h
> > 
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xen.org
> > http://lists.xen.org/xen-devel
> > 
>
Tiejun Chen Sept. 28, 2015, 2:01 a.m. UTC | #10
On 9/22/2015 12:03 AM, Stefano Stabellini wrote:
> It is going to be in QEMU 2.5 and qemu-xen 4.7.

Thanks for your reply.

Do we have any possibility of just merging this series into qemu-xen 
4.6? We really want to support IGD passthrough on xen 4.6 if possible :)

Thanks
Tiejun

>
> On Mon, 21 Sep 2015, Chen, Tiejun wrote:
>> Stefano,
>>
>> I have two questions,
>>
>> #1. Which qemu version is this igd stuff going into? 2.6?
>> #2. Is this igd stuff going into qemu-xen inside xen? Any plan to go into xen
>> 4.6?
>>
>> Thanks
>> Tiejun
>>
>> On 9/9/2015 1:21 AM, Stefano Stabellini wrote:
>> > The following changes since commit 8611280505119e296757a60711a881341603fa5a:
>> >
>> >    target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
>> >
>> > are available in the git repository at:
>> >
>> >    git://xenbits.xen.org/people/sstabellini/qemu-dm.git
>> > tags/xen-2015-09-08-tag
>> >
>> > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
>> >
>> >    xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
>> > (2015-09-08 15:21:56 +0000)
>> >
>> > ----------------------------------------------------------------
>> > Xen branch xen-2015-09-08
>> >
>> > ----------------------------------------------------------------
>> > Don Slutz (1):
>> >        xen-hvm: Add trace to ioreq
>> >
>> > Jan Beulich (1):
>> >        xen/HVM: atomically access pointers in bufioreq handling
>> >
>> > Konrad Rzeszutek Wilk (7):
>> >        xen-hvm: When using xc_domain_add_to_physmap also include errno when
>> > reporting
>> >        xen/pt: Update comments with proper function name.
>> >        xen/pt: Make xen_pt_msi_set_enable static
>> >        xen/pt: xen_host_pci_config_read returns -errno, not -1 on failure
>> >        xen: use errno instead of rc for xc_domain_add_to_physmap
>> >        xen/pt/msi: Add the register value when printing logging and error
>> > messages
>> >        xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
>> >
>> > Michael S. Tsirkin (1):
>> >        i440fx: make types configurable at run-time
>> >
>> > Tiejun Chen (9):
>> >        pc_init1: pass parameters just with types
>> >        piix: create host bridge to passthrough
>> >        hw/pci-assign: split pci-assign.c
>> >        xen, gfx passthrough: basic graphics passthrough support
>> >        xen, gfx passthrough: retrieve VGA BIOS to work
>> >        igd gfx passthrough: create a isa bridge
>> >        xen, gfx passthrough: register a isa bridge
>> >        xen, gfx passthrough: register host bridge specific to passthrough
>> >        xen, gfx passthrough: add opregion mapping
>> >
>> >   configure                     |   28 +++++
>> >   hw/core/machine.c             |   20 +++
>> >   hw/i386/Makefile.objs         |    1 +
>> >   hw/i386/kvm/pci-assign.c      |   82 ++-----------
>> >   hw/i386/pc_piix.c             |  139 ++++++++++++++++++++-
>> >   hw/i386/pci-assign-load-rom.c |   93 ++++++++++++++
>> >   hw/pci-host/piix.c            |   91 +++++++++++++-
>> >   hw/xen/Makefile.objs          |    1 +
>> >   hw/xen/xen-host-pci-device.c  |    5 +
>> >   hw/xen/xen-host-pci-device.h  |    1 +
>> >   hw/xen/xen_pt.c               |   42 ++++++-
>> >   hw/xen/xen_pt.h               |   22 +++-
>> >   hw/xen/xen_pt_config_init.c   |   59 ++++++++-
>> >   hw/xen/xen_pt_graphics.c      |  272
>> > +++++++++++++++++++++++++++++++++++++++++
>> >   hw/xen/xen_pt_msi.c           |    2 +-
>> >   include/hw/boards.h           |    1 +
>> >   include/hw/i386/pc.h          |    9 +-
>> >   include/hw/pci/pci-assign.h   |   27 ++++
>> >   include/hw/xen/xen_common.h   |   34 +++++-
>> >   qemu-options.hx               |    3 +
>> >   trace-events                  |    7 ++
>> >   vl.c                          |   10 ++
>> >   xen-hvm.c                     |   55 +++++++--
>> >   23 files changed, 891 insertions(+), 113 deletions(-)
>> >   create mode 100644 hw/i386/pci-assign-load-rom.c
>> >   create mode 100644 hw/xen/xen_pt_graphics.c
>> >   create mode 100644 include/hw/pci/pci-assign.h
>> >
>> > _______________________________________________
>> > Xen-devel mailing list
>> > Xen-devel@lists.xen.org
>> > http://lists.xen.org/xen-devel
>> >
>>
>
>
Stefano Stabellini Sept. 28, 2015, 10:01 a.m. UTC | #11
No, unfortunately it is not possible at this stage of the release cycle.
But users can still use QEMU 2.5 (as soon as it is released, which
should be in a couple of months) with Xen 4.6 as there is not a strong
tie between QEMU releases and Xen releases.

On Mon, 28 Sep 2015, Chen, Tiejun wrote:
> On 9/22/2015 12:03 AM, Stefano Stabellini wrote:
> > It is going to be in QEMU 2.5 and qemu-xen 4.7.
> 
> Thanks for your reply.
> 
> Do we have any possibility of just merging this series into qemu-xen 4.6? We
> really want to support IGD passthrough on xen 4.6 if possible :)
> 
> Thanks
> Tiejun
> 
> > 
> > On Mon, 21 Sep 2015, Chen, Tiejun wrote:
> > > Stefano,
> > > 
> > > I have two questions,
> > > 
> > > #1. Which qemu version is this igd stuff going into? 2.6?
> > > #2. Is this igd stuff going into qemu-xen inside xen? Any plan to go into
> > > xen
> > > 4.6?
> > > 
> > > Thanks
> > > Tiejun
> > > 
> > > On 9/9/2015 1:21 AM, Stefano Stabellini wrote:
> > > > The following changes since commit
> > > 8611280505119e296757a60711a881341603fa5a:
> > > >
> > > >    target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
> > > >
> > > > are available in the git repository at:
> > > >
> > > >    git://xenbits.xen.org/people/sstabellini/qemu-dm.git
> > > > tags/xen-2015-09-08-tag
> > > >
> > > > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
> > > >
> > > >    xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
> > > > (2015-09-08 15:21:56 +0000)
> > > >
> > > > ----------------------------------------------------------------
> > > > Xen branch xen-2015-09-08
> > > >
> > > > ----------------------------------------------------------------
> > > > Don Slutz (1):
> > > >        xen-hvm: Add trace to ioreq
> > > >
> > > > Jan Beulich (1):
> > > >        xen/HVM: atomically access pointers in bufioreq handling
> > > >
> > > > Konrad Rzeszutek Wilk (7):
> > > >        xen-hvm: When using xc_domain_add_to_physmap also include errno
> > > when
> > > > reporting
> > > >        xen/pt: Update comments with proper function name.
> > > >        xen/pt: Make xen_pt_msi_set_enable static
> > > >        xen/pt: xen_host_pci_config_read returns -errno, not -1 on
> > > failure
> > > >        xen: use errno instead of rc for xc_domain_add_to_physmap
> > > >        xen/pt/msi: Add the register value when printing logging and
> > > error
> > > > messages
> > > >        xen/pt: Use XEN_PT_LOG properly to guard against compiler
> > > warnings.
> > > >
> > > > Michael S. Tsirkin (1):
> > > >        i440fx: make types configurable at run-time
> > > >
> > > > Tiejun Chen (9):
> > > >        pc_init1: pass parameters just with types
> > > >        piix: create host bridge to passthrough
> > > >        hw/pci-assign: split pci-assign.c
> > > >        xen, gfx passthrough: basic graphics passthrough support
> > > >        xen, gfx passthrough: retrieve VGA BIOS to work
> > > >        igd gfx passthrough: create a isa bridge
> > > >        xen, gfx passthrough: register a isa bridge
> > > >        xen, gfx passthrough: register host bridge specific to
> > > passthrough
> > > >        xen, gfx passthrough: add opregion mapping
> > > >
> > > >   configure                     |   28 +++++
> > > >   hw/core/machine.c             |   20 +++
> > > >   hw/i386/Makefile.objs         |    1 +
> > > >   hw/i386/kvm/pci-assign.c      |   82 ++-----------
> > > >   hw/i386/pc_piix.c             |  139 ++++++++++++++++++++-
> > > >   hw/i386/pci-assign-load-rom.c |   93 ++++++++++++++
> > > >   hw/pci-host/piix.c            |   91 +++++++++++++-
> > > >   hw/xen/Makefile.objs          |    1 +
> > > >   hw/xen/xen-host-pci-device.c  |    5 +
> > > >   hw/xen/xen-host-pci-device.h  |    1 +
> > > >   hw/xen/xen_pt.c               |   42 ++++++-
> > > >   hw/xen/xen_pt.h               |   22 +++-
> > > >   hw/xen/xen_pt_config_init.c   |   59 ++++++++-
> > > >   hw/xen/xen_pt_graphics.c      |  272
> > > > +++++++++++++++++++++++++++++++++++++++++
> > > >   hw/xen/xen_pt_msi.c           |    2 +-
> > > >   include/hw/boards.h           |    1 +
> > > >   include/hw/i386/pc.h          |    9 +-
> > > >   include/hw/pci/pci-assign.h   |   27 ++++
> > > >   include/hw/xen/xen_common.h   |   34 +++++-
> > > >   qemu-options.hx               |    3 +
> > > >   trace-events                  |    7 ++
> > > >   vl.c                          |   10 ++
> > > >   xen-hvm.c                     |   55 +++++++--
> > > >   23 files changed, 891 insertions(+), 113 deletions(-)
> > > >   create mode 100644 hw/i386/pci-assign-load-rom.c
> > > >   create mode 100644 hw/xen/xen_pt_graphics.c
> > > >   create mode 100644 include/hw/pci/pci-assign.h
> > > >
> > > > _______________________________________________
> > > > Xen-devel mailing list
> > > > Xen-devel@lists.xen.org
> > > > http://lists.xen.org/xen-devel
> > > >
> > > 
> > 
> > 
>
Pasi Kärkkäinen Sept. 28, 2015, 5:20 p.m. UTC | #12
Hi,

On Mon, Sep 28, 2015 at 11:01:11AM +0100, Stefano Stabellini wrote:
> No, unfortunately it is not possible at this stage of the release cycle.
> But users can still use QEMU 2.5 (as soon as it is released, which
> should be in a couple of months) with Xen 4.6 as there is not a strong
> tie between QEMU releases and Xen releases.
> 

Is backport to qemu-xen-4.6 possible *after* Xen 4.6.0 has been released? So basicly for Xen 4.6.1.

There's value in having this feature available out-of-the-box, with stock Xen, without having to replace qemu with a custom build/version.


Thanks,

-- Pasi

> On Mon, 28 Sep 2015, Chen, Tiejun wrote:
> > On 9/22/2015 12:03 AM, Stefano Stabellini wrote:
> > > It is going to be in QEMU 2.5 and qemu-xen 4.7.
> > 
> > Thanks for your reply.
> > 
> > Do we have any possibility of just merging this series into qemu-xen 4.6? We
> > really want to support IGD passthrough on xen 4.6 if possible :)
> > 
> > Thanks
> > Tiejun
> > 
> > > 
> > > On Mon, 21 Sep 2015, Chen, Tiejun wrote:
> > > > Stefano,
> > > > 
> > > > I have two questions,
> > > > 
> > > > #1. Which qemu version is this igd stuff going into? 2.6?
> > > > #2. Is this igd stuff going into qemu-xen inside xen? Any plan to go into
> > > > xen
> > > > 4.6?
> > > > 
> > > > Thanks
> > > > Tiejun
> > > > 
> > > > On 9/9/2015 1:21 AM, Stefano Stabellini wrote:
> > > > > The following changes since commit
> > > > 8611280505119e296757a60711a881341603fa5a:
> > > > >
> > > > >    target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
> > > > >
> > > > > are available in the git repository at:
> > > > >
> > > > >    git://xenbits.xen.org/people/sstabellini/qemu-dm.git
> > > > > tags/xen-2015-09-08-tag
> > > > >
> > > > > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
> > > > >
> > > > >    xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
> > > > > (2015-09-08 15:21:56 +0000)
> > > > >
> > > > > ----------------------------------------------------------------
> > > > > Xen branch xen-2015-09-08
> > > > >
> > > > > ----------------------------------------------------------------
> > > > > Don Slutz (1):
> > > > >        xen-hvm: Add trace to ioreq
> > > > >
> > > > > Jan Beulich (1):
> > > > >        xen/HVM: atomically access pointers in bufioreq handling
> > > > >
> > > > > Konrad Rzeszutek Wilk (7):
> > > > >        xen-hvm: When using xc_domain_add_to_physmap also include errno
> > > > when
> > > > > reporting
> > > > >        xen/pt: Update comments with proper function name.
> > > > >        xen/pt: Make xen_pt_msi_set_enable static
> > > > >        xen/pt: xen_host_pci_config_read returns -errno, not -1 on
> > > > failure
> > > > >        xen: use errno instead of rc for xc_domain_add_to_physmap
> > > > >        xen/pt/msi: Add the register value when printing logging and
> > > > error
> > > > > messages
> > > > >        xen/pt: Use XEN_PT_LOG properly to guard against compiler
> > > > warnings.
> > > > >
> > > > > Michael S. Tsirkin (1):
> > > > >        i440fx: make types configurable at run-time
> > > > >
> > > > > Tiejun Chen (9):
> > > > >        pc_init1: pass parameters just with types
> > > > >        piix: create host bridge to passthrough
> > > > >        hw/pci-assign: split pci-assign.c
> > > > >        xen, gfx passthrough: basic graphics passthrough support
> > > > >        xen, gfx passthrough: retrieve VGA BIOS to work
> > > > >        igd gfx passthrough: create a isa bridge
> > > > >        xen, gfx passthrough: register a isa bridge
> > > > >        xen, gfx passthrough: register host bridge specific to
> > > > passthrough
> > > > >        xen, gfx passthrough: add opregion mapping
> > > > >
> > > > >   configure                     |   28 +++++
> > > > >   hw/core/machine.c             |   20 +++
> > > > >   hw/i386/Makefile.objs         |    1 +
> > > > >   hw/i386/kvm/pci-assign.c      |   82 ++-----------
> > > > >   hw/i386/pc_piix.c             |  139 ++++++++++++++++++++-
> > > > >   hw/i386/pci-assign-load-rom.c |   93 ++++++++++++++
> > > > >   hw/pci-host/piix.c            |   91 +++++++++++++-
> > > > >   hw/xen/Makefile.objs          |    1 +
> > > > >   hw/xen/xen-host-pci-device.c  |    5 +
> > > > >   hw/xen/xen-host-pci-device.h  |    1 +
> > > > >   hw/xen/xen_pt.c               |   42 ++++++-
> > > > >   hw/xen/xen_pt.h               |   22 +++-
> > > > >   hw/xen/xen_pt_config_init.c   |   59 ++++++++-
> > > > >   hw/xen/xen_pt_graphics.c      |  272
> > > > > +++++++++++++++++++++++++++++++++++++++++
> > > > >   hw/xen/xen_pt_msi.c           |    2 +-
> > > > >   include/hw/boards.h           |    1 +
> > > > >   include/hw/i386/pc.h          |    9 +-
> > > > >   include/hw/pci/pci-assign.h   |   27 ++++
> > > > >   include/hw/xen/xen_common.h   |   34 +++++-
> > > > >   qemu-options.hx               |    3 +
> > > > >   trace-events                  |    7 ++
> > > > >   vl.c                          |   10 ++
> > > > >   xen-hvm.c                     |   55 +++++++--
> > > > >   23 files changed, 891 insertions(+), 113 deletions(-)
> > > > >   create mode 100644 hw/i386/pci-assign-load-rom.c
> > > > >   create mode 100644 hw/xen/xen_pt_graphics.c
> > > > >   create mode 100644 include/hw/pci/pci-assign.h
> > > > >
> > > > > _______________________________________________
> > > > > 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
Stefano Stabellini Sept. 29, 2015, 9:53 a.m. UTC | #13
On Mon, 28 Sep 2015, Pasi Kärkkäinen wrote:
> Hi,
> 
> On Mon, Sep 28, 2015 at 11:01:11AM +0100, Stefano Stabellini wrote:
> > No, unfortunately it is not possible at this stage of the release cycle.
> > But users can still use QEMU 2.5 (as soon as it is released, which
> > should be in a couple of months) with Xen 4.6 as there is not a strong
> > tie between QEMU releases and Xen releases.
> > 
> 
> Is backport to qemu-xen-4.6 possible *after* Xen 4.6.0 has been released? So basicly for Xen 4.6.1.

I don't think so, this is not backport material (which is basically only
fixes).


> There's value in having this feature available out-of-the-box, with stock Xen, without having to replace qemu with a custom build/version.

Of course, and it will be, in Xen 4.7.


> Thanks,
> 
> -- Pasi
> 
> > On Mon, 28 Sep 2015, Chen, Tiejun wrote:
> > > On 9/22/2015 12:03 AM, Stefano Stabellini wrote:
> > > > It is going to be in QEMU 2.5 and qemu-xen 4.7.
> > > 
> > > Thanks for your reply.
> > > 
> > > Do we have any possibility of just merging this series into qemu-xen 4.6? We
> > > really want to support IGD passthrough on xen 4.6 if possible :)
> > > 
> > > Thanks
> > > Tiejun
> > > 
> > > > 
> > > > On Mon, 21 Sep 2015, Chen, Tiejun wrote:
> > > > > Stefano,
> > > > > 
> > > > > I have two questions,
> > > > > 
> > > > > #1. Which qemu version is this igd stuff going into? 2.6?
> > > > > #2. Is this igd stuff going into qemu-xen inside xen? Any plan to go into
> > > > > xen
> > > > > 4.6?
> > > > > 
> > > > > Thanks
> > > > > Tiejun
> > > > > 
> > > > > On 9/9/2015 1:21 AM, Stefano Stabellini wrote:
> > > > > > The following changes since commit
> > > > > 8611280505119e296757a60711a881341603fa5a:
> > > > > >
> > > > > >    target-microblaze: Use setcond for pcmp* (2015-09-08 08:49:33 +0200)
> > > > > >
> > > > > > are available in the git repository at:
> > > > > >
> > > > > >    git://xenbits.xen.org/people/sstabellini/qemu-dm.git
> > > > > > tags/xen-2015-09-08-tag
> > > > > >
> > > > > > for you to fetch changes up to ba2250ad148997b1352aba976aac66b55410e7e4:
> > > > > >
> > > > > >    xen/pt: Use XEN_PT_LOG properly to guard against compiler warnings.
> > > > > > (2015-09-08 15:21:56 +0000)
> > > > > >
> > > > > > ----------------------------------------------------------------
> > > > > > Xen branch xen-2015-09-08
> > > > > >
> > > > > > ----------------------------------------------------------------
> > > > > > Don Slutz (1):
> > > > > >        xen-hvm: Add trace to ioreq
> > > > > >
> > > > > > Jan Beulich (1):
> > > > > >        xen/HVM: atomically access pointers in bufioreq handling
> > > > > >
> > > > > > Konrad Rzeszutek Wilk (7):
> > > > > >        xen-hvm: When using xc_domain_add_to_physmap also include errno
> > > > > when
> > > > > > reporting
> > > > > >        xen/pt: Update comments with proper function name.
> > > > > >        xen/pt: Make xen_pt_msi_set_enable static
> > > > > >        xen/pt: xen_host_pci_config_read returns -errno, not -1 on
> > > > > failure
> > > > > >        xen: use errno instead of rc for xc_domain_add_to_physmap
> > > > > >        xen/pt/msi: Add the register value when printing logging and
> > > > > error
> > > > > > messages
> > > > > >        xen/pt: Use XEN_PT_LOG properly to guard against compiler
> > > > > warnings.
> > > > > >
> > > > > > Michael S. Tsirkin (1):
> > > > > >        i440fx: make types configurable at run-time
> > > > > >
> > > > > > Tiejun Chen (9):
> > > > > >        pc_init1: pass parameters just with types
> > > > > >        piix: create host bridge to passthrough
> > > > > >        hw/pci-assign: split pci-assign.c
> > > > > >        xen, gfx passthrough: basic graphics passthrough support
> > > > > >        xen, gfx passthrough: retrieve VGA BIOS to work
> > > > > >        igd gfx passthrough: create a isa bridge
> > > > > >        xen, gfx passthrough: register a isa bridge
> > > > > >        xen, gfx passthrough: register host bridge specific to
> > > > > passthrough
> > > > > >        xen, gfx passthrough: add opregion mapping
> > > > > >
> > > > > >   configure                     |   28 +++++
> > > > > >   hw/core/machine.c             |   20 +++
> > > > > >   hw/i386/Makefile.objs         |    1 +
> > > > > >   hw/i386/kvm/pci-assign.c      |   82 ++-----------
> > > > > >   hw/i386/pc_piix.c             |  139 ++++++++++++++++++++-
> > > > > >   hw/i386/pci-assign-load-rom.c |   93 ++++++++++++++
> > > > > >   hw/pci-host/piix.c            |   91 +++++++++++++-
> > > > > >   hw/xen/Makefile.objs          |    1 +
> > > > > >   hw/xen/xen-host-pci-device.c  |    5 +
> > > > > >   hw/xen/xen-host-pci-device.h  |    1 +
> > > > > >   hw/xen/xen_pt.c               |   42 ++++++-
> > > > > >   hw/xen/xen_pt.h               |   22 +++-
> > > > > >   hw/xen/xen_pt_config_init.c   |   59 ++++++++-
> > > > > >   hw/xen/xen_pt_graphics.c      |  272
> > > > > > +++++++++++++++++++++++++++++++++++++++++
> > > > > >   hw/xen/xen_pt_msi.c           |    2 +-
> > > > > >   include/hw/boards.h           |    1 +
> > > > > >   include/hw/i386/pc.h          |    9 +-
> > > > > >   include/hw/pci/pci-assign.h   |   27 ++++
> > > > > >   include/hw/xen/xen_common.h   |   34 +++++-
> > > > > >   qemu-options.hx               |    3 +
> > > > > >   trace-events                  |    7 ++
> > > > > >   vl.c                          |   10 ++
> > > > > >   xen-hvm.c                     |   55 +++++++--
> > > > > >   23 files changed, 891 insertions(+), 113 deletions(-)
> > > > > >   create mode 100644 hw/i386/pci-assign-load-rom.c
> > > > > >   create mode 100644 hw/xen/xen_pt_graphics.c
> > > > > >   create mode 100644 include/hw/pci/pci-assign.h
> > > > > >
> > > > > > _______________________________________________
> > > > > > 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
>