Isaku Yamahata
2010-Apr-05 07:57 UTC
[Xen-devel] [PATCH] qemu-dm: pass-through: fix header type register emulation.
This patch fixes the emulation of latency timer and header type. The change set of cc1a204423475ff7a918b11d78b9ae637f320e23 deleted the header type register emulation. On the other hand, the change set of ec5e52d5cb2e6f8851c345b7c3095fe2030fff9c tries to update header type emulation, however it wrongly touches latency timer emulation part. I think this was caused by mis-merging. This patch sorts it out. Cc: Dexuan Cui <dexuan.cui@intel.com> Cc: Masaki Kanno <kanno.masaki@jp.fujitsu.com> Cc: Simon Horman <horms@verge.net.au> Cc: Ian Jackson <Ian.Jackson@eu.citrix.com> Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> --- This is compile-tested only. Please review/test it --- hw/pass-through.c | 14 +++++++++++++- 1 files changed, 13 insertions(+), 1 deletions(-) diff --git a/hw/pass-through.c b/hw/pass-through.c index fadd358..5a76e8d 100644 --- a/hw/pass-through.c +++ b/hw/pass-through.c @@ -329,11 +329,23 @@ static struct pt_reg_info_tbl pt_emu_reg_header0_tbl[] = { .init_val = 0x00, .ro_mask = 0x00, .emu_mask = 0xFF, - .init = pt_header_type_reg_init, + .init = pt_common_reg_init, .u.b.read = pt_byte_reg_read, .u.b.write = pt_byte_reg_write, .u.b.restore = pt_byte_reg_restore, }, + /* Header Type reg */ + { + .offset = PCI_HEADER_TYPE, + .size = 1, + .init_val = 0x00, + .ro_mask = 0xFF, + .emu_mask = 0x00, + .init = pt_header_type_reg_init, + .u.b.read = pt_byte_reg_read, + .u.b.write = pt_byte_reg_write, + .u.b.restore = NULL, + }, /* Interrupt Line reg */ { .offset = PCI_INTERRUPT_LINE, -- 1.6.6.1 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Simon Horman
2010-Apr-06 03:06 UTC
Re: [Xen-devel] [PATCH] qemu-dm: pass-through: fix header type register emulation.
On Mon, Apr 05, 2010 at 04:57:53PM +0900, Isaku Yamahata wrote:> > This patch fixes the emulation of latency timer and header type. > The change set of cc1a204423475ff7a918b11d78b9ae637f320e23 > deleted the header type register emulation. > On the other hand, the change set of ec5e52d5cb2e6f8851c345b7c3095fe2030fff9c > tries to update header type emulation, however it wrongly > touches latency timer emulation part. > I think this was caused by mis-merging. This patch sorts it out. > > Cc: Dexuan Cui <dexuan.cui@intel.com> > Cc: Masaki Kanno <kanno.masaki@jp.fujitsu.com> > Cc: Simon Horman <horms@verge.net.au> > Cc: Ian Jackson <Ian.Jackson@eu.citrix.com> > Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>Acked-by: Simon Horman <horms@verge.net.au>> --- > This is compile-tested only. > Please review/test it > --- > hw/pass-through.c | 14 +++++++++++++- > 1 files changed, 13 insertions(+), 1 deletions(-) > > diff --git a/hw/pass-through.c b/hw/pass-through.c > index fadd358..5a76e8d 100644 > --- a/hw/pass-through.c > +++ b/hw/pass-through.c > @@ -329,11 +329,23 @@ static struct pt_reg_info_tbl pt_emu_reg_header0_tbl[] = { > .init_val = 0x00, > .ro_mask = 0x00, > .emu_mask = 0xFF, > - .init = pt_header_type_reg_init, > + .init = pt_common_reg_init, > .u.b.read = pt_byte_reg_read, > .u.b.write = pt_byte_reg_write, > .u.b.restore = pt_byte_reg_restore, > }, > + /* Header Type reg */ > + { > + .offset = PCI_HEADER_TYPE, > + .size = 1, > + .init_val = 0x00, > + .ro_mask = 0xFF, > + .emu_mask = 0x00, > + .init = pt_header_type_reg_init, > + .u.b.read = pt_byte_reg_read, > + .u.b.write = pt_byte_reg_write, > + .u.b.restore = NULL, > + }, > /* Interrupt Line reg */ > { > .offset = PCI_INTERRUPT_LINE, > -- > 1.6.6.1 > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Ian Jackson
2010-Apr-06 14:38 UTC
Re: [Xen-devel] [PATCH] qemu-dm: pass-through: fix header type register emulation.
Simon Horman writes ("Re: [Xen-devel] [PATCH] qemu-dm: pass-through: fix header type register emulation."):> Acked-by: Simon Horman <horms@verge.net.au>Thanks. I assume this is for -unstable, not for the imminent 4.0.0. Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel