[PATCH 0/2] iommu-common build warning fixes.

All of lore.kernel.org
 help / color / mirror / Atom feed

* [PATCH 0/2] iommu-common build warning fixes.
@ 2015-04-19 17:13 ` Sowmini Varadhan
  0 siblings, 0 replies; 12+ messages in thread
From: Sowmini Varadhan @ 2015-04-19 17:13 UTC (permalink / raw)
  To: sparclinux; +Cc: davem, linux, sowmini.varadhan, linux-kernel

This patchset has fixes for 2 architectures
- Part 1: x86_64 warnings generated by sparse 
- Part 2: allmodconfig build error for powerpc, identified by Guenter Roeck.

Sowmini Varadhan (2):
  fix x86_64 sparse warnings
  rename iommu_pool_hash to iommu_hash_common

 lib/iommu-common.c |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)


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

* [PATCH 0/2] iommu-common build warning fixes.
@ 2015-04-19 17:13 ` Sowmini Varadhan
  0 siblings, 0 replies; 12+ messages in thread
From: Sowmini Varadhan @ 2015-04-19 17:13 UTC (permalink / raw)
  To: sparclinux; +Cc: davem, linux, sowmini.varadhan, linux-kernel

This patchset has fixes for 2 architectures
- Part 1: x86_64 warnings generated by sparse 
- Part 2: allmodconfig build error for powerpc, identified by Guenter Roeck.

Sowmini Varadhan (2):
  fix x86_64 sparse warnings
  rename iommu_pool_hash to iommu_hash_common

 lib/iommu-common.c |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)


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

* [PATCH 1/2] iommu-common: fix x86_64 compiler warnings
  2015-04-19 17:13 ` Sowmini Varadhan
@ 2015-04-19 17:13   ` Sowmini Varadhan
  -1 siblings, 0 replies; 12+ messages in thread
From: Sowmini Varadhan @ 2015-04-19 17:13 UTC (permalink / raw)
  To: sparclinux; +Cc: davem, linux, sowmini.varadhan, linux-kernel

Declare iommu_large_alloc as static. Remove extern definition  for
iommu_tbl_pool_init().

Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
---
 lib/iommu-common.c |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/lib/iommu-common.c b/lib/iommu-common.c
index a1a517c..a9a53f5 100644
--- a/lib/iommu-common.c
+++ b/lib/iommu-common.c
@@ -15,7 +15,7 @@
 #define	DMA_ERROR_CODE (~(dma_addr_t)0x0)
 #endif
 
-unsigned long iommu_large_alloc = 15;
+static unsigned long iommu_large_alloc = 15;
 
 static	DEFINE_PER_CPU(unsigned int, iommu_pool_hash);
 
@@ -53,12 +53,12 @@ static void setup_iommu_pool_hash(void)
  * the top 1/4 of the table will be set aside for pool allocations
  * of more than iommu_large_alloc pages.
  */
-extern void iommu_tbl_pool_init(struct iommu_map_table *iommu,
-				unsigned long num_entries,
-				u32 table_shift,
-				void (*lazy_flush)(struct iommu_map_table *),
-				bool large_pool, u32 npools,
-				bool skip_span_boundary_check)
+void iommu_tbl_pool_init(struct iommu_map_table *iommu,
+			 unsigned long num_entries,
+			 u32 table_shift,
+			 void (*lazy_flush)(struct iommu_map_table *),
+			 bool large_pool, u32 npools,
+			 bool skip_span_boundary_check)
 {
 	unsigned int start, i;
 	struct iommu_pool *p = &(iommu->large_pool);
-- 
1.7.1


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

* [PATCH 1/2] iommu-common: fix x86_64 compiler warnings
@ 2015-04-19 17:13   ` Sowmini Varadhan
  0 siblings, 0 replies; 12+ messages in thread
From: Sowmini Varadhan @ 2015-04-19 17:13 UTC (permalink / raw)
  To: sparclinux; +Cc: davem, linux, sowmini.varadhan, linux-kernel

Declare iommu_large_alloc as static. Remove extern definition  for
iommu_tbl_pool_init().

Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
---
 lib/iommu-common.c |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/lib/iommu-common.c b/lib/iommu-common.c
index a1a517c..a9a53f5 100644
--- a/lib/iommu-common.c
+++ b/lib/iommu-common.c
@@ -15,7 +15,7 @@
 #define	DMA_ERROR_CODE (~(dma_addr_t)0x0)
 #endif
 
-unsigned long iommu_large_alloc = 15;
+static unsigned long iommu_large_alloc = 15;
 
 static	DEFINE_PER_CPU(unsigned int, iommu_pool_hash);
 
@@ -53,12 +53,12 @@ static void setup_iommu_pool_hash(void)
  * the top 1/4 of the table will be set aside for pool allocations
  * of more than iommu_large_alloc pages.
  */
-extern void iommu_tbl_pool_init(struct iommu_map_table *iommu,
-				unsigned long num_entries,
-				u32 table_shift,
-				void (*lazy_flush)(struct iommu_map_table *),
-				bool large_pool, u32 npools,
-				bool skip_span_boundary_check)
+void iommu_tbl_pool_init(struct iommu_map_table *iommu,
+			 unsigned long num_entries,
+			 u32 table_shift,
+			 void (*lazy_flush)(struct iommu_map_table *),
+			 bool large_pool, u32 npools,
+			 bool skip_span_boundary_check)
 {
 	unsigned int start, i;
 	struct iommu_pool *p = &(iommu->large_pool);
-- 
1.7.1


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

* Re: [PATCH 1/2] iommu-common: fix x86_64 compiler warnings
  2015-04-19 17:13   ` Sowmini Varadhan
@ 2015-04-20  0:48     ` Guenter Roeck
  -1 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2015-04-20  0:48 UTC (permalink / raw)
  To: Sowmini Varadhan, sparclinux; +Cc: davem, linux-kernel

On 04/19/2015 10:13 AM, Sowmini Varadhan wrote:
> Declare iommu_large_alloc as static. Remove extern definition  for
> iommu_tbl_pool_init().
>
> Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>

Tested-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>


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

* Re: [PATCH 1/2] iommu-common: fix x86_64 compiler warnings
@ 2015-04-20  0:48     ` Guenter Roeck
  0 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2015-04-20  0:48 UTC (permalink / raw)
  To: Sowmini Varadhan, sparclinux; +Cc: davem, linux-kernel

On 04/19/2015 10:13 AM, Sowmini Varadhan wrote:
> Declare iommu_large_alloc as static. Remove extern definition  for
> iommu_tbl_pool_init().
>
> Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>

Tested-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>


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

* [PATCH 2/2] iommu-common: rename iommu_pool_hash to iommu_hash_common
  2015-04-19 17:13 ` Sowmini Varadhan
@ 2015-04-19 17:13   ` Sowmini Varadhan
  -1 siblings, 0 replies; 12+ messages in thread
From: Sowmini Varadhan @ 2015-04-19 17:13 UTC (permalink / raw)
  To: sparclinux; +Cc: davem, linux, sowmini.varadhan, linux-kernel

When CONFIG_DEBUG_FORCE_WEAK_PER_CPU is set, the DEFINE_PER_CPU_SECTION
macro will define an extern __pcpu_unique_##name variable that could
conflict with the same definition in powerpc at this time. Avoid that
conflict by renaming iommu_pool_hash in iommu-common.c

Thanks to Guenter Roeck for catching this, and helping to test the fix.

Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
---
 lib/iommu-common.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/iommu-common.c b/lib/iommu-common.c
index a9a53f5..df30632 100644
--- a/lib/iommu-common.c
+++ b/lib/iommu-common.c
@@ -17,7 +17,7 @@
 
 static unsigned long iommu_large_alloc = 15;
 
-static	DEFINE_PER_CPU(unsigned int, iommu_pool_hash);
+static	DEFINE_PER_CPU(unsigned int, iommu_hash_common);
 
 static inline bool need_flush(struct iommu_map_table *iommu)
 {
@@ -44,7 +44,7 @@ static void setup_iommu_pool_hash(void)
 		return;
 	do_once = true;
 	for_each_possible_cpu(i)
-		per_cpu(iommu_pool_hash, i) = hash_32(i, IOMMU_POOL_HASHBITS);
+		per_cpu(iommu_hash_common, i) = hash_32(i, IOMMU_POOL_HASHBITS);
 }
 
 /*
@@ -106,7 +106,7 @@ unsigned long iommu_tbl_range_alloc(struct device *dev,
 				unsigned long mask,
 				unsigned int align_order)
 {
-	unsigned int pool_hash = __this_cpu_read(iommu_pool_hash);
+	unsigned int pool_hash = __this_cpu_read(iommu_hash_common);
 	unsigned long n, end, start, limit, boundary_size;
 	struct iommu_pool *pool;
 	int pass = 0;
-- 
1.7.1


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

* [PATCH 2/2] iommu-common: rename iommu_pool_hash to iommu_hash_common
@ 2015-04-19 17:13   ` Sowmini Varadhan
  0 siblings, 0 replies; 12+ messages in thread
From: Sowmini Varadhan @ 2015-04-19 17:13 UTC (permalink / raw)
  To: sparclinux; +Cc: davem, linux, sowmini.varadhan, linux-kernel

When CONFIG_DEBUG_FORCE_WEAK_PER_CPU is set, the DEFINE_PER_CPU_SECTION
macro will define an extern __pcpu_unique_##name variable that could
conflict with the same definition in powerpc at this time. Avoid that
conflict by renaming iommu_pool_hash in iommu-common.c

Thanks to Guenter Roeck for catching this, and helping to test the fix.

Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
---
 lib/iommu-common.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/iommu-common.c b/lib/iommu-common.c
index a9a53f5..df30632 100644
--- a/lib/iommu-common.c
+++ b/lib/iommu-common.c
@@ -17,7 +17,7 @@
 
 static unsigned long iommu_large_alloc = 15;
 
-static	DEFINE_PER_CPU(unsigned int, iommu_pool_hash);
+static	DEFINE_PER_CPU(unsigned int, iommu_hash_common);
 
 static inline bool need_flush(struct iommu_map_table *iommu)
 {
@@ -44,7 +44,7 @@ static void setup_iommu_pool_hash(void)
 		return;
 	do_once = true;
 	for_each_possible_cpu(i)
-		per_cpu(iommu_pool_hash, i) = hash_32(i, IOMMU_POOL_HASHBITS);
+		per_cpu(iommu_hash_common, i) = hash_32(i, IOMMU_POOL_HASHBITS);
 }
 
 /*
@@ -106,7 +106,7 @@ unsigned long iommu_tbl_range_alloc(struct device *dev,
 				unsigned long mask,
 				unsigned int align_order)
 {
-	unsigned int pool_hash = __this_cpu_read(iommu_pool_hash);
+	unsigned int pool_hash = __this_cpu_read(iommu_hash_common);
 	unsigned long n, end, start, limit, boundary_size;
 	struct iommu_pool *pool;
 	int pass = 0;
-- 
1.7.1


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

* Re: [PATCH 2/2] iommu-common: rename iommu_pool_hash to iommu_hash_common
  2015-04-19 17:13   ` Sowmini Varadhan
@ 2015-04-20  0:50     ` Guenter Roeck
  -1 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2015-04-20  0:50 UTC (permalink / raw)
  To: Sowmini Varadhan, sparclinux; +Cc: davem, linux-kernel

On 04/19/2015 10:13 AM, Sowmini Varadhan wrote:
> When CONFIG_DEBUG_FORCE_WEAK_PER_CPU is set, the DEFINE_PER_CPU_SECTION
> macro will define an extern __pcpu_unique_##name variable that could
> conflict with the same definition in powerpc at this time. Avoid that
> conflict by renaming iommu_pool_hash in iommu-common.c
>
> Thanks to Guenter Roeck for catching this, and helping to test the fix.
>
> Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>

Tested-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>


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

* Re: [PATCH 2/2] iommu-common: rename iommu_pool_hash to iommu_hash_common
@ 2015-04-20  0:50     ` Guenter Roeck
  0 siblings, 0 replies; 12+ messages in thread
From: Guenter Roeck @ 2015-04-20  0:50 UTC (permalink / raw)
  To: Sowmini Varadhan, sparclinux; +Cc: davem, linux-kernel

On 04/19/2015 10:13 AM, Sowmini Varadhan wrote:
> When CONFIG_DEBUG_FORCE_WEAK_PER_CPU is set, the DEFINE_PER_CPU_SECTION
> macro will define an extern __pcpu_unique_##name variable that could
> conflict with the same definition in powerpc at this time. Avoid that
> conflict by renaming iommu_pool_hash in iommu-common.c
>
> Thanks to Guenter Roeck for catching this, and helping to test the fix.
>
> Signed-off-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>

Tested-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>


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

* Re: [PATCH 0/2] iommu-common build warning fixes.
  2015-04-19 17:13 ` Sowmini Varadhan
@ 2015-04-20 18:10   ` David Miller
  -1 siblings, 0 replies; 12+ messages in thread
From: David Miller @ 2015-04-20 18:10 UTC (permalink / raw)
  To: sowmini.varadhan; +Cc: sparclinux, linux, linux-kernel

From: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Date: Sun, 19 Apr 2015 13:13:29 -0400

> This patchset has fixes for 2 architectures
> - Part 1: x86_64 warnings generated by sparse 
> - Part 2: allmodconfig build error for powerpc, identified by Guenter Roeck.

Series applied, thank you.

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

* Re: [PATCH 0/2] iommu-common build warning fixes.
@ 2015-04-20 18:10   ` David Miller
  0 siblings, 0 replies; 12+ messages in thread
From: David Miller @ 2015-04-20 18:10 UTC (permalink / raw)
  To: sowmini.varadhan; +Cc: sparclinux, linux, linux-kernel

From: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Date: Sun, 19 Apr 2015 13:13:29 -0400

> This patchset has fixes for 2 architectures
> - Part 1: x86_64 warnings generated by sparse 
> - Part 2: allmodconfig build error for powerpc, identified by Guenter Roeck.

Series applied, thank you.

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

end of thread, other threads:[~2015-04-20 18:10 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-19 17:13 [PATCH 0/2] iommu-common build warning fixes Sowmini Varadhan
2015-04-19 17:13 ` Sowmini Varadhan
2015-04-19 17:13 ` [PATCH 1/2] iommu-common: fix x86_64 compiler warnings Sowmini Varadhan
2015-04-19 17:13   ` Sowmini Varadhan
2015-04-20  0:48   ` Guenter Roeck
2015-04-20  0:48     ` Guenter Roeck
2015-04-19 17:13 ` [PATCH 2/2] iommu-common: rename iommu_pool_hash to iommu_hash_common Sowmini Varadhan
2015-04-19 17:13   ` Sowmini Varadhan
2015-04-20  0:50   ` Guenter Roeck
2015-04-20  0:50     ` Guenter Roeck
2015-04-20 18:10 ` [PATCH 0/2] iommu-common build warning fixes David Miller
2015-04-20 18:10   ` David Miller

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.