* [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
* [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 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
* 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.