RE: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-05 Thread Zhang, Xiantao
Isaku Yamahata wrote: On Thu, Mar 05, 2009 at 11:55:10AM +0800, Zhang, Xiantao wrote: Isaku Yamahata wrote: On Wed, Mar 04, 2009 at 05:26:41PM +0800, Zhang, Xiantao wrote: So far, we just found the msi-x case. Maybe we will add msi-x support later, so this fix is also required. Okay,

Re: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-05 Thread Isaku Yamahata
On Fri, Mar 06, 2009 at 10:11:42AM +0800, Zhang, Xiantao wrote: Isaku Yamahata wrote: On Thu, Mar 05, 2009 at 11:55:10AM +0800, Zhang, Xiantao wrote: Isaku Yamahata wrote: On Wed, Mar 04, 2009 at 05:26:41PM +0800, Zhang, Xiantao wrote: So far, we just found the msi-x case. Maybe we will

RE: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-04 Thread Zhang, Xiantao
Isaku Yamahata wrote: On Tue, Mar 03, 2009 at 05:32:42PM +0800, Zhang, Xiantao wrote: Isaku Yamahata wrote: Could you elaborate on the concrete issue which you're seeing? I guess the issue occurs when passed through pci device is unplugged. But in that case, the region was occupied by the

Re: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-04 Thread Isaku Yamahata
On Wed, Mar 04, 2009 at 05:26:41PM +0800, Zhang, Xiantao wrote: So far, we just found the msi-x case. Maybe we will add msi-x support later, so this fix is also required. Okay, makes sense. And why GPFN_LOW_MMIO independently of addr? Shouldn't it be aware of io_ranges[]? For the

RE: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-04 Thread Zhang, Xiantao
Isaku Yamahata wrote: On Wed, Mar 04, 2009 at 05:26:41PM +0800, Zhang, Xiantao wrote: So far, we just found the msi-x case. Maybe we will add msi-x support later, so this fix is also required. Okay, makes sense. And why GPFN_LOW_MMIO independently of addr? Shouldn't it be aware of

Re: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-04 Thread Isaku Yamahata
On Thu, Mar 05, 2009 at 11:55:10AM +0800, Zhang, Xiantao wrote: Isaku Yamahata wrote: On Wed, Mar 04, 2009 at 05:26:41PM +0800, Zhang, Xiantao wrote: So far, we just found the msi-x case. Maybe we will add msi-x support later, so this fix is also required. Okay, makes sense. And

Re: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-03 Thread Isaku Yamahata
Could you elaborate on the concrete issue which you're seeing? I guess the issue occurs when passed through pci device is unplugged. But in that case, the region was occupied by the device so that qemu haven't seen io on the area anyway. And why GPFN_LOW_MMIO independently of addr? Shouldn't it

RE: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-03 Thread Zhang, Xiantao
Isaku Yamahata wrote: Could you elaborate on the concrete issue which you're seeing? I guess the issue occurs when passed through pci device is unplugged. But in that case, the region was occupied by the device so that qemu haven't seen io on the area anyway. For assigning a device to a hvm

Re: [Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-03 Thread Isaku Yamahata
On Tue, Mar 03, 2009 at 05:32:42PM +0800, Zhang, Xiantao wrote: Isaku Yamahata wrote: Could you elaborate on the concrete issue which you're seeing? I guess the issue occurs when passed through pci device is unplugged. But in that case, the region was occupied by the device so that qemu

[Xen-ia64-devel] [PATCH]Fix the logic when deassign the mmio ranges for vti-domain.

2009-03-02 Thread Zhang, Xiantao
PATCH: Fix the logic when deassign the mmio ranges for vti-domain. When de-assign the mmio range, it should resume its original value for p2m value, otherwise, it may fail to determin mmio range's type. Signed-off-by: Xiantao Zhang xiantao.zh...@intel.com diff -r 67f2e14613ef