* [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf
@ 2012-02-28 18:49 Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 1/9] sparc: Use vsprintf extention %pf with builtin_return_address Joe Perches
` (8 more replies)
0 siblings, 9 replies; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: linux-parisc, linux-scsi, linux-usb
Cc: linux-alpha, linux-kernel, linux-arm-kernel, microblaze-uclinux,
linuxppc-dev, sparclinux, devel
Emit the actual function name when possible
Joe Perches (9):
sparc: Use vsprintf extention %pf with builtin_return_address
alpha: Use vsprintf extention %pf with builtin_return_address
arm: Use vsprintf extention %pf with builtin_return_address
microblaze: Use vsprintf extention %pf with builtin_return_address
powerpc: Use vsprintf extention %pf with builtin_return_address
parisc: Use vsprintf extention %pf with builtin_return_address
scsi: Use vsprintf extention %pf with builtin_return_address
staging: ramster: Use vsprintf extention %pf with builtin_return_address
gadget: Use vsprintf extention %pf with builtin_return_address
arch/alpha/kernel/pci_iommu.c | 20 ++++++++++----------
arch/arm/nwfpe/fpmodule.c | 2 +-
arch/microblaze/mm/pgtable.c | 2 +-
arch/powerpc/mm/pgtable_32.c | 2 +-
arch/sparc/kernel/ds.c | 2 +-
arch/sparc/mm/srmmu.c | 2 +-
drivers/parisc/superio.c | 2 +-
drivers/scsi/esp_scsi.c | 2 +-
drivers/staging/ramster/cluster/heartbeat.c | 4 ++--
drivers/usb/gadget/u_serial.c | 2 +-
10 files changed, 20 insertions(+), 20 deletions(-)
--
1.7.8.111.gad25c.dirty
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH trivial next 1/9] sparc: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 21:08 ` David Miller
2012-02-28 18:49 ` [PATCH trivial next 2/9] alpha: " Joe Perches
` (7 subsequent siblings)
8 siblings, 1 reply; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: Jiri Kosina; +Cc: David S. Miller, sparclinux, linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
arch/sparc/kernel/ds.c | 2 +-
arch/sparc/mm/srmmu.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/sparc/kernel/ds.c b/arch/sparc/kernel/ds.c
index 381edcd..e65b9fc 100644
--- a/arch/sparc/kernel/ds.c
+++ b/arch/sparc/kernel/ds.c
@@ -868,7 +868,7 @@ void ldom_power_off(void)
static void ds_conn_reset(struct ds_info *dp)
{
- printk(KERN_ERR "ds-%llu: ds_conn_reset() from %p\n",
+ printk(KERN_ERR "ds-%llu: ds_conn_reset() from %pf\n",
dp->id, __builtin_return_address(0));
}
diff --git a/arch/sparc/mm/srmmu.c b/arch/sparc/mm/srmmu.c
index cbef74e..f37fbb2 100644
--- a/arch/sparc/mm/srmmu.c
+++ b/arch/sparc/mm/srmmu.c
@@ -705,7 +705,7 @@ static void swift_update_mmu_cache(struct vm_area_struct * vma, unsigned long ad
val = srmmu_hwprobe(address);
if (val != 0 && pte_val(*ptep) != val) {
printk("swift_update_mmu_cache: "
- "addr %lx put %08x probed %08x from %p\n",
+ "addr %lx put %08x probed %08x from %pf\n",
address, pte_val(*ptep), val,
__builtin_return_address(0));
srmmu_flush_whole_tlb();
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 2/9] alpha: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 1/9] sparc: Use vsprintf extention %pf with builtin_return_address Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 3/9] arm: " Joe Perches
` (6 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: Jiri Kosina
Cc: Richard Henderson, Ivan Kokshaysky, Matt Turner, linux-alpha,
linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
arch/alpha/kernel/pci_iommu.c | 20 ++++++++++----------
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/arch/alpha/kernel/pci_iommu.c b/arch/alpha/kernel/pci_iommu.c
index cd63479..3f844d2 100644
--- a/arch/alpha/kernel/pci_iommu.c
+++ b/arch/alpha/kernel/pci_iommu.c
@@ -236,7 +236,7 @@ static int pci_dac_dma_supported(struct pci_dev *dev, u64 mask)
ok = 0;
/* If both conditions above are met, we are fine. */
- DBGA("pci_dac_dma_supported %s from %p\n",
+ DBGA("pci_dac_dma_supported %s from %pf\n",
ok ? "yes" : "no", __builtin_return_address(0));
return ok;
@@ -268,7 +268,7 @@ pci_map_single_1(struct pci_dev *pdev, void *cpu_addr, size_t size,
&& paddr + size <= __direct_map_size) {
ret = paddr + __direct_map_base;
- DBGA2("pci_map_single: [%p,%zx] -> direct %llx from %p\n",
+ DBGA2("pci_map_single: [%p,%zx] -> direct %llx from %pf\n",
cpu_addr, size, ret, __builtin_return_address(0));
return ret;
@@ -279,7 +279,7 @@ pci_map_single_1(struct pci_dev *pdev, void *cpu_addr, size_t size,
if (dac_allowed) {
ret = paddr + alpha_mv.pci_dac_offset;
- DBGA2("pci_map_single: [%p,%zx] -> DAC %llx from %p\n",
+ DBGA2("pci_map_single: [%p,%zx] -> DAC %llx from %pf\n",
cpu_addr, size, ret, __builtin_return_address(0));
return ret;
@@ -316,7 +316,7 @@ pci_map_single_1(struct pci_dev *pdev, void *cpu_addr, size_t size,
ret = arena->dma_base + dma_ofs * PAGE_SIZE;
ret += (unsigned long)cpu_addr & ~PAGE_MASK;
- DBGA2("pci_map_single: [%p,%zx] np %ld -> sg %llx from %p\n",
+ DBGA2("pci_map_single: [%p,%zx] np %ld -> sg %llx from %pf\n",
cpu_addr, size, npages, ret, __builtin_return_address(0));
return ret;
@@ -385,14 +385,14 @@ static void alpha_pci_unmap_page(struct device *dev, dma_addr_t dma_addr,
&& dma_addr < __direct_map_base + __direct_map_size) {
/* Nothing to do. */
- DBGA2("pci_unmap_single: direct [%llx,%zx] from %p\n",
+ DBGA2("pci_unmap_single: direct [%llx,%zx] from %pf\n",
dma_addr, size, __builtin_return_address(0));
return;
}
if (dma_addr > 0xffffffff) {
- DBGA2("pci64_unmap_single: DAC [%llx,%zx] from %p\n",
+ DBGA2("pci64_unmap_single: DAC [%llx,%zx] from %pf\n",
dma_addr, size, __builtin_return_address(0));
return;
}
@@ -424,7 +424,7 @@ static void alpha_pci_unmap_page(struct device *dev, dma_addr_t dma_addr,
spin_unlock_irqrestore(&arena->lock, flags);
- DBGA2("pci_unmap_single: sg [%llx,%zx] np %ld from %p\n",
+ DBGA2("pci_unmap_single: sg [%llx,%zx] np %ld from %pf\n",
dma_addr, size, npages, __builtin_return_address(0));
}
@@ -447,7 +447,7 @@ try_again:
cpu_addr = (void *)__get_free_pages(gfp, order);
if (! cpu_addr) {
printk(KERN_INFO "pci_alloc_consistent: "
- "get_free_pages failed from %p\n",
+ "get_free_pages failed from %pf\n",
__builtin_return_address(0));
/* ??? Really atomic allocation? Otherwise we could play
with vmalloc and sg if we can't find contiguous memory. */
@@ -466,7 +466,7 @@ try_again:
goto try_again;
}
- DBGA2("pci_alloc_consistent: %zx -> [%p,%llx] from %p\n",
+ DBGA2("pci_alloc_consistent: %zx -> [%p,%llx] from %pf\n",
size, cpu_addr, *dma_addrp, __builtin_return_address(0));
return cpu_addr;
@@ -486,7 +486,7 @@ static void alpha_pci_free_coherent(struct device *dev, size_t size,
pci_unmap_single(pdev, dma_addr, size, PCI_DMA_BIDIRECTIONAL);
free_pages((unsigned long)cpu_addr, get_order(size));
- DBGA2("pci_free_consistent: [%llx,%zx] from %p\n",
+ DBGA2("pci_free_consistent: [%llx,%zx] from %pf\n",
dma_addr, size, __builtin_return_address(0));
}
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 3/9] arm: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 1/9] sparc: Use vsprintf extention %pf with builtin_return_address Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 2/9] alpha: " Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 4/9] microblaze: " Joe Perches
` (5 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: Jiri Kosina; +Cc: Russell King, linux-arm-kernel, linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
arch/arm/nwfpe/fpmodule.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/nwfpe/fpmodule.c b/arch/arm/nwfpe/fpmodule.c
index cb7658e..4e729f0 100644
--- a/arch/arm/nwfpe/fpmodule.c
+++ b/arch/arm/nwfpe/fpmodule.c
@@ -147,7 +147,7 @@ void float_raise(signed char flags)
#ifdef CONFIG_DEBUG_USER
if (flags & debug)
printk(KERN_DEBUG
- "NWFPE: %s[%d] takes exception %08x at %p from %08lx\n",
+ "NWFPE: %s[%d] takes exception %08x at %pf from %08lx\n",
current->comm, current->pid, flags,
__builtin_return_address(0), GET_USERREG()->ARM_pc);
#endif
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 4/9] microblaze: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
` (2 preceding siblings ...)
2012-02-28 18:49 ` [PATCH trivial next 3/9] arm: " Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-03-13 8:24 ` Michal Simek
2012-02-28 18:49 ` [PATCH trivial next 5/9] powerpc: " Joe Perches
` (4 subsequent siblings)
8 siblings, 1 reply; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: Jiri Kosina; +Cc: Michal Simek, microblaze-uclinux, linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
arch/microblaze/mm/pgtable.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/microblaze/mm/pgtable.c b/arch/microblaze/mm/pgtable.c
index 59bf233..e3a68bb 100644
--- a/arch/microblaze/mm/pgtable.c
+++ b/arch/microblaze/mm/pgtable.c
@@ -80,7 +80,7 @@ static void __iomem *__ioremap(phys_addr_t addr, unsigned long size,
!(p >= virt_to_phys((unsigned long)&__bss_stop) &&
p < virt_to_phys((unsigned long)__bss_stop))) {
printk(KERN_WARNING "__ioremap(): phys addr "PTE_FMT
- " is RAM lr %p\n", (unsigned long)p,
+ " is RAM lr %pf\n", (unsigned long)p,
__builtin_return_address(0));
return NULL;
}
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 5/9] powerpc: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
` (3 preceding siblings ...)
2012-02-28 18:49 ` [PATCH trivial next 4/9] microblaze: " Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 6/9] parisc: " Joe Perches
` (3 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: Jiri Kosina
Cc: Benjamin Herrenschmidt, Paul Mackerras, linuxppc-dev,
linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
arch/powerpc/mm/pgtable_32.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/powerpc/mm/pgtable_32.c b/arch/powerpc/mm/pgtable_32.c
index 51f8795..0907f92 100644
--- a/arch/powerpc/mm/pgtable_32.c
+++ b/arch/powerpc/mm/pgtable_32.c
@@ -207,7 +207,7 @@ __ioremap_caller(phys_addr_t addr, unsigned long size, unsigned long flags,
*/
if (mem_init_done && (p < virt_to_phys(high_memory)) &&
!(__allow_ioremap_reserved && memblock_is_region_reserved(p, size))) {
- printk("__ioremap(): phys addr 0x%llx is RAM lr %p\n",
+ printk("__ioremap(): phys addr 0x%llx is RAM lr %pf\n",
(unsigned long long)p, __builtin_return_address(0));
return NULL;
}
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 6/9] parisc: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
` (4 preceding siblings ...)
2012-02-28 18:49 ` [PATCH trivial next 5/9] powerpc: " Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 7/9] scsi: " Joe Perches
` (2 subsequent siblings)
8 siblings, 0 replies; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: James E.J. Bottomley, Helge Deller, Jiri Kosina
Cc: linux-parisc, linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/parisc/superio.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/parisc/superio.c b/drivers/parisc/superio.c
index e3b76d4..5003458 100644
--- a/drivers/parisc/superio.c
+++ b/drivers/parisc/superio.c
@@ -348,7 +348,7 @@ int superio_fixup_irq(struct pci_dev *pcidev)
BUG();
return -1;
}
- printk("superio_fixup_irq(%s) ven 0x%x dev 0x%x from %p\n",
+ printk("superio_fixup_irq(%s) ven 0x%x dev 0x%x from %pf\n",
pci_name(pcidev),
pcidev->vendor, pcidev->device,
__builtin_return_address(0));
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 7/9] scsi: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
` (5 preceding siblings ...)
2012-02-28 18:49 ` [PATCH trivial next 6/9] parisc: " Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 21:07 ` David Miller
2012-02-28 18:49 ` [PATCH trivial next 8/9] staging: ramster: " Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 9/9] gadget: " Joe Perches
8 siblings, 1 reply; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: James E.J. Bottomley, Jiri Kosina; +Cc: linux-scsi, linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/scsi/esp_scsi.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/scsi/esp_scsi.c b/drivers/scsi/esp_scsi.c
index 394ed9e..34552bf 100644
--- a/drivers/scsi/esp_scsi.c
+++ b/drivers/scsi/esp_scsi.c
@@ -1000,7 +1000,7 @@ static int esp_check_spur_intr(struct esp *esp)
static void esp_schedule_reset(struct esp *esp)
{
- esp_log_reset("ESP: esp_schedule_reset() from %p\n",
+ esp_log_reset("ESP: esp_schedule_reset() from %pf\n",
__builtin_return_address(0));
esp->flags |= ESP_FLAG_RESETTING;
esp_event(esp, ESP_EVENT_RESET);
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 8/9] staging: ramster: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
` (6 preceding siblings ...)
2012-02-28 18:49 ` [PATCH trivial next 7/9] scsi: " Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 9/9] gadget: " Joe Perches
8 siblings, 0 replies; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: Jiri Kosina; +Cc: Greg Kroah-Hartman, devel, linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/staging/ramster/cluster/heartbeat.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/ramster/cluster/heartbeat.c b/drivers/staging/ramster/cluster/heartbeat.c
index 0020949..fe1db52 100644
--- a/drivers/staging/ramster/cluster/heartbeat.c
+++ b/drivers/staging/ramster/cluster/heartbeat.c
@@ -398,7 +398,7 @@ int r2hb_register_callback(const char *region_uuid,
up_write(&r2hb_callback_sem);
ret = 0;
out:
- mlog(ML_CLUSTER, "returning %d on behalf of %p for funcs %p\n",
+ mlog(ML_CLUSTER, "returning %d on behalf of %pf for funcs %pf\n",
ret, __builtin_return_address(0), hc);
return ret;
}
@@ -409,7 +409,7 @@ void r2hb_unregister_callback(const char *region_uuid,
{
BUG_ON(hc->hc_magic != R2HB_CB_MAGIC);
- mlog(ML_CLUSTER, "on behalf of %p for funcs %p\n",
+ mlog(ML_CLUSTER, "on behalf of %pf for funcs %pf\n",
__builtin_return_address(0), hc);
/* XXX Can this happen _with_ a region reference? */
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH trivial next 9/9] gadget: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
` (7 preceding siblings ...)
2012-02-28 18:49 ` [PATCH trivial next 8/9] staging: ramster: " Joe Perches
@ 2012-02-28 18:49 ` Joe Perches
2012-02-28 19:39 ` Felipe Balbi
8 siblings, 1 reply; 14+ messages in thread
From: Joe Perches @ 2012-02-28 18:49 UTC (permalink / raw
To: Felipe Balbi, Jiri Kosina; +Cc: Greg Kroah-Hartman, linux-usb, linux-kernel
Emit the function name not the address when possible.
builtin_return_address() gives an address. When building
a kernel with CONFIG_KALLSYMS, emit the actual function
name not the address.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/usb/gadget/u_serial.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/usb/gadget/u_serial.c b/drivers/usb/gadget/u_serial.c
index 6597a68..a553bee 100644
--- a/drivers/usb/gadget/u_serial.c
+++ b/drivers/usb/gadget/u_serial.c
@@ -920,7 +920,7 @@ static int gs_put_char(struct tty_struct *tty, unsigned char ch)
unsigned long flags;
int status;
- pr_vdebug("gs_put_char: (%d,%p) char=0x%x, called from %p\n",
+ pr_vdebug("gs_put_char: (%d,%p) char=0x%x, called from %pf\n",
port->port_num, tty, ch, __builtin_return_address(0));
spin_lock_irqsave(&port->port_lock, flags);
--
1.7.8.111.gad25c.dirty
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH trivial next 9/9] gadget: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 ` [PATCH trivial next 9/9] gadget: " Joe Perches
@ 2012-02-28 19:39 ` Felipe Balbi
0 siblings, 0 replies; 14+ messages in thread
From: Felipe Balbi @ 2012-02-28 19:39 UTC (permalink / raw
To: Joe Perches
Cc: Felipe Balbi, Jiri Kosina, Greg Kroah-Hartman, linux-usb,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1098 bytes --]
On Tue, Feb 28, 2012 at 10:49:38AM -0800, Joe Perches wrote:
> Emit the function name not the address when possible.
>
> builtin_return_address() gives an address. When building
> a kernel with CONFIG_KALLSYMS, emit the actual function
> name not the address.
>
> Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Felipe Balbi <balbi@ti.com>
> ---
> drivers/usb/gadget/u_serial.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/usb/gadget/u_serial.c b/drivers/usb/gadget/u_serial.c
> index 6597a68..a553bee 100644
> --- a/drivers/usb/gadget/u_serial.c
> +++ b/drivers/usb/gadget/u_serial.c
> @@ -920,7 +920,7 @@ static int gs_put_char(struct tty_struct *tty, unsigned char ch)
> unsigned long flags;
> int status;
>
> - pr_vdebug("gs_put_char: (%d,%p) char=0x%x, called from %p\n",
> + pr_vdebug("gs_put_char: (%d,%p) char=0x%x, called from %pf\n",
> port->port_num, tty, ch, __builtin_return_address(0));
>
> spin_lock_irqsave(&port->port_lock, flags);
> --
> 1.7.8.111.gad25c.dirty
>
--
balbi
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH trivial next 7/9] scsi: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 ` [PATCH trivial next 7/9] scsi: " Joe Perches
@ 2012-02-28 21:07 ` David Miller
0 siblings, 0 replies; 14+ messages in thread
From: David Miller @ 2012-02-28 21:07 UTC (permalink / raw
To: joe; +Cc: JBottomley, trivial, linux-scsi, linux-kernel
From: Joe Perches <joe@perches.com>
Date: Tue, 28 Feb 2012 10:49:36 -0800
> Emit the function name not the address when possible.
>
> builtin_return_address() gives an address. When building
> a kernel with CONFIG_KALLSYMS, emit the actual function
> name not the address.
>
> Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH trivial next 1/9] sparc: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 ` [PATCH trivial next 1/9] sparc: Use vsprintf extention %pf with builtin_return_address Joe Perches
@ 2012-02-28 21:08 ` David Miller
0 siblings, 0 replies; 14+ messages in thread
From: David Miller @ 2012-02-28 21:08 UTC (permalink / raw
To: joe; +Cc: trivial, sparclinux, linux-kernel
From: Joe Perches <joe@perches.com>
Date: Tue, 28 Feb 2012 10:49:30 -0800
> Emit the function name not the address when possible.
>
> builtin_return_address() gives an address. When building
> a kernel with CONFIG_KALLSYMS, emit the actual function
> name not the address.
>
> Signed-off-by: Joe Perches <joe@perches.com>
Applied to sparc-next, thanks Joe.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH trivial next 4/9] microblaze: Use vsprintf extention %pf with builtin_return_address
2012-02-28 18:49 ` [PATCH trivial next 4/9] microblaze: " Joe Perches
@ 2012-03-13 8:24 ` Michal Simek
0 siblings, 0 replies; 14+ messages in thread
From: Michal Simek @ 2012-03-13 8:24 UTC (permalink / raw
To: Joe Perches; +Cc: Jiri Kosina, microblaze-uclinux, linux-kernel
Joe Perches wrote:
> Emit the function name not the address when possible.
>
> builtin_return_address() gives an address. When building
> a kernel with CONFIG_KALLSYMS, emit the actual function
> name not the address.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> arch/microblaze/mm/pgtable.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
Applied.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2012-03-13 8:24 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-28 18:49 [PATCH trivial next 0/9] treewide: Use vsprintf extention %pf Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 1/9] sparc: Use vsprintf extention %pf with builtin_return_address Joe Perches
2012-02-28 21:08 ` David Miller
2012-02-28 18:49 ` [PATCH trivial next 2/9] alpha: " Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 3/9] arm: " Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 4/9] microblaze: " Joe Perches
2012-03-13 8:24 ` Michal Simek
2012-02-28 18:49 ` [PATCH trivial next 5/9] powerpc: " Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 6/9] parisc: " Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 7/9] scsi: " Joe Perches
2012-02-28 21:07 ` David Miller
2012-02-28 18:49 ` [PATCH trivial next 8/9] staging: ramster: " Joe Perches
2012-02-28 18:49 ` [PATCH trivial next 9/9] gadget: " Joe Perches
2012-02-28 19:39 ` Felipe Balbi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).