qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* Re: [Qemu-devel] [PATCH v3] xen-hvm: ignore background I/O sections
       [not found] <1462811480-16295-1-git-send-email-paul.durrant@citrix.com>
@ 2016-05-25 15:52 ` Anthony PERARD
  2016-05-25 18:04   ` Paolo Bonzini
  0 siblings, 1 reply; 3+ messages in thread
From: Anthony PERARD @ 2016-05-25 15:52 UTC (permalink / raw)
  To: Paul Durrant; +Cc: qemu-devel, xen-devel, Stefano Stabellini, Paolo Bonzini

On Mon, May 09, 2016 at 05:31:20PM +0100, Paul Durrant wrote:
> Since Xen will correctly handle accesses to unimplemented I/O ports (by
> returning all 1's for reads and ignoring writes) there is no need for
> QEMU to register backgroud I/O sections.
> 
> This patch therefore adds checks to xen_io_add/del so that sections with
> memory-region ops pointing at 'unassigned_io_ops' are ignored.
> 
> Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
> Cc: Stefano Stabellini <sstabellini@kernel.org>
> Cc: Anthony Perard <anthony.perard@citrix.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>

Acked-by: Anthony PERARD <anthony.perard@citrix.com>

-- 
Anthony PERARD

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH v3] xen-hvm: ignore background I/O sections
  2016-05-25 15:52 ` [Qemu-devel] [PATCH v3] xen-hvm: ignore background I/O sections Anthony PERARD
@ 2016-05-25 18:04   ` Paolo Bonzini
  2016-05-26  9:09     ` Stefano Stabellini
  0 siblings, 1 reply; 3+ messages in thread
From: Paolo Bonzini @ 2016-05-25 18:04 UTC (permalink / raw)
  To: Anthony PERARD; +Cc: Paul Durrant, qemu-devel, xen-devel, Stefano Stabellini



----- Original Message -----
> From: "Anthony PERARD" <anthony.perard@citrix.com>
> To: "Paul Durrant" <paul.durrant@citrix.com>
> Cc: qemu-devel@nongnu.org, xen-devel@lists.xenproject.org, "Stefano Stabellini" <sstabellini@kernel.org>, "Paolo
> Bonzini" <pbonzini@redhat.com>
> Sent: Wednesday, May 25, 2016 5:52:32 PM
> Subject: Re: [PATCH v3] xen-hvm: ignore background I/O sections
> 
> On Mon, May 09, 2016 at 05:31:20PM +0100, Paul Durrant wrote:
> > Since Xen will correctly handle accesses to unimplemented I/O ports (by
> > returning all 1's for reads and ignoring writes) there is no need for
> > QEMU to register backgroud I/O sections.
> > 
> > This patch therefore adds checks to xen_io_add/del so that sections with
> > memory-region ops pointing at 'unassigned_io_ops' are ignored.
> > 
> > Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
> > Cc: Stefano Stabellini <sstabellini@kernel.org>
> > Cc: Anthony Perard <anthony.perard@citrix.com>
> > Cc: Paolo Bonzini <pbonzini@redhat.com>
> 
> Acked-by: Anthony PERARD <anthony.perard@citrix.com>

Do you have a signed GPG key or do I need to apply this for you?

Thanks,

Paolo

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH v3] xen-hvm: ignore background I/O sections
  2016-05-25 18:04   ` Paolo Bonzini
@ 2016-05-26  9:09     ` Stefano Stabellini
  0 siblings, 0 replies; 3+ messages in thread
From: Stefano Stabellini @ 2016-05-26  9:09 UTC (permalink / raw)
  To: Paolo Bonzini
  Cc: Anthony PERARD, Paul Durrant, qemu-devel, xen-devel,
	Stefano Stabellini

On Wed, 25 May 2016, Paolo Bonzini wrote:
> > From: "Anthony PERARD" <anthony.perard@citrix.com>
> > To: "Paul Durrant" <paul.durrant@citrix.com>
> > Cc: qemu-devel@nongnu.org, xen-devel@lists.xenproject.org, "Stefano Stabellini" <sstabellini@kernel.org>, "Paolo
> > Bonzini" <pbonzini@redhat.com>
> > Sent: Wednesday, May 25, 2016 5:52:32 PM
> > Subject: Re: [PATCH v3] xen-hvm: ignore background I/O sections
> > 
> > On Mon, May 09, 2016 at 05:31:20PM +0100, Paul Durrant wrote:
> > > Since Xen will correctly handle accesses to unimplemented I/O ports (by
> > > returning all 1's for reads and ignoring writes) there is no need for
> > > QEMU to register backgroud I/O sections.
> > > 
> > > This patch therefore adds checks to xen_io_add/del so that sections with
> > > memory-region ops pointing at 'unassigned_io_ops' are ignored.
> > > 
> > > Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
> > > Cc: Stefano Stabellini <sstabellini@kernel.org>
> > > Cc: Anthony Perard <anthony.perard@citrix.com>
> > > Cc: Paolo Bonzini <pbonzini@redhat.com>
> > 
> > Acked-by: Anthony PERARD <anthony.perard@citrix.com>
> 
> Do you have a signed GPG key or do I need to apply this for you?

I was going to say, I'll just queue this up in my tree, but I don't have
any other commits at this time, so if you are OK with applying this, I
am fine with it too.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-05-26  9:09 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1462811480-16295-1-git-send-email-paul.durrant@citrix.com>
2016-05-25 15:52 ` [Qemu-devel] [PATCH v3] xen-hvm: ignore background I/O sections Anthony PERARD
2016-05-25 18:04   ` Paolo Bonzini
2016-05-26  9:09     ` Stefano Stabellini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).