* [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12
@ 2017-12-08 4:26 Peter Xu
2017-12-08 4:26 ` [Qemu-devel] [PATCH 1/2] intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX Peter Xu
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Peter Xu @ 2017-12-08 4:26 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S . Tsirkin, peterx, Jason Wang
Please review. Thanks,
Peter Xu (2):
intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX
intel_iommu: fix error param in string
hw/i386/intel_iommu.c | 12 ++++++------
include/hw/i386/x86-iommu.h | 1 -
2 files changed, 6 insertions(+), 7 deletions(-)
--
2.14.3
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Qemu-devel] [PATCH 1/2] intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX
2017-12-08 4:26 [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
@ 2017-12-08 4:26 ` Peter Xu
2017-12-08 6:50 ` Liu, Yi L
2017-12-08 4:26 ` [Qemu-devel] [PATCH 2/2] intel_iommu: fix error param in string Peter Xu
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Peter Xu @ 2017-12-08 4:26 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S . Tsirkin, peterx, Jason Wang
We have PCI_DEVFN_MAX now.
Signed-off-by: Peter Xu <peterx@redhat.com>
---
hw/i386/intel_iommu.c | 10 +++++-----
include/hw/i386/x86-iommu.h | 1 -
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
index 3a5bb0bc2e..c1fa08d205 100644
--- a/hw/i386/intel_iommu.c
+++ b/hw/i386/intel_iommu.c
@@ -186,7 +186,7 @@ static void vtd_reset_context_cache(IntelIOMMUState *s)
g_hash_table_iter_init(&bus_it, s->vtd_as_by_busptr);
while (g_hash_table_iter_next (&bus_it, NULL, (void**)&vtd_bus)) {
- for (devfn_it = 0; devfn_it < X86_IOMMU_PCI_DEVFN_MAX; ++devfn_it) {
+ for (devfn_it = 0; devfn_it < PCI_DEVFN_MAX; ++devfn_it) {
vtd_as = vtd_bus->dev_as[devfn_it];
if (!vtd_as) {
continue;
@@ -1002,7 +1002,7 @@ static void vtd_switch_address_space_all(IntelIOMMUState *s)
g_hash_table_iter_init(&iter, s->vtd_as_by_busptr);
while (g_hash_table_iter_next(&iter, NULL, (void **)&vtd_bus)) {
- for (i = 0; i < X86_IOMMU_PCI_DEVFN_MAX; i++) {
+ for (i = 0; i < PCI_DEVFN_MAX; i++) {
if (!vtd_bus->dev_as[i]) {
continue;
}
@@ -1294,7 +1294,7 @@ static void vtd_context_device_invalidate(IntelIOMMUState *s,
vtd_bus = vtd_find_as_from_bus_num(s, bus_n);
if (vtd_bus) {
devfn = VTD_SID_TO_DEVFN(source_id);
- for (devfn_it = 0; devfn_it < X86_IOMMU_PCI_DEVFN_MAX; ++devfn_it) {
+ for (devfn_it = 0; devfn_it < PCI_DEVFN_MAX; ++devfn_it) {
vtd_as = vtd_bus->dev_as[devfn_it];
if (vtd_as && ((devfn_it & mask) == (devfn & mask))) {
trace_vtd_inv_desc_cc_device(bus_n, VTD_PCI_SLOT(devfn_it),
@@ -2699,7 +2699,7 @@ VTDAddressSpace *vtd_find_add_as(IntelIOMMUState *s, PCIBus *bus, int devfn)
*new_key = (uintptr_t)bus;
/* No corresponding free() */
vtd_bus = g_malloc0(sizeof(VTDBus) + sizeof(VTDAddressSpace *) * \
- X86_IOMMU_PCI_DEVFN_MAX);
+ PCI_DEVFN_MAX);
vtd_bus->bus = bus;
g_hash_table_insert(s->vtd_as_by_busptr, new_key, vtd_bus);
}
@@ -2982,7 +2982,7 @@ static AddressSpace *vtd_host_dma_iommu(PCIBus *bus, void *opaque, int devfn)
IntelIOMMUState *s = opaque;
VTDAddressSpace *vtd_as;
- assert(0 <= devfn && devfn < X86_IOMMU_PCI_DEVFN_MAX);
+ assert(0 <= devfn && devfn < PCI_DEVFN_MAX);
vtd_as = vtd_find_add_as(s, bus, devfn);
return &vtd_as->as;
diff --git a/include/hw/i386/x86-iommu.h b/include/hw/i386/x86-iommu.h
index ef89c0c646..7c71fc7470 100644
--- a/include/hw/i386/x86-iommu.h
+++ b/include/hw/i386/x86-iommu.h
@@ -31,7 +31,6 @@
#define X86_IOMMU_GET_CLASS(obj) \
OBJECT_GET_CLASS(X86IOMMUClass, obj, TYPE_X86_IOMMU_DEVICE)
-#define X86_IOMMU_PCI_DEVFN_MAX 256
#define X86_IOMMU_SID_INVALID (0xffff)
typedef struct X86IOMMUState X86IOMMUState;
--
2.14.3
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Qemu-devel] [PATCH 2/2] intel_iommu: fix error param in string
2017-12-08 4:26 [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
2017-12-08 4:26 ` [Qemu-devel] [PATCH 1/2] intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX Peter Xu
@ 2017-12-08 4:26 ` Peter Xu
2017-12-08 6:52 ` Liu, Yi L
2017-12-20 3:16 ` [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
2017-12-20 8:43 ` Jason Wang
3 siblings, 1 reply; 7+ messages in thread
From: Peter Xu @ 2017-12-08 4:26 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S . Tsirkin, peterx, Jason Wang
It should be caching-mode. It may confuse people when it pops up.
Signed-off-by: Peter Xu <peterx@redhat.com>
---
hw/i386/intel_iommu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
index c1fa08d205..fe15d3ba84 100644
--- a/hw/i386/intel_iommu.c
+++ b/hw/i386/intel_iommu.c
@@ -2327,7 +2327,7 @@ static void vtd_iommu_notify_flag_changed(IOMMUMemoryRegion *iommu,
IntelIOMMUNotifierNode *next_node = NULL;
if (!s->caching_mode && new & IOMMU_NOTIFIER_MAP) {
- error_report("We need to set cache_mode=1 for intel-iommu to enable "
+ error_report("We need to set caching-mode=1 for intel-iommu to enable "
"device assignment with IOMMU protection.");
exit(1);
}
--
2.14.3
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH 1/2] intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX
2017-12-08 4:26 ` [Qemu-devel] [PATCH 1/2] intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX Peter Xu
@ 2017-12-08 6:50 ` Liu, Yi L
0 siblings, 0 replies; 7+ messages in thread
From: Liu, Yi L @ 2017-12-08 6:50 UTC (permalink / raw)
To: Peter Xu; +Cc: qemu-devel, Jason Wang, Michael S . Tsirkin
On Fri, Dec 08, 2017 at 12:26:53PM +0800, Peter Xu wrote:
Looks good to me.
Reviewed-by: Liu, Yi L <yi.l.liu@intel.com>
> We have PCI_DEVFN_MAX now.
>
> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
> hw/i386/intel_iommu.c | 10 +++++-----
> include/hw/i386/x86-iommu.h | 1 -
> 2 files changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
> index 3a5bb0bc2e..c1fa08d205 100644
> --- a/hw/i386/intel_iommu.c
> +++ b/hw/i386/intel_iommu.c
> @@ -186,7 +186,7 @@ static void vtd_reset_context_cache(IntelIOMMUState *s)
> g_hash_table_iter_init(&bus_it, s->vtd_as_by_busptr);
>
> while (g_hash_table_iter_next (&bus_it, NULL, (void**)&vtd_bus)) {
> - for (devfn_it = 0; devfn_it < X86_IOMMU_PCI_DEVFN_MAX; ++devfn_it) {
> + for (devfn_it = 0; devfn_it < PCI_DEVFN_MAX; ++devfn_it) {
> vtd_as = vtd_bus->dev_as[devfn_it];
> if (!vtd_as) {
> continue;
> @@ -1002,7 +1002,7 @@ static void vtd_switch_address_space_all(IntelIOMMUState *s)
>
> g_hash_table_iter_init(&iter, s->vtd_as_by_busptr);
> while (g_hash_table_iter_next(&iter, NULL, (void **)&vtd_bus)) {
> - for (i = 0; i < X86_IOMMU_PCI_DEVFN_MAX; i++) {
> + for (i = 0; i < PCI_DEVFN_MAX; i++) {
> if (!vtd_bus->dev_as[i]) {
> continue;
> }
> @@ -1294,7 +1294,7 @@ static void vtd_context_device_invalidate(IntelIOMMUState *s,
> vtd_bus = vtd_find_as_from_bus_num(s, bus_n);
> if (vtd_bus) {
> devfn = VTD_SID_TO_DEVFN(source_id);
> - for (devfn_it = 0; devfn_it < X86_IOMMU_PCI_DEVFN_MAX; ++devfn_it) {
> + for (devfn_it = 0; devfn_it < PCI_DEVFN_MAX; ++devfn_it) {
> vtd_as = vtd_bus->dev_as[devfn_it];
> if (vtd_as && ((devfn_it & mask) == (devfn & mask))) {
> trace_vtd_inv_desc_cc_device(bus_n, VTD_PCI_SLOT(devfn_it),
> @@ -2699,7 +2699,7 @@ VTDAddressSpace *vtd_find_add_as(IntelIOMMUState *s, PCIBus *bus, int devfn)
> *new_key = (uintptr_t)bus;
> /* No corresponding free() */
> vtd_bus = g_malloc0(sizeof(VTDBus) + sizeof(VTDAddressSpace *) * \
> - X86_IOMMU_PCI_DEVFN_MAX);
> + PCI_DEVFN_MAX);
> vtd_bus->bus = bus;
> g_hash_table_insert(s->vtd_as_by_busptr, new_key, vtd_bus);
> }
> @@ -2982,7 +2982,7 @@ static AddressSpace *vtd_host_dma_iommu(PCIBus *bus, void *opaque, int devfn)
> IntelIOMMUState *s = opaque;
> VTDAddressSpace *vtd_as;
>
> - assert(0 <= devfn && devfn < X86_IOMMU_PCI_DEVFN_MAX);
> + assert(0 <= devfn && devfn < PCI_DEVFN_MAX);
>
> vtd_as = vtd_find_add_as(s, bus, devfn);
> return &vtd_as->as;
> diff --git a/include/hw/i386/x86-iommu.h b/include/hw/i386/x86-iommu.h
> index ef89c0c646..7c71fc7470 100644
> --- a/include/hw/i386/x86-iommu.h
> +++ b/include/hw/i386/x86-iommu.h
> @@ -31,7 +31,6 @@
> #define X86_IOMMU_GET_CLASS(obj) \
> OBJECT_GET_CLASS(X86IOMMUClass, obj, TYPE_X86_IOMMU_DEVICE)
>
> -#define X86_IOMMU_PCI_DEVFN_MAX 256
> #define X86_IOMMU_SID_INVALID (0xffff)
>
> typedef struct X86IOMMUState X86IOMMUState;
> --
> 2.14.3
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH 2/2] intel_iommu: fix error param in string
2017-12-08 4:26 ` [Qemu-devel] [PATCH 2/2] intel_iommu: fix error param in string Peter Xu
@ 2017-12-08 6:52 ` Liu, Yi L
0 siblings, 0 replies; 7+ messages in thread
From: Liu, Yi L @ 2017-12-08 6:52 UTC (permalink / raw)
To: Peter Xu; +Cc: qemu-devel, Jason Wang, Michael S . Tsirkin
Looks good.
Reviewed-by: Liu, Yi L <yi.l.liu@intel.com>
On Fri, Dec 08, 2017 at 12:26:54PM +0800, Peter Xu wrote:
> It should be caching-mode. It may confuse people when it pops up.
>
> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
> hw/i386/intel_iommu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
> index c1fa08d205..fe15d3ba84 100644
> --- a/hw/i386/intel_iommu.c
> +++ b/hw/i386/intel_iommu.c
> @@ -2327,7 +2327,7 @@ static void vtd_iommu_notify_flag_changed(IOMMUMemoryRegion *iommu,
> IntelIOMMUNotifierNode *next_node = NULL;
>
> if (!s->caching_mode && new & IOMMU_NOTIFIER_MAP) {
> - error_report("We need to set cache_mode=1 for intel-iommu to enable "
> + error_report("We need to set caching-mode=1 for intel-iommu to enable "
> "device assignment with IOMMU protection.");
> exit(1);
> }
> --
> 2.14.3
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12
2017-12-08 4:26 [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
2017-12-08 4:26 ` [Qemu-devel] [PATCH 1/2] intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX Peter Xu
2017-12-08 4:26 ` [Qemu-devel] [PATCH 2/2] intel_iommu: fix error param in string Peter Xu
@ 2017-12-20 3:16 ` Peter Xu
2017-12-20 8:43 ` Jason Wang
3 siblings, 0 replies; 7+ messages in thread
From: Peter Xu @ 2017-12-20 3:16 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S . Tsirkin, Jason Wang
On Fri, Dec 08, 2017 at 12:26:52PM +0800, Peter Xu wrote:
> Please review. Thanks,
Ping for 2.12.
>
> Peter Xu (2):
> intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX
> intel_iommu: fix error param in string
>
> hw/i386/intel_iommu.c | 12 ++++++------
> include/hw/i386/x86-iommu.h | 1 -
> 2 files changed, 6 insertions(+), 7 deletions(-)
>
> --
> 2.14.3
>
--
Peter Xu
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12
2017-12-08 4:26 [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
` (2 preceding siblings ...)
2017-12-20 3:16 ` [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
@ 2017-12-20 8:43 ` Jason Wang
3 siblings, 0 replies; 7+ messages in thread
From: Jason Wang @ 2017-12-20 8:43 UTC (permalink / raw)
To: Peter Xu, qemu-devel; +Cc: Michael S . Tsirkin
On 2017年12月08日 12:26, Peter Xu wrote:
> Please review. Thanks,
>
> Peter Xu (2):
> intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX
> intel_iommu: fix error param in string
>
> hw/i386/intel_iommu.c | 12 ++++++------
> include/hw/i386/x86-iommu.h | 1 -
> 2 files changed, 6 insertions(+), 7 deletions(-)
>
Reviewed-by: Jason Wang <jasowang@redhat.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-12-20 8:44 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-08 4:26 [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
2017-12-08 4:26 ` [Qemu-devel] [PATCH 1/2] intel_iommu: remove X86_IOMMU_PCI_DEVFN_MAX Peter Xu
2017-12-08 6:50 ` Liu, Yi L
2017-12-08 4:26 ` [Qemu-devel] [PATCH 2/2] intel_iommu: fix error param in string Peter Xu
2017-12-08 6:52 ` Liu, Yi L
2017-12-20 3:16 ` [Qemu-devel] [PATCH 0/2] vtd: tiny fixes for 2.12 Peter Xu
2017-12-20 8:43 ` Jason Wang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.