Re: [Xen-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-12 Thread Jan Beulich
>>> On 05.08.15 at 04:02, wrote: > @@ -491,8 +474,9 @@ static int xen_pt_bar_reg_write(XenPCIPassthroughState > *s, XenPTReg *cfg_entry, > bar_ro_mask = XEN_PT_BAR_IO_RO_MASK | (r_size - 1); > break; > case XEN_PT_BAR_FLAG_UPPER: > +r = &d->io_regions[index-1]; Per

Re: [Xen-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-06 Thread Wu, Feng
> -Original Message- > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com] > Sent: Thursday, August 06, 2015 6:43 PM > To: Wu, Feng > Cc: stefano.stabell...@eu.citrix.com; xen-de...@lists.xensource.com; > qemu-de...@nongnu.org > Subject: Re: [PATCH v1] xenpt: Properly handl

Re: [Xen-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-06 Thread Stefano Stabellini
On Wed, 5 Aug 2015, Feng Wu wrote: > This patch corrects a logic error when handling 64-bt bar with > more than 4G size. > > With 64-bit Bar, it has two items in PCIDevice: io_regions[x] > and io_regions[x+1], io_regions[x] has all the informations for > this BAR, while io_regions[x+1] contains no

[Xen-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-04 Thread Feng Wu
This patch corrects a logic error when handling 64-bt bar with more than 4G size. With 64-bit Bar, it has two items in PCIDevice: io_regions[x] and io_regions[x+1], io_regions[x] has all the informations for this BAR, while io_regions[x+1] contains nothing, so we need to get the size from io_regio