Re: [Xen-devel] [PATCH 1/2] treewide: Switch printk users from %pf and %pF to %ps and %pS, respectively
On Fri, Mar 22, 2019 at 2:21 PM Sakari Ailus
wrote:
>
> %pF and %pf are functionally equivalent to %pS and %ps conversion
> specifiers. The former are deprecated, therefore switch the current users
> to use the preferred variant.
>
> The changes have been produced by the following command:
>
> git grep -l '%p[fF]' | grep -v '^\(tools\|Documentation\)/' | \
> while read i; do perl -i -pe 's/%pf/%ps/g; s/%pF/%pS/g;' $i; done
>
> And verifying the result.
>
> Signed-off-by: Sakari Ailus
Acked-by: Rafael J. Wysocki
> ---
> arch/alpha/kernel/pci_iommu.c | 20 ++--
> arch/arm/mach-imx/pm-imx6.c | 2 +-
> arch/arm/mm/alignment.c | 2 +-
> arch/arm/nwfpe/fpmodule.c | 2 +-
> arch/microblaze/mm/pgtable.c| 2 +-
> arch/sparc/kernel/ds.c | 2 +-
> arch/um/kernel/sysrq.c | 2 +-
> arch/x86/include/asm/trace/exceptions.h | 2 +-
> arch/x86/kernel/irq_64.c| 2 +-
> arch/x86/mm/extable.c | 4 ++--
> arch/x86/xen/multicalls.c | 2 +-
> drivers/acpi/device_pm.c| 2 +-
> drivers/base/power/main.c | 6 +++---
> drivers/base/syscore.c | 12 ++--
> drivers/block/drbd/drbd_receiver.c | 2 +-
> drivers/block/floppy.c | 10 +-
> drivers/cpufreq/cpufreq.c | 2 +-
> drivers/mmc/core/quirks.h | 2 +-
> drivers/nvdimm/bus.c| 2 +-
> drivers/nvdimm/dimm_devs.c | 2 +-
> drivers/pci/pci-driver.c| 14 +++---
> drivers/pci/quirks.c| 4 ++--
> drivers/pnp/quirks.c| 2 +-
> drivers/scsi/esp_scsi.c | 2 +-
> fs/btrfs/tests/free-space-tree-tests.c | 4 ++--
> fs/f2fs/f2fs.h | 2 +-
> fs/pstore/inode.c | 2 +-
> include/trace/events/btrfs.h| 2 +-
> include/trace/events/cpuhp.h| 4 ++--
> include/trace/events/preemptirq.h | 2 +-
> include/trace/events/rcu.h | 4 ++--
> include/trace/events/sunrpc.h | 2 +-
> include/trace/events/timer.h| 8
> include/trace/events/vmscan.h | 4 ++--
> include/trace/events/workqueue.h| 4 ++--
> include/trace/events/xen.h | 2 +-
> init/main.c | 6 +++---
> kernel/async.c | 4 ++--
> kernel/events/uprobes.c | 2 +-
> kernel/fail_function.c | 2 +-
> kernel/irq/debugfs.c| 2 +-
> kernel/irq/handle.c | 2 +-
> kernel/irq/manage.c | 2 +-
> kernel/irq/spurious.c | 4 ++--
> kernel/rcu/tree.c | 2 +-
> kernel/stop_machine.c | 2 +-
> kernel/time/sched_clock.c | 2 +-
> kernel/time/timer.c | 2 +-
> kernel/workqueue.c | 12 ++--
> lib/error-inject.c | 2 +-
> lib/percpu-refcount.c | 4 ++--
> mm/memblock.c | 12 ++--
> mm/memory.c | 2 +-
> mm/vmscan.c | 2 +-
> net/ceph/osd_client.c | 2 +-
> net/core/net-procfs.c | 2 +-
> net/core/netpoll.c | 4 ++--
> 57 files changed, 109 insertions(+), 109 deletions(-)
>
> diff --git a/arch/alpha/kernel/pci_iommu.c b/arch/alpha/kernel/pci_iommu.c
> index 3034d6d936d2..242108439f42 100644
> --- a/arch/alpha/kernel/pci_iommu.c
> +++ b/arch/alpha/kernel/pci_iommu.c
> @@ -249,7 +249,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 %pf\n",
> + DBGA("pci_dac_dma_supported %s from %ps\n",
> ok ? "yes" : "no", __builtin_return_address(0));
>
> return ok;
> @@ -281,7 +281,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 %pf\n",
> + DBGA2("pci_map_single: [%p,%zx] -> direct %llx from %ps\n",
> cpu_addr, size, ret, __builtin_return_address(0));
>
> return ret;
> @@ -292,7 +292,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 %pf\n",
> + DBGA2("pci_map_single: [%p,%zx] -> DAC %llx from %ps\n",
>
Re: [Xen-devel] [PATCH 1/2] treewide: Switch printk users from %pf and %pF to %ps and %pS, respectively
On Fri, Mar 22, 2019 at 03:21:07PM +0200, Sakari Ailus wrote: > %pF and %pf are functionally equivalent to %pS and %ps conversion > specifiers. The former are deprecated, therefore switch the current users > to use the preferred variant. > > The changes have been produced by the following command: > > git grep -l '%p[fF]' | grep -v '^\(tools\|Documentation\)/' | \ > while read i; do perl -i -pe 's/%pf/%ps/g; s/%pF/%pS/g;' $i; done > > And verifying the result. > > Signed-off-by: Sakari Ailus > --- For > mm/memblock.c | 12 ++-- Acked-by: Mike Rapoport ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH 1/2] treewide: Switch printk users from %pf and %pF to %ps and %pS, respectively
On Fri, Mar 22, 2019 at 03:21:07PM +0200, Sakari Ailus wrote: > %pF and %pf are functionally equivalent to %pS and %ps conversion > specifiers. The former are deprecated, therefore switch the current users > to use the preferred variant. > > The changes have been produced by the following command: > > git grep -l '%p[fF]' | grep -v '^\(tools\|Documentation\)/' | \ > while read i; do perl -i -pe 's/%pf/%ps/g; s/%pF/%pS/g;' $i; done > > And verifying the result. > > Signed-off-by: Sakari Ailus > --- For > fs/btrfs/tests/free-space-tree-tests.c | 4 ++-- > include/trace/events/btrfs.h| 2 +- Acked-by: David Sterba ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
[Xen-devel] [PATCH 1/2] treewide: Switch printk users from %pf and %pF to %ps and %pS, respectively
%pF and %pf are functionally equivalent to %pS and %ps conversion
specifiers. The former are deprecated, therefore switch the current users
to use the preferred variant.
The changes have been produced by the following command:
git grep -l '%p[fF]' | grep -v '^\(tools\|Documentation\)/' | \
while read i; do perl -i -pe 's/%pf/%ps/g; s/%pF/%pS/g;' $i; done
And verifying the result.
Signed-off-by: Sakari Ailus
---
arch/alpha/kernel/pci_iommu.c | 20 ++--
arch/arm/mach-imx/pm-imx6.c | 2 +-
arch/arm/mm/alignment.c | 2 +-
arch/arm/nwfpe/fpmodule.c | 2 +-
arch/microblaze/mm/pgtable.c| 2 +-
arch/sparc/kernel/ds.c | 2 +-
arch/um/kernel/sysrq.c | 2 +-
arch/x86/include/asm/trace/exceptions.h | 2 +-
arch/x86/kernel/irq_64.c| 2 +-
arch/x86/mm/extable.c | 4 ++--
arch/x86/xen/multicalls.c | 2 +-
drivers/acpi/device_pm.c| 2 +-
drivers/base/power/main.c | 6 +++---
drivers/base/syscore.c | 12 ++--
drivers/block/drbd/drbd_receiver.c | 2 +-
drivers/block/floppy.c | 10 +-
drivers/cpufreq/cpufreq.c | 2 +-
drivers/mmc/core/quirks.h | 2 +-
drivers/nvdimm/bus.c| 2 +-
drivers/nvdimm/dimm_devs.c | 2 +-
drivers/pci/pci-driver.c| 14 +++---
drivers/pci/quirks.c| 4 ++--
drivers/pnp/quirks.c| 2 +-
drivers/scsi/esp_scsi.c | 2 +-
fs/btrfs/tests/free-space-tree-tests.c | 4 ++--
fs/f2fs/f2fs.h | 2 +-
fs/pstore/inode.c | 2 +-
include/trace/events/btrfs.h| 2 +-
include/trace/events/cpuhp.h| 4 ++--
include/trace/events/preemptirq.h | 2 +-
include/trace/events/rcu.h | 4 ++--
include/trace/events/sunrpc.h | 2 +-
include/trace/events/timer.h| 8
include/trace/events/vmscan.h | 4 ++--
include/trace/events/workqueue.h| 4 ++--
include/trace/events/xen.h | 2 +-
init/main.c | 6 +++---
kernel/async.c | 4 ++--
kernel/events/uprobes.c | 2 +-
kernel/fail_function.c | 2 +-
kernel/irq/debugfs.c| 2 +-
kernel/irq/handle.c | 2 +-
kernel/irq/manage.c | 2 +-
kernel/irq/spurious.c | 4 ++--
kernel/rcu/tree.c | 2 +-
kernel/stop_machine.c | 2 +-
kernel/time/sched_clock.c | 2 +-
kernel/time/timer.c | 2 +-
kernel/workqueue.c | 12 ++--
lib/error-inject.c | 2 +-
lib/percpu-refcount.c | 4 ++--
mm/memblock.c | 12 ++--
mm/memory.c | 2 +-
mm/vmscan.c | 2 +-
net/ceph/osd_client.c | 2 +-
net/core/net-procfs.c | 2 +-
net/core/netpoll.c | 4 ++--
57 files changed, 109 insertions(+), 109 deletions(-)
diff --git a/arch/alpha/kernel/pci_iommu.c b/arch/alpha/kernel/pci_iommu.c
index 3034d6d936d2..242108439f42 100644
--- a/arch/alpha/kernel/pci_iommu.c
+++ b/arch/alpha/kernel/pci_iommu.c
@@ -249,7 +249,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 %pf\n",
+ DBGA("pci_dac_dma_supported %s from %ps\n",
ok ? "yes" : "no", __builtin_return_address(0));
return ok;
@@ -281,7 +281,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 %pf\n",
+ DBGA2("pci_map_single: [%p,%zx] -> direct %llx from %ps\n",
cpu_addr, size, ret, __builtin_return_address(0));
return ret;
@@ -292,7 +292,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 %pf\n",
+ DBGA2("pci_map_single: [%p,%zx] -> DAC %llx from %ps\n",
cpu_addr, size, ret, __builtin_return_address(0));
return ret;
@@ -329,7 +329,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;
