[PATCH RT 0/2] Warning fixes for 3.2-rt (and other branches)

[PATCH RT 0/2] Warning fixes for 3.2-rt (and other branches)

[Ben Hutchings]

[-- Attachment #1: Type: text/plain, Size: 333 bytes --]

These patches fix longstanding warnings seen when building 3.2-rt.

Ben.

Ben Hutchings (2):
  genirq: Fix pointer cast warning for randomness on 32-bit
    architectures
  work-simple: Add missing #include <linux/export.h>

 kernel/irq/manage.c  | 3 ++-
 kernel/work-simple.c | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 811 bytes --]

[PATCH RT 1/2] genirq: Fix pointer cast warning for randomness on 32-bit architectures

[Ben Hutchings]

[-- Attachment #1: Type: text/plain, Size: 803 bytes --]

gcc warns about casting directly from a 32-bit pointer to u64.  In the
current rt patch series we cast to unsigned long to avoid the warning,
but this fix hasn't been backported to all versions.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Cc: <stable-rt@vger.kernel.org> # 3.2
---
 kernel/irq/manage.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index c4677795d7c8..b1683ba80c88 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -897,7 +897,8 @@ static int irq_thread(void *data)
 #ifdef CONFIG_PREEMPT_RT_FULL
 			migrate_disable();
 			add_interrupt_randomness(action->irq, 0,
-						 desc->random_ip ^ (u64) action);
+						 desc->random_ip ^
+						 (unsigned long) action);
 			migrate_enable();
 #endif
 		}


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 811 bytes --]

[PATCH RT 2/2] work-simple: Add missing #include <linux/export.h>

[Ben Hutchings]

[-- Attachment #1: Type: text/plain, Size: 670 bytes --]

work-simple.c uses EXPORT_SYMBOL_GPL(), defined in <linux/export.h>.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Cc: stable-rt@vger.kernel.org # 3.2
---
I'm not sure there's any need for these exports so the correct fix
might be to remove them.

Ben.

 kernel/work-simple.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/kernel/work-simple.c b/kernel/work-simple.c
index c996f755dba6..e57a0522573f 100644
--- a/kernel/work-simple.c
+++ b/kernel/work-simple.c
@@ -10,6 +10,7 @@
 #include <linux/kthread.h>
 #include <linux/slab.h>
 #include <linux/spinlock.h>
+#include <linux/export.h>
 
 #define SWORK_EVENT_PENDING     (1 << 0)
 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 811 bytes --]