linux-kernel - [PATCH RFC 00/13] Simplify the print format string with new '%pD'

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 15 Jul 2021 11:15:20 +0800
From:   Jia He <justin.he@....com>
To:     linux-kernel@...r.kernel.org
Cc:     Linus Torvalds <torvalds@...ux-foundation.org>,
        Christoph Hellwig <hch@...radead.org>, nd@....com,
        Jia He <justin.he@....com>
Subject: [PATCH RFC 00/13] Simplify the print format string with new '%pD'

This is the followup work after changing the behavior of '%pD' to
print the full path of file.

Background
==========
Linus suggested printing the full path of file instead of printing
the components as '%pd'.

This series is based on my patch series of making '%pD' print the full
path of file [1], which is stable now.

[1]: https://lkml.org/lkml/2021/7/14/1519

Test
====
I only tested it with basic compilation and booting. All the changed
codes are compiled and built successfully on Arm64. Therefore set it as
RFC.

Patch details
=============
Patches 01 to 06 with minor changes are easy for review. They are to
remove the hard coding and the postfix number of previous '%pD'. This
should be removed after the '%pD' behavior is changed.

Patches 07 to 13 are changed to simplify the printing helpers.

Jia He (13):
  s390/hmcdrv: remove the redundant directory path in format string
  afs: Remove the number postfix of '%pD' in format string
  fs: Remove the number postfix of '%pD' in format string
  NFS: Remove the number postfix of '%pD' in format string
  NFSD: Remove the number postfix of '%pD' in format string
  ovl: remove the number postfix of '%pD' in format string
  iomap: simplify iomap_swapfile_fail() with '%pD' specifier
  fs/coredump: simplify the printing with '%pD' and '%pd' specifier
  mm/filemap: simplify the printing with '%pD' specifier
  usb: gadget: simplify the printing with '%pD' specifier
  md/bitmap: simplify the printing with '%pD' specifier
  mm: simplify the printing with '%pd' specifier
  ext4: simplify the printing with '%pD' specifier

 drivers/md/md-bitmap.c                       | 13 ++-------
 drivers/s390/char/hmcdrv_dev.c               | 10 +++----
 drivers/usb/gadget/function/f_mass_storage.c | 28 ++++++++------------
 fs/afs/mntpt.c                               |  2 +-
 fs/coredump.c                                | 26 +++---------------
 fs/exec.c                                    |  2 +-
 fs/ext4/super.c                              | 12 +++------
 fs/ioctl.c                                   |  2 +-
 fs/iomap/direct-io.c                         |  2 +-
 fs/iomap/swapfile.c                          |  8 +-----
 fs/nfs/dir.c                                 | 12 ++++-----
 fs/nfs/direct.c                              |  4 +--
 fs/nfs/file.c                                | 26 +++++++++---------
 fs/nfs/nfs4file.c                            |  2 +-
 fs/nfs/write.c                               |  2 +-
 fs/nfsd/nfs4state.c                          |  2 +-
 fs/overlayfs/file.c                          |  2 +-
 fs/read_write.c                              |  2 +-
 fs/splice.c                                  |  2 +-
 mm/filemap.c                                 |  7 +----
 mm/memory.c                                  | 16 +++++------
 21 files changed, 65 insertions(+), 117 deletions(-)

-- 
2.17.1