[PATCH 0/9] remove some of unneeded kallsyms includes

[PATCH 0/9] remove some of unneeded kallsyms includes

[Sergey Senozhatsky]

Hello,

	A small patch set that removes some kallsyms includes
here and there. Mostly those kallsyms includes are leftovers:
printk() gained %pS/%pF modifiers support some time ago, so
print_symbol() and friends became sort of unneeded [along with
print_fn_descriptor_symbol() deprecation], thus some of the
users were converted to pS/pF. This patch set just cleans up
that convertion.

	We still have a number of print_symbol() users [which
must be converted to ps/pf, print_symbol() uses a stack buffer
KSYM_SYMBOL_LEN to do what printk(ps/pf) can do], but this is
out of scope.

	I compile tested the patch set; but, as always and
usual, would be great if 0day build robot double check it.

Sergey Senozhatsky (9):
  sched/autogroup: remove unneeded kallsyms include
  mm: remove unneeded kallsyms include
  power: remove unneeded kallsyms include
  pci: remove unneeded kallsyms include
  pnp: remove unneeded kallsyms include
  mm: remove unneeded kallsyms include
  workqueue: remove unneeded kallsyms include
  hrtimer: remove unneeded kallsyms include
  genirq: remove unneeded kallsyms include

 drivers/base/power/main.c | 1 -
 drivers/pci/quirks.c      | 1 -
 drivers/pnp/quirks.c      | 1 -
 kernel/irq/spurious.c     | 1 -
 kernel/sched/autogroup.c  | 1 -
 kernel/time/hrtimer.c     | 1 -
 kernel/workqueue.c        | 1 -
 mm/memory.c               | 4 ----
 mm/vmalloc.c              | 1 -
 9 files changed, 12 deletions(-)

-- 
2.15.1

[PATCH 0/9] remove some of unneeded kallsyms includes

[Sergey Senozhatsky]

Hello,

	A small patch set that removes some kallsyms includes
here and there. Mostly those kallsyms includes are leftovers:
printk() gained %pS/%pF modifiers support some time ago, so
print_symbol() and friends became sort of unneeded [along with
print_fn_descriptor_symbol() deprecation], thus some of the
users were converted to pS/pF. This patch set just cleans up
that convertion.

	We still have a number of print_symbol() users [which
must be converted to ps/pf, print_symbol() uses a stack buffer
KSYM_SYMBOL_LEN to do what printk(ps/pf) can do], but this is
out of scope.

	I compile tested the patch set; but, as always and
usual, would be great if 0day build robot double check it.

Sergey Senozhatsky (9):
  sched/autogroup: remove unneeded kallsyms include
  mm: remove unneeded kallsyms include
  power: remove unneeded kallsyms include
  pci: remove unneeded kallsyms include
  pnp: remove unneeded kallsyms include
  mm: remove unneeded kallsyms include
  workqueue: remove unneeded kallsyms include
  hrtimer: remove unneeded kallsyms include
  genirq: remove unneeded kallsyms include

 drivers/base/power/main.c | 1 -
 drivers/pci/quirks.c      | 1 -
 drivers/pnp/quirks.c      | 1 -
 kernel/irq/spurious.c     | 1 -
 kernel/sched/autogroup.c  | 1 -
 kernel/time/hrtimer.c     | 1 -
 kernel/workqueue.c        | 1 -
 mm/memory.c               | 4 ----
 mm/vmalloc.c              | 1 -
 9 files changed, 12 deletions(-)

-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Sergey Senozhatsky]

Autogroup does not seem to use any of kallsyms functions/defines.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
---
 kernel/sched/autogroup.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
index a43df5193538..0786227a3f48 100644
--- a/kernel/sched/autogroup.c
+++ b/kernel/sched/autogroup.c
@@ -3,7 +3,6 @@
 
 #include <linux/proc_fs.h>
 #include <linux/seq_file.h>
-#include <linux/kallsyms.h>
 #include <linux/utsname.h>
 #include <linux/security.h>
 #include <linux/export.h>
-- 
2.15.1

[PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Sergey Senozhatsky]

Autogroup does not seem to use any of kallsyms functions/defines.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
---
 kernel/sched/autogroup.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
index a43df5193538..0786227a3f48 100644
--- a/kernel/sched/autogroup.c
+++ b/kernel/sched/autogroup.c
@@ -3,7 +3,6 @@
 
 #include <linux/proc_fs.h>
 #include <linux/seq_file.h>
-#include <linux/kallsyms.h>
 #include <linux/utsname.h>
 #include <linux/security.h>
 #include <linux/export.h>
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 2/9] mm: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_symbol() to %pSR a while
ago (071361d3473ebb814 "mm: Convert print_symbol to %pSR").
kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Michal Hocko <mhocko@suse.com>
---
 mm/memory.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/mm/memory.c b/mm/memory.c
index 24e9e1d5488f..551b323c9ab6 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -59,7 +59,6 @@
 #include <linux/writeback.h>
 #include <linux/memcontrol.h>
 #include <linux/mmu_notifier.h>
-#include <linux/kallsyms.h>
 #include <linux/swapops.h>
 #include <linux/elf.h>
 #include <linux/gfp.h>
@@ -760,9 +759,6 @@ static void print_bad_pte(struct vm_area_struct *vma, unsigned long addr,
 		dump_page(page, "bad pte");
 	pr_alert("addr:%p vm_flags:%08lx anon_vma:%p mapping:%p index:%lx\n",
 		 (void *)addr, vma->vm_flags, vma->anon_vma, mapping, index);
-	/*
-	 * Choose text because data symbols depend on CONFIG_KALLSYMS_ALL=y
-	 */
 	pr_alert("file:%pD fault:%pf mmap:%pf readpage:%pf\n",
 		 vma->vm_file,
 		 vma->vm_ops ? vma->vm_ops->fault : NULL,
-- 
2.15.1

[PATCH 2/9] mm: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_symbol() to %pSR a while
ago (071361d3473ebb814 "mm: Convert print_symbol to %pSR").
kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Michal Hocko <mhocko@suse.com>
---
 mm/memory.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/mm/memory.c b/mm/memory.c
index 24e9e1d5488f..551b323c9ab6 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -59,7 +59,6 @@
 #include <linux/writeback.h>
 #include <linux/memcontrol.h>
 #include <linux/mmu_notifier.h>
-#include <linux/kallsyms.h>
 #include <linux/swapops.h>
 #include <linux/elf.h>
 #include <linux/gfp.h>
@@ -760,9 +759,6 @@ static void print_bad_pte(struct vm_area_struct *vma, unsigned long addr,
 		dump_page(page, "bad pte");
 	pr_alert("addr:%p vm_flags:%08lx anon_vma:%p mapping:%p index:%lx\n",
 		 (void *)addr, vma->vm_flags, vma->anon_vma, mapping, index);
-	/*
-	 * Choose text because data symbols depend on CONFIG_KALLSYMS_ALL=y
-	 */
 	pr_alert("file:%pD fault:%pf mmap:%pf readpage:%pf\n",
 		 vma->vm_file,
 		 vma->vm_ops ? vma->vm_ops->fault : NULL,
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 3/9] power: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_fn_descriptor_symbol()
to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
vsprintf symbolic function pointer format"). kallsyms does
not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Rafael Wysocki <rjw@rjwysocki.net>
Cc: Len Brown <len.brown@intel.com>
---
 drivers/base/power/main.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
index 5bc2cf1f812c..e2539d8423f7 100644
--- a/drivers/base/power/main.c
+++ b/drivers/base/power/main.c
@@ -18,7 +18,6 @@
  */
 
 #include <linux/device.h>
-#include <linux/kallsyms.h>
 #include <linux/export.h>
 #include <linux/mutex.h>
 #include <linux/pm.h>
-- 
2.15.1

[PATCH 3/9] power: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_fn_descriptor_symbol()
to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
vsprintf symbolic function pointer format"). kallsyms does
not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Rafael Wysocki <rjw@rjwysocki.net>
Cc: Len Brown <len.brown@intel.com>
---
 drivers/base/power/main.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
index 5bc2cf1f812c..e2539d8423f7 100644
--- a/drivers/base/power/main.c
+++ b/drivers/base/power/main.c
@@ -18,7 +18,6 @@
  */
 
 #include <linux/device.h>
-#include <linux/kallsyms.h>
 #include <linux/export.h>
 #include <linux/mutex.h>
 #include <linux/pm.h>
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 4/9] pci: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_fn_descriptor_symbol()
to %pF some time ago (c9bbb4abb658da "PCI: use %pF instead
of print_fn_descriptor_symbol() in quirks.c"). kallsyms does
not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
---
 drivers/pci/quirks.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index 10684b17d0bd..fd49b976973f 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -19,7 +19,6 @@
 #include <linux/init.h>
 #include <linux/delay.h>
 #include <linux/acpi.h>
-#include <linux/kallsyms.h>
 #include <linux/dmi.h>
 #include <linux/pci-aspm.h>
 #include <linux/ioport.h>
-- 
2.15.1

[PATCH 4/9] pci: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_fn_descriptor_symbol()
to %pF some time ago (c9bbb4abb658da "PCI: use %pF instead
of print_fn_descriptor_symbol() in quirks.c"). kallsyms does
not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
---
 drivers/pci/quirks.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index 10684b17d0bd..fd49b976973f 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -19,7 +19,6 @@
 #include <linux/init.h>
 #include <linux/delay.h>
 #include <linux/acpi.h>
-#include <linux/kallsyms.h>
 #include <linux/dmi.h>
 #include <linux/pci-aspm.h>
 #include <linux/ioport.h>
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 5/9] pnp: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_fn_descriptor_symbol()
to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
don't use deprecated print_fn_descriptor_symbol()). kallsyms
does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
---
 drivers/pnp/quirks.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
index f054cdddfef8..803666ae3635 100644
--- a/drivers/pnp/quirks.c
+++ b/drivers/pnp/quirks.c
@@ -21,7 +21,6 @@
 #include <linux/slab.h>
 #include <linux/pnp.h>
 #include <linux/io.h>
-#include <linux/kallsyms.h>
 #include "base.h"
 
 static void quirk_awe32_add_ports(struct pnp_dev *dev,
-- 
2.15.1

[PATCH 5/9] pnp: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_fn_descriptor_symbol()
to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
don't use deprecated print_fn_descriptor_symbol()). kallsyms
does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
---
 drivers/pnp/quirks.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
index f054cdddfef8..803666ae3635 100644
--- a/drivers/pnp/quirks.c
+++ b/drivers/pnp/quirks.c
@@ -21,7 +21,6 @@
 #include <linux/slab.h>
 #include <linux/pnp.h>
 #include <linux/io.h>
-#include <linux/kallsyms.h>
 #include "base.h"
 
 static void quirk_awe32_add_ports(struct pnp_dev *dev,
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 6/9] mm: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from sprint_symbol() to %pS some time
ago (62c70bce8ac23651 "mm: convert sprintf_symbol to %pS").
kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Michal Hocko <mhocko@suse.com>
Cc: Vlastimil Babka <vbabka@suse.cz>
---
 mm/vmalloc.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index 673942094328..fdda6534e9a0 100644
--- a/mm/vmalloc.c
+++ b/mm/vmalloc.c
@@ -19,7 +19,6 @@
 #include <linux/proc_fs.h>
 #include <linux/seq_file.h>
 #include <linux/debugobjects.h>
-#include <linux/kallsyms.h>
 #include <linux/list.h>
 #include <linux/notifier.h>
 #include <linux/rbtree.h>
-- 
2.15.1

[PATCH 6/9] mm: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from sprint_symbol() to %pS some time
ago (62c70bce8ac23651 "mm: convert sprintf_symbol to %pS").
kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Michal Hocko <mhocko@suse.com>
Cc: Vlastimil Babka <vbabka@suse.cz>
---
 mm/vmalloc.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index 673942094328..fdda6534e9a0 100644
--- a/mm/vmalloc.c
+++ b/mm/vmalloc.c
@@ -19,7 +19,6 @@
 #include <linux/proc_fs.h>
 #include <linux/seq_file.h>
 #include <linux/debugobjects.h>
-#include <linux/kallsyms.h>
 #include <linux/list.h>
 #include <linux/notifier.h>
 #include <linux/rbtree.h>
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 7/9] workqueue: remove unneeded kallsyms include

[Sergey Senozhatsky]

The filw was converted from print_symbol() to %pf some time
ago (044c782ce3a901fb "workqueue: fix checkpatch issues").
kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Tejun Heo <tj@kernel.org>
Cc: Lai Jiangshan <jiangshanlai@gmail.com>
---
 kernel/workqueue.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index f4ff9fa5586a..acf485eada1b 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
@@ -38,7 +38,6 @@
 #include <linux/hardirq.h>
 #include <linux/mempolicy.h>
 #include <linux/freezer.h>
-#include <linux/kallsyms.h>
 #include <linux/debug_locks.h>
 #include <linux/lockdep.h>
 #include <linux/idr.h>
-- 
2.15.1

[PATCH 7/9] workqueue: remove unneeded kallsyms include

[Sergey Senozhatsky]

The filw was converted from print_symbol() to %pf some time
ago (044c782ce3a901fb "workqueue: fix checkpatch issues").
kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Tejun Heo <tj@kernel.org>
Cc: Lai Jiangshan <jiangshanlai@gmail.com>
---
 kernel/workqueue.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index f4ff9fa5586a..acf485eada1b 100644
--- a/kernel/workqueue.c
+++ b/kernel/workqueue.c
@@ -38,7 +38,6 @@
 #include <linux/hardirq.h>
 #include <linux/mempolicy.h>
 #include <linux/freezer.h>
-#include <linux/kallsyms.h>
 #include <linux/debug_locks.h>
 #include <linux/lockdep.h>
 #include <linux/idr.h>
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 8/9] hrtimer: remove unneeded kallsyms include

[Sergey Senozhatsky]

hrtimer does not seem to use any of kallsyms functions/defines.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
---
 kernel/time/hrtimer.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c
index d32520840fde..6d8183b38e35 100644
--- a/kernel/time/hrtimer.c
+++ b/kernel/time/hrtimer.c
@@ -37,7 +37,6 @@
 #include <linux/hrtimer.h>
 #include <linux/notifier.h>
 #include <linux/syscalls.h>
-#include <linux/kallsyms.h>
 #include <linux/interrupt.h>
 #include <linux/tick.h>
 #include <linux/seq_file.h>
-- 
2.15.1

[PATCH 8/9] hrtimer: remove unneeded kallsyms include

[Sergey Senozhatsky]

hrtimer does not seem to use any of kallsyms functions/defines.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
---
 kernel/time/hrtimer.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c
index d32520840fde..6d8183b38e35 100644
--- a/kernel/time/hrtimer.c
+++ b/kernel/time/hrtimer.c
@@ -37,7 +37,6 @@
 #include <linux/hrtimer.h>
 #include <linux/notifier.h>
 #include <linux/syscalls.h>
-#include <linux/kallsyms.h>
 #include <linux/interrupt.h>
 #include <linux/tick.h>
 #include <linux/seq_file.h>
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 9/9] genirq: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_symbol() to %pf some time ago
(ef26f20cd117eb3c18 "genirq: Print threaded handler in spurious
debug output"). kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
---
 kernel/irq/spurious.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/irq/spurious.c b/kernel/irq/spurious.c
index ef2a47e0eab6..6cdecc6f4c53 100644
--- a/kernel/irq/spurious.c
+++ b/kernel/irq/spurious.c
@@ -10,7 +10,6 @@
 #include <linux/jiffies.h>
 #include <linux/irq.h>
 #include <linux/module.h>
-#include <linux/kallsyms.h>
 #include <linux/interrupt.h>
 #include <linux/moduleparam.h>
 #include <linux/timer.h>
-- 
2.15.1

[PATCH 9/9] genirq: remove unneeded kallsyms include

[Sergey Senozhatsky]

The file was converted from print_symbol() to %pf some time ago
(ef26f20cd117eb3c18 "genirq: Print threaded handler in spurious
debug output"). kallsyms does not seem to be needed anymore.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
---
 kernel/irq/spurious.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/kernel/irq/spurious.c b/kernel/irq/spurious.c
index ef2a47e0eab6..6cdecc6f4c53 100644
--- a/kernel/irq/spurious.c
+++ b/kernel/irq/spurious.c
@@ -10,7 +10,6 @@
 #include <linux/jiffies.h>
 #include <linux/irq.h>
 #include <linux/module.h>
-#include <linux/kallsyms.h>
 #include <linux/interrupt.h>
 #include <linux/moduleparam.h>
 #include <linux/timer.h>
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH] sched/autogroup: move sched.h include

[Sergey Senozhatsky]

Move local "sched.h" include to the bottom. sched.h defines
several macros that are getting redefined in ARCH-specific
code, for instance, finish_arch_post_lock_switch() and
prepare_arch_switch(), so we need ARCH-specific definitions
to come in first.

Suggested-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
---
 kernel/sched/autogroup.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
index 0786227a3f48..bb4b9fe026a1 100644
--- a/kernel/sched/autogroup.c
+++ b/kernel/sched/autogroup.c
@@ -1,12 +1,12 @@
 // SPDX-License-Identifier: GPL-2.0
-#include "sched.h"
-
 #include <linux/proc_fs.h>
 #include <linux/seq_file.h>
 #include <linux/utsname.h>
 #include <linux/security.h>
 #include <linux/export.h>
 
+#include "sched.h"
+
 unsigned int __read_mostly sysctl_sched_autogroup_enabled = 1;
 static struct autogroup autogroup_default;
 static atomic_t autogroup_seq_nr;
-- 
2.15.1

[PATCH] sched/autogroup: move sched.h include

[Sergey Senozhatsky]

Move local "sched.h" include to the bottom. sched.h defines
several macros that are getting redefined in ARCH-specific
code, for instance, finish_arch_post_lock_switch() and
prepare_arch_switch(), so we need ARCH-specific definitions
to come in first.

Suggested-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
---
 kernel/sched/autogroup.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
index 0786227a3f48..bb4b9fe026a1 100644
--- a/kernel/sched/autogroup.c
+++ b/kernel/sched/autogroup.c
@@ -1,12 +1,12 @@
 // SPDX-License-Identifier: GPL-2.0
-#include "sched.h"
-
 #include <linux/proc_fs.h>
 #include <linux/seq_file.h>
 #include <linux/utsname.h>
 #include <linux/security.h>
 #include <linux/export.h>
 
+#include "sched.h"
+
 unsigned int __read_mostly sysctl_sched_autogroup_enabled = 1;
 static struct autogroup autogroup_default;
 static atomic_t autogroup_seq_nr;
-- 
2.15.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] sched/autogroup: move sched.h include

[Petr Mladek]

On Fri 2017-12-08 17:24:22, Sergey Senozhatsky wrote:
> Move local "sched.h" include to the bottom. sched.h defines
> several macros that are getting redefined in ARCH-specific
> code, for instance, finish_arch_post_lock_switch() and
> prepare_arch_switch(), so we need ARCH-specific definitions
> to come in first.

This patch is needed to fix compilation error [1] caused by a patchset
that deprecates %pf/%pF printk modifiers[2].

IMHO, we should make sure that this fix goes into Linus' tree
before the printk-related patchset. What is the best practice,
please?

I see two reasonable possibilities. Either sched people could
push this for-4.15-rcX. Or I could put it into printk.git for-4.16
in the right order.

What do you think?

Referece:
[0] http://lkml.kernel.org/r/201712080259.tvO64XfA%fengguang.wu@intel.com
[1] https://git.kernel.org/pub/scm/linux/kernel/git/pmladek/printk.git/commit/?h=for-next&id=98fff2c57b7e88d643cb42ffd910fe9905b33176

Best Regards,
Petr

Re: [PATCH] sched/autogroup: move sched.h include

[Petr Mladek]

On Fri 2017-12-08 17:24:22, Sergey Senozhatsky wrote:
> Move local "sched.h" include to the bottom. sched.h defines
> several macros that are getting redefined in ARCH-specific
> code, for instance, finish_arch_post_lock_switch() and
> prepare_arch_switch(), so we need ARCH-specific definitions
> to come in first.

This patch is needed to fix compilation error [1] caused by a patchset
that deprecates %pf/%pF printk modifiers[2].

IMHO, we should make sure that this fix goes into Linus' tree
before the printk-related patchset. What is the best practice,
please?

I see two reasonable possibilities. Either sched people could
push this for-4.15-rcX. Or I could put it into printk.git for-4.16
in the right order.

What do you think?

Referece:
[0] http://lkml.kernel.org/r/201712080259.tvO64XfA%fengguang.wu@intel.com
[1] https://git.kernel.org/pub/scm/linux/kernel/git/pmladek/printk.git/commit/?h=for-next&id=98fff2c57b7e88d643cb42ffd910fe9905b33176

Best Regards,
Petr

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] sched/autogroup: move sched.h include

[Sergey Senozhatsky]

Hi,

On (12/08/17 09:57), Petr Mladek wrote:
> On Fri 2017-12-08 17:24:22, Sergey Senozhatsky wrote:
> > Move local "sched.h" include to the bottom. sched.h defines
> > several macros that are getting redefined in ARCH-specific
> > code, for instance, finish_arch_post_lock_switch() and
> > prepare_arch_switch(), so we need ARCH-specific definitions
> > to come in first.
> 
> This patch is needed to fix compilation error [1] caused by a patchset
> that deprecates %pf/%pF printk modifiers[2].
> 
> IMHO, we should make sure that this fix goes into Linus' tree
> before the printk-related patchset. What is the best practice,
> please?

as long as sched pull request goes before printk pull request we
are fine. but I see your point.

> I see two reasonable possibilities. Either sched people could
> push this for-4.15-rcX. Or I could put it into printk.git for-4.16
> in the right order.

agreed.

> What do you think?

either way is fine with me. we can have it in print.git (no objections
from my side) or in sched tree and just make sure that sched pull request
has "bigger priority", or it can go to Linus's tree as a potential fix
(I'd prefer the last option, I think).

	-ss

Re: [PATCH] sched/autogroup: move sched.h include

[Sergey Senozhatsky]

Hi,

On (12/08/17 09:57), Petr Mladek wrote:
> On Fri 2017-12-08 17:24:22, Sergey Senozhatsky wrote:
> > Move local "sched.h" include to the bottom. sched.h defines
> > several macros that are getting redefined in ARCH-specific
> > code, for instance, finish_arch_post_lock_switch() and
> > prepare_arch_switch(), so we need ARCH-specific definitions
> > to come in first.
> 
> This patch is needed to fix compilation error [1] caused by a patchset
> that deprecates %pf/%pF printk modifiers[2].
> 
> IMHO, we should make sure that this fix goes into Linus' tree
> before the printk-related patchset. What is the best practice,
> please?

as long as sched pull request goes before printk pull request we
are fine. but I see your point.

> I see two reasonable possibilities. Either sched people could
> push this for-4.15-rcX. Or I could put it into printk.git for-4.16
> in the right order.

agreed.

> What do you think?

either way is fine with me. we can have it in print.git (no objections
from my side) or in sched tree and just make sure that sched pull request
has "bigger priority", or it can go to Linus's tree as a potential fix
(I'd prefer the last option, I think).

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Peter Zijlstra]

On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote:
> Autogroup does not seem to use any of kallsyms functions/defines.
> 
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Peter Zijlstra <peterz@infradead.org>

Feel free to take this through whatever tree you need this in.

Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

> ---
>  kernel/sched/autogroup.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
> index a43df5193538..0786227a3f48 100644
> --- a/kernel/sched/autogroup.c
> +++ b/kernel/sched/autogroup.c
> @@ -3,7 +3,6 @@
>  
>  #include <linux/proc_fs.h>
>  #include <linux/seq_file.h>
> -#include <linux/kallsyms.h>
>  #include <linux/utsname.h>
>  #include <linux/security.h>
>  #include <linux/export.h>
> -- 
> 2.15.1
> 

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Peter Zijlstra]

On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote:
> Autogroup does not seem to use any of kallsyms functions/defines.
> 
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Peter Zijlstra <peterz@infradead.org>

Feel free to take this through whatever tree you need this in.

Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

> ---
>  kernel/sched/autogroup.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
> index a43df5193538..0786227a3f48 100644
> --- a/kernel/sched/autogroup.c
> +++ b/kernel/sched/autogroup.c
> @@ -3,7 +3,6 @@
>  
>  #include <linux/proc_fs.h>
>  #include <linux/seq_file.h>
> -#include <linux/kallsyms.h>
>  #include <linux/utsname.h>
>  #include <linux/security.h>
>  #include <linux/export.h>
> -- 
> 2.15.1
> 

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] sched/autogroup: move sched.h include

[Peter Zijlstra]

On Fri, Dec 08, 2017 at 05:24:22PM +0900, Sergey Senozhatsky wrote:
> Move local "sched.h" include to the bottom. sched.h defines
> several macros that are getting redefined in ARCH-specific
> code, for instance, finish_arch_post_lock_switch() and
> prepare_arch_switch(), so we need ARCH-specific definitions
> to come in first.
> 
> Suggested-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>

Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

> ---
>  kernel/sched/autogroup.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
> index 0786227a3f48..bb4b9fe026a1 100644
> --- a/kernel/sched/autogroup.c
> +++ b/kernel/sched/autogroup.c
> @@ -1,12 +1,12 @@
>  // SPDX-License-Identifier: GPL-2.0
> -#include "sched.h"
> -
>  #include <linux/proc_fs.h>
>  #include <linux/seq_file.h>
>  #include <linux/utsname.h>
>  #include <linux/security.h>
>  #include <linux/export.h>
>  
> +#include "sched.h"
> +
>  unsigned int __read_mostly sysctl_sched_autogroup_enabled = 1;
>  static struct autogroup autogroup_default;
>  static atomic_t autogroup_seq_nr;
> -- 
> 2.15.1
> 

Re: [PATCH] sched/autogroup: move sched.h include

[Peter Zijlstra]

On Fri, Dec 08, 2017 at 05:24:22PM +0900, Sergey Senozhatsky wrote:
> Move local "sched.h" include to the bottom. sched.h defines
> several macros that are getting redefined in ARCH-specific
> code, for instance, finish_arch_post_lock_switch() and
> prepare_arch_switch(), so we need ARCH-specific definitions
> to come in first.
> 
> Suggested-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>

Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

> ---
>  kernel/sched/autogroup.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
> index 0786227a3f48..bb4b9fe026a1 100644
> --- a/kernel/sched/autogroup.c
> +++ b/kernel/sched/autogroup.c
> @@ -1,12 +1,12 @@
>  // SPDX-License-Identifier: GPL-2.0
> -#include "sched.h"
> -
>  #include <linux/proc_fs.h>
>  #include <linux/seq_file.h>
>  #include <linux/utsname.h>
>  #include <linux/security.h>
>  #include <linux/export.h>
>  
> +#include "sched.h"
> +
>  unsigned int __read_mostly sysctl_sched_autogroup_enabled = 1;
>  static struct autogroup autogroup_default;
>  static atomic_t autogroup_seq_nr;
> -- 
> 2.15.1
> 

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 11:56), Peter Zijlstra wrote:
> On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote:
> > Autogroup does not seem to use any of kallsyms functions/defines.
> > 
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Peter Zijlstra <peterz@infradead.org>
> 
> Feel free to take this through whatever tree you need this in.
> 
> Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

Thanks!


Petr, let's pick up sched/autogroup patches then. Thank you.

	-ss

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 11:56), Peter Zijlstra wrote:
> On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote:
> > Autogroup does not seem to use any of kallsyms functions/defines.
> > 
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Peter Zijlstra <peterz@infradead.org>
> 
> Feel free to take this through whatever tree you need this in.
> 
> Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

Thanks!


Petr, let's pick up sched/autogroup patches then. Thank you.

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Petr Mladek]

On Fri 2017-12-08 20:20:48, Sergey Senozhatsky wrote:
> On (12/08/17 11:56), Peter Zijlstra wrote:
> > On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote:
> > > Autogroup does not seem to use any of kallsyms functions/defines.
> > > 
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Cc: Peter Zijlstra <peterz@infradead.org>
> > 
> > Feel free to take this through whatever tree you need this in.
> > 
> > Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

Thanks!

> Petr, let's pick up sched/autogroup patches then. Thank you.

Yup, I have pushed both patches into printk.git for-4.16:

https://git.kernel.org/pub/scm/linux/kernel/git/pmladek/printk.git/commit/?h=for-4.16&id=79ee842891595293be37c5aed0e75b4630166c5a
https://git.kernel.org/pub/scm/linux/kernel/git/pmladek/printk.git/commit/?h=for-4.16&id=25493e5fba2f7b8cdade29d0fc8945114ee7732b

Best Regards,
Petr

Re: [PATCH 1/9] sched/autogroup: remove unneeded kallsyms include

[Petr Mladek]

On Fri 2017-12-08 20:20:48, Sergey Senozhatsky wrote:
> On (12/08/17 11:56), Peter Zijlstra wrote:
> > On Fri, Dec 08, 2017 at 11:56:08AM +0900, Sergey Senozhatsky wrote:
> > > Autogroup does not seem to use any of kallsyms functions/defines.
> > > 
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Cc: Peter Zijlstra <peterz@infradead.org>
> > 
> > Feel free to take this through whatever tree you need this in.
> > 
> > Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>

Thanks!

> Petr, let's pick up sched/autogroup patches then. Thank you.

Yup, I have pushed both patches into printk.git for-4.16:

https://git.kernel.org/pub/scm/linux/kernel/git/pmladek/printk.git/commit/?h=for-4.16&id=79ee842891595293be37c5aed0e75b4630166c5a
https://git.kernel.org/pub/scm/linux/kernel/git/pmladek/printk.git/commit/?h=for-4.16&id=25493e5fba2f7b8cdade29d0fc8945114ee7732b

Best Regards,
Petr

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
<sergey.senozhatsky.work@gmail.com> wrote:
> The file was converted from print_fn_descriptor_symbol()
> to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
> vsprintf symbolic function pointer format"). kallsyms does
> not seem to be needed anymore.
>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Rafael Wysocki <rjw@rjwysocki.net>
> Cc: Len Brown <len.brown@intel.com>
> ---
>  drivers/base/power/main.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> index 5bc2cf1f812c..e2539d8423f7 100644
> --- a/drivers/base/power/main.c
> +++ b/drivers/base/power/main.c
> @@ -18,7 +18,6 @@
>   */
>
>  #include <linux/device.h>
> -#include <linux/kallsyms.h>
>  #include <linux/export.h>
>  #include <linux/mutex.h>
>  #include <linux/pm.h>
> --

Do you want me to apply this or do you want to route it differently?

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
<sergey.senozhatsky.work@gmail.com> wrote:
> The file was converted from print_fn_descriptor_symbol()
> to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
> vsprintf symbolic function pointer format"). kallsyms does
> not seem to be needed anymore.
>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Rafael Wysocki <rjw@rjwysocki.net>
> Cc: Len Brown <len.brown@intel.com>
> ---
>  drivers/base/power/main.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> index 5bc2cf1f812c..e2539d8423f7 100644
> --- a/drivers/base/power/main.c
> +++ b/drivers/base/power/main.c
> @@ -18,7 +18,6 @@
>   */
>
>  #include <linux/device.h>
> -#include <linux/kallsyms.h>
>  #include <linux/export.h>
>  #include <linux/mutex.h>
>  #include <linux/pm.h>
> --

Do you want me to apply this or do you want to route it differently?

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 5/9] pnp: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
<sergey.senozhatsky.work@gmail.com> wrote:
> The file was converted from print_fn_descriptor_symbol()
> to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
> don't use deprecated print_fn_descriptor_symbol()). kallsyms
> does not seem to be needed anymore.
>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> ---
>  drivers/pnp/quirks.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
> index f054cdddfef8..803666ae3635 100644
> --- a/drivers/pnp/quirks.c
> +++ b/drivers/pnp/quirks.c
> @@ -21,7 +21,6 @@
>  #include <linux/slab.h>
>  #include <linux/pnp.h>
>  #include <linux/io.h>
> -#include <linux/kallsyms.h>
>  #include "base.h"
>
>  static void quirk_awe32_add_ports(struct pnp_dev *dev,
> --

Do you want me to apply this or do you want to route it differently?

Re: [PATCH 5/9] pnp: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
<sergey.senozhatsky.work@gmail.com> wrote:
> The file was converted from print_fn_descriptor_symbol()
> to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
> don't use deprecated print_fn_descriptor_symbol()). kallsyms
> does not seem to be needed anymore.
>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> ---
>  drivers/pnp/quirks.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
> index f054cdddfef8..803666ae3635 100644
> --- a/drivers/pnp/quirks.c
> +++ b/drivers/pnp/quirks.c
> @@ -21,7 +21,6 @@
>  #include <linux/slab.h>
>  #include <linux/pnp.h>
>  #include <linux/io.h>
> -#include <linux/kallsyms.h>
>  #include "base.h"
>
>  static void quirk_awe32_add_ports(struct pnp_dev *dev,
> --

Do you want me to apply this or do you want to route it differently?

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 15:01), Rafael J. Wysocki wrote:
> On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> <sergey.senozhatsky.work@gmail.com> wrote:
> > The file was converted from print_fn_descriptor_symbol()
> > to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
> > vsprintf symbolic function pointer format"). kallsyms does
> > not seem to be needed anymore.
> >
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Rafael Wysocki <rjw@rjwysocki.net>
> > Cc: Len Brown <len.brown@intel.com>
> > ---
> >  drivers/base/power/main.c | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> > index 5bc2cf1f812c..e2539d8423f7 100644
> > --- a/drivers/base/power/main.c
> > +++ b/drivers/base/power/main.c
> > @@ -18,7 +18,6 @@
> >   */
> >
> >  #include <linux/device.h>
> > -#include <linux/kallsyms.h>
> >  #include <linux/export.h>
> >  #include <linux/mutex.h>
> >  #include <linux/pm.h>
> > --
> 
> Do you want me to apply this or do you want to route it differently?

Hello Rafael,

don't mind if you'll pick it up.

	-ss

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 15:01), Rafael J. Wysocki wrote:
> On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> <sergey.senozhatsky.work@gmail.com> wrote:
> > The file was converted from print_fn_descriptor_symbol()
> > to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
> > vsprintf symbolic function pointer format"). kallsyms does
> > not seem to be needed anymore.
> >
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Rafael Wysocki <rjw@rjwysocki.net>
> > Cc: Len Brown <len.brown@intel.com>
> > ---
> >  drivers/base/power/main.c | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> > index 5bc2cf1f812c..e2539d8423f7 100644
> > --- a/drivers/base/power/main.c
> > +++ b/drivers/base/power/main.c
> > @@ -18,7 +18,6 @@
> >   */
> >
> >  #include <linux/device.h>
> > -#include <linux/kallsyms.h>
> >  #include <linux/export.h>
> >  #include <linux/mutex.h>
> >  #include <linux/pm.h>
> > --
> 
> Do you want me to apply this or do you want to route it differently?

Hello Rafael,

don't mind if you'll pick it up.

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 5/9] pnp: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 15:02), Rafael J. Wysocki wrote:
> On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> <sergey.senozhatsky.work@gmail.com> wrote:
> > The file was converted from print_fn_descriptor_symbol()
> > to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
> > don't use deprecated print_fn_descriptor_symbol()). kallsyms
> > does not seem to be needed anymore.
> >
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Bjorn Helgaas <bhelgaas@google.com>
> > ---
> >  drivers/pnp/quirks.c | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
> > index f054cdddfef8..803666ae3635 100644
> > --- a/drivers/pnp/quirks.c
> > +++ b/drivers/pnp/quirks.c
> > @@ -21,7 +21,6 @@
> >  #include <linux/slab.h>
> >  #include <linux/pnp.h>
> >  #include <linux/io.h>
> > -#include <linux/kallsyms.h>
> >  #include "base.h"
> >
> >  static void quirk_awe32_add_ports(struct pnp_dev *dev,
> > --
> 
> Do you want me to apply this or do you want to route it differently?

please pick it up.

	-ss

Re: [PATCH 5/9] pnp: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 15:02), Rafael J. Wysocki wrote:
> On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> <sergey.senozhatsky.work@gmail.com> wrote:
> > The file was converted from print_fn_descriptor_symbol()
> > to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
> > don't use deprecated print_fn_descriptor_symbol()). kallsyms
> > does not seem to be needed anymore.
> >
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Bjorn Helgaas <bhelgaas@google.com>
> > ---
> >  drivers/pnp/quirks.c | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
> > index f054cdddfef8..803666ae3635 100644
> > --- a/drivers/pnp/quirks.c
> > +++ b/drivers/pnp/quirks.c
> > @@ -21,7 +21,6 @@
> >  #include <linux/slab.h>
> >  #include <linux/pnp.h>
> >  #include <linux/io.h>
> > -#include <linux/kallsyms.h>
> >  #include "base.h"
> >
> >  static void quirk_awe32_add_ports(struct pnp_dev *dev,
> > --
> 
> Do you want me to apply this or do you want to route it differently?

please pick it up.

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 4/9] pci: remove unneeded kallsyms include

[Bjorn Helgaas]

On Fri, Dec 08, 2017 at 11:56:11AM +0900, Sergey Senozhatsky wrote:
> The file was converted from print_fn_descriptor_symbol()
> to %pF some time ago (c9bbb4abb658da "PCI: use %pF instead
> of print_fn_descriptor_symbol() in quirks.c"). kallsyms does
> not seem to be needed anymore.
> 
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>

Applied to pci/misc for v4.16, thanks for cleaning this up!

I *assume* there's no ordering dependency like the one you mentioned
for sched/printk.  If there is, let me know, and I'll move this to
for-linus to get it in v4.15.

> ---
>  drivers/pci/quirks.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index 10684b17d0bd..fd49b976973f 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -19,7 +19,6 @@
>  #include <linux/init.h>
>  #include <linux/delay.h>
>  #include <linux/acpi.h>
> -#include <linux/kallsyms.h>
>  #include <linux/dmi.h>
>  #include <linux/pci-aspm.h>
>  #include <linux/ioport.h>
> -- 
> 2.15.1
> 

Re: [PATCH 4/9] pci: remove unneeded kallsyms include

[Bjorn Helgaas]

On Fri, Dec 08, 2017 at 11:56:11AM +0900, Sergey Senozhatsky wrote:
> The file was converted from print_fn_descriptor_symbol()
> to %pF some time ago (c9bbb4abb658da "PCI: use %pF instead
> of print_fn_descriptor_symbol() in quirks.c"). kallsyms does
> not seem to be needed anymore.
> 
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>

Applied to pci/misc for v4.16, thanks for cleaning this up!

I *assume* there's no ordering dependency like the one you mentioned
for sched/printk.  If there is, let me know, and I'll move this to
for-linus to get it in v4.15.

> ---
>  drivers/pci/quirks.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index 10684b17d0bd..fd49b976973f 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -19,7 +19,6 @@
>  #include <linux/init.h>
>  #include <linux/delay.h>
>  #include <linux/acpi.h>
> -#include <linux/kallsyms.h>
>  #include <linux/dmi.h>
>  #include <linux/pci-aspm.h>
>  #include <linux/ioport.h>
> -- 
> 2.15.1
> 

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 0/9] remove some of unneeded kallsyms includes

[Andrew Morton]

On Fri,  8 Dec 2017 11:56:07 +0900 Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> wrote:

> 	A small patch set that removes some kallsyms includes
> here and there. Mostly those kallsyms includes are leftovers:
> printk() gained %pS/%pF modifiers support some time ago, so
> print_symbol() and friends became sort of unneeded [along with
> print_fn_descriptor_symbol() deprecation], thus some of the
> users were converted to pS/pF. This patch set just cleans up
> that convertion.
> 
> 	We still have a number of print_symbol() users [which
> must be converted to ps/pf, print_symbol() uses a stack buffer
> KSYM_SYMBOL_LEN to do what printk(ps/pf) can do], but this is
> out of scope.
> 
> 	I compile tested the patch set; but, as always and
> usual, would be great if 0day build robot double check it.

I grabbed everything and shall drop any patches which later turn up in
the various subsystem trees.

Re: [PATCH 0/9] remove some of unneeded kallsyms includes

[Andrew Morton]

On Fri,  8 Dec 2017 11:56:07 +0900 Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> wrote:

> 	A small patch set that removes some kallsyms includes
> here and there. Mostly those kallsyms includes are leftovers:
> printk() gained %pS/%pF modifiers support some time ago, so
> print_symbol() and friends became sort of unneeded [along with
> print_fn_descriptor_symbol() deprecation], thus some of the
> users were converted to pS/pF. This patch set just cleans up
> that convertion.
> 
> 	We still have a number of print_symbol() users [which
> must be converted to ps/pf, print_symbol() uses a stack buffer
> KSYM_SYMBOL_LEN to do what printk(ps/pf) can do], but this is
> out of scope.
> 
> 	I compile tested the patch set; but, as always and
> usual, would be great if 0day build robot double check it.

I grabbed everything and shall drop any patches which later turn up in
the various subsystem trees.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 4/9] pci: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 11:53), Bjorn Helgaas wrote:
> On Fri, Dec 08, 2017 at 11:56:11AM +0900, Sergey Senozhatsky wrote:
> > The file was converted from print_fn_descriptor_symbol()
> > to %pF some time ago (c9bbb4abb658da "PCI: use %pF instead
> > of print_fn_descriptor_symbol() in quirks.c"). kallsyms does
> > not seem to be needed anymore.
> > 
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Bjorn Helgaas <bhelgaas@google.com>
> 
> Applied to pci/misc for v4.16, thanks for cleaning this up!

thanks!

> I *assume* there's no ordering dependency like the one you mentioned
> for sched/printk.

no dependency. you are right.

	-ss

Re: [PATCH 4/9] pci: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/08/17 11:53), Bjorn Helgaas wrote:
> On Fri, Dec 08, 2017 at 11:56:11AM +0900, Sergey Senozhatsky wrote:
> > The file was converted from print_fn_descriptor_symbol()
> > to %pF some time ago (c9bbb4abb658da "PCI: use %pF instead
> > of print_fn_descriptor_symbol() in quirks.c"). kallsyms does
> > not seem to be needed anymore.
> > 
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Bjorn Helgaas <bhelgaas@google.com>
> 
> Applied to pci/misc for v4.16, thanks for cleaning this up!

thanks!

> I *assume* there's no ordering dependency like the one you mentioned
> for sched/printk.

no dependency. you are right.

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 0/9] remove some of unneeded kallsyms includes

[Sergey Senozhatsky]

On (12/08/17 14:24), Andrew Morton wrote:
> On Fri,  8 Dec 2017 11:56:07 +0900 Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> wrote:
> 
> > 	A small patch set that removes some kallsyms includes
> > here and there. Mostly those kallsyms includes are leftovers:
> > printk() gained %pS/%pF modifiers support some time ago, so
> > print_symbol() and friends became sort of unneeded [along with
> > print_fn_descriptor_symbol() deprecation], thus some of the
> > users were converted to pS/pF. This patch set just cleans up
> > that convertion.
> > 
> > 	We still have a number of print_symbol() users [which
> > must be converted to ps/pf, print_symbol() uses a stack buffer
> > KSYM_SYMBOL_LEN to do what printk(ps/pf) can do], but this is
> > out of scope.
> > 
> > 	I compile tested the patch set; but, as always and
> > usual, would be great if 0day build robot double check it.
> 
> I grabbed everything and shall drop any patches which later turn up in
> the various subsystem trees.

thank you!

	-ss

Re: [PATCH 0/9] remove some of unneeded kallsyms includes

[Sergey Senozhatsky]

On (12/08/17 14:24), Andrew Morton wrote:
> On Fri,  8 Dec 2017 11:56:07 +0900 Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> wrote:
> 
> > 	A small patch set that removes some kallsyms includes
> > here and there. Mostly those kallsyms includes are leftovers:
> > printk() gained %pS/%pF modifiers support some time ago, so
> > print_symbol() and friends became sort of unneeded [along with
> > print_fn_descriptor_symbol() deprecation], thus some of the
> > users were converted to pS/pF. This patch set just cleans up
> > that convertion.
> > 
> > 	We still have a number of print_symbol() users [which
> > must be converted to ps/pf, print_symbol() uses a stack buffer
> > KSYM_SYMBOL_LEN to do what printk(ps/pf) can do], but this is
> > out of scope.
> > 
> > 	I compile tested the patch set; but, as always and
> > usual, would be great if 0day build robot double check it.
> 
> I grabbed everything and shall drop any patches which later turn up in
> the various subsystem trees.

thank you!

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 7/9] workqueue: remove unneeded kallsyms include

[Tejun Heo]

On Fri, Dec 08, 2017 at 11:56:14AM +0900, Sergey Senozhatsky wrote:
> The filw was converted from print_symbol() to %pf some time
> ago (044c782ce3a901fb "workqueue: fix checkpatch issues").
> kallsyms does not seem to be needed anymore.
> 
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Tejun Heo <tj@kernel.org>
> Cc: Lai Jiangshan <jiangshanlai@gmail.com>

Applied to wq/for-4.15-fixes.

Thanks.

-- 
tejun

Re: [PATCH 7/9] workqueue: remove unneeded kallsyms include

[Tejun Heo]

On Fri, Dec 08, 2017 at 11:56:14AM +0900, Sergey Senozhatsky wrote:
> The filw was converted from print_symbol() to %pf some time
> ago (044c782ce3a901fb "workqueue: fix checkpatch issues").
> kallsyms does not seem to be needed anymore.
> 
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> Cc: Tejun Heo <tj@kernel.org>
> Cc: Lai Jiangshan <jiangshanlai@gmail.com>

Applied to wq/for-4.15-fixes.

Thanks.

-- 
tejun

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 7/9] workqueue: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/11/17 07:16), Tejun Heo wrote:
> On Fri, Dec 08, 2017 at 11:56:14AM +0900, Sergey Senozhatsky wrote:
> > The filw was converted from print_symbol() to %pf some time
> > ago (044c782ce3a901fb "workqueue: fix checkpatch issues").
> > kallsyms does not seem to be needed anymore.
> > 
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Tejun Heo <tj@kernel.org>
> > Cc: Lai Jiangshan <jiangshanlai@gmail.com>
> 
> Applied to wq/for-4.15-fixes.

thanks.

	-ss

Re: [PATCH 7/9] workqueue: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/11/17 07:16), Tejun Heo wrote:
> On Fri, Dec 08, 2017 at 11:56:14AM +0900, Sergey Senozhatsky wrote:
> > The filw was converted from print_symbol() to %pf some time
> > ago (044c782ce3a901fb "workqueue: fix checkpatch issues").
> > kallsyms does not seem to be needed anymore.
> > 
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > Cc: Tejun Heo <tj@kernel.org>
> > Cc: Lai Jiangshan <jiangshanlai@gmail.com>
> 
> Applied to wq/for-4.15-fixes.

thanks.

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Friday, December 8, 2017 3:48:44 PM CET Sergey Senozhatsky wrote:
> On (12/08/17 15:01), Rafael J. Wysocki wrote:
> > On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> > <sergey.senozhatsky.work@gmail.com> wrote:
> > > The file was converted from print_fn_descriptor_symbol()
> > > to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
> > > vsprintf symbolic function pointer format"). kallsyms does
> > > not seem to be needed anymore.
> > >
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Cc: Rafael Wysocki <rjw@rjwysocki.net>
> > > Cc: Len Brown <len.brown@intel.com>
> > > ---
> > >  drivers/base/power/main.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> > > index 5bc2cf1f812c..e2539d8423f7 100644
> > > --- a/drivers/base/power/main.c
> > > +++ b/drivers/base/power/main.c
> > > @@ -18,7 +18,6 @@
> > >   */
> > >
> > >  #include <linux/device.h>
> > > -#include <linux/kallsyms.h>
> > >  #include <linux/export.h>
> > >  #include <linux/mutex.h>
> > >  #include <linux/pm.h>
> > > --
> > 
> > Do you want me to apply this or do you want to route it differently?
> 
> Hello Rafael,
> 
> don't mind if you'll pick it up.

OK, applied now.

Thanks!

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Friday, December 8, 2017 3:48:44 PM CET Sergey Senozhatsky wrote:
> On (12/08/17 15:01), Rafael J. Wysocki wrote:
> > On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> > <sergey.senozhatsky.work@gmail.com> wrote:
> > > The file was converted from print_fn_descriptor_symbol()
> > > to %pF some time ago (c80cfb0406c01bb "vsprintf: use new
> > > vsprintf symbolic function pointer format"). kallsyms does
> > > not seem to be needed anymore.
> > >
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Cc: Rafael Wysocki <rjw@rjwysocki.net>
> > > Cc: Len Brown <len.brown@intel.com>
> > > ---
> > >  drivers/base/power/main.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> > > index 5bc2cf1f812c..e2539d8423f7 100644
> > > --- a/drivers/base/power/main.c
> > > +++ b/drivers/base/power/main.c
> > > @@ -18,7 +18,6 @@
> > >   */
> > >
> > >  #include <linux/device.h>
> > > -#include <linux/kallsyms.h>
> > >  #include <linux/export.h>
> > >  #include <linux/mutex.h>
> > >  #include <linux/pm.h>
> > > --
> > 
> > Do you want me to apply this or do you want to route it differently?
> 
> Hello Rafael,
> 
> don't mind if you'll pick it up.

OK, applied now.

Thanks!

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 5/9] pnp: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Friday, December 8, 2017 3:49:09 PM CET Sergey Senozhatsky wrote:
> On (12/08/17 15:02), Rafael J. Wysocki wrote:
> > On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> > <sergey.senozhatsky.work@gmail.com> wrote:
> > > The file was converted from print_fn_descriptor_symbol()
> > > to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
> > > don't use deprecated print_fn_descriptor_symbol()). kallsyms
> > > does not seem to be needed anymore.
> > >
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Cc: Bjorn Helgaas <bhelgaas@google.com>
> > > ---
> > >  drivers/pnp/quirks.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
> > > index f054cdddfef8..803666ae3635 100644
> > > --- a/drivers/pnp/quirks.c
> > > +++ b/drivers/pnp/quirks.c
> > > @@ -21,7 +21,6 @@
> > >  #include <linux/slab.h>
> > >  #include <linux/pnp.h>
> > >  #include <linux/io.h>
> > > -#include <linux/kallsyms.h>
> > >  #include "base.h"
> > >
> > >  static void quirk_awe32_add_ports(struct pnp_dev *dev,
> > > --
> > 
> > Do you want me to apply this or do you want to route it differently?
> 
> please pick it up.

Applied, thanks!

Re: [PATCH 5/9] pnp: remove unneeded kallsyms include

[Rafael J. Wysocki]

On Friday, December 8, 2017 3:49:09 PM CET Sergey Senozhatsky wrote:
> On (12/08/17 15:02), Rafael J. Wysocki wrote:
> > On Fri, Dec 8, 2017 at 3:56 AM, Sergey Senozhatsky
> > <sergey.senozhatsky.work@gmail.com> wrote:
> > > The file was converted from print_fn_descriptor_symbol()
> > > to %pF some time ago (2e532d68a2b3e2aa {pci,pnp} quirks.c:
> > > don't use deprecated print_fn_descriptor_symbol()). kallsyms
> > > does not seem to be needed anymore.
> > >
> > > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
> > > Cc: Bjorn Helgaas <bhelgaas@google.com>
> > > ---
> > >  drivers/pnp/quirks.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c
> > > index f054cdddfef8..803666ae3635 100644
> > > --- a/drivers/pnp/quirks.c
> > > +++ b/drivers/pnp/quirks.c
> > > @@ -21,7 +21,6 @@
> > >  #include <linux/slab.h>
> > >  #include <linux/pnp.h>
> > >  #include <linux/io.h>
> > > -#include <linux/kallsyms.h>
> > >  #include "base.h"
> > >
> > >  static void quirk_awe32_add_ports(struct pnp_dev *dev,
> > > --
> > 
> > Do you want me to apply this or do you want to route it differently?
> 
> please pick it up.

Applied, thanks!

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/17/17 19:04), Rafael J. Wysocki wrote:
[..]
> > > Do you want me to apply this or do you want to route it differently?
> > 
> > Hello Rafael,
> > 
> > don't mind if you'll pick it up.
> 
> OK, applied now.
> 
> Thanks!

thanks!

	-ss

Re: [PATCH 3/9] power: remove unneeded kallsyms include

[Sergey Senozhatsky]

On (12/17/17 19:04), Rafael J. Wysocki wrote:
[..]
> > > Do you want me to apply this or do you want to route it differently?
> > 
> > Hello Rafael,
> > 
> > don't mind if you'll pick it up.
> 
> OK, applied now.
> 
> Thanks!

thanks!

	-ss

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>