diff options
author | Tobin C. Harding <me@tobin.cc> | 2017-11-23 00:59:45 +0100 |
---|---|---|
committer | Tobin C. Harding <me@tobin.cc> | 2017-11-29 02:13:14 +0100 |
commit | 7b1924a1d930eb27fc79c4e4e2a6c1c970623e68 (patch) | |
tree | cdd958371ba49a69d64aae5df68ac0f181ba945d /scripts/checkpatch.pl | |
parent | printk: hash addresses printed with %p (diff) | |
download | linux-7b1924a1d930eb27fc79c4e4e2a6c1c970623e68.tar.xz linux-7b1924a1d930eb27fc79c4e4e2a6c1c970623e68.zip |
vsprintf: add printk specifier %px
printk specifier %p now hashes all addresses before printing. Sometimes
we need to see the actual unmodified address. This can be achieved using
%lx but then we face the risk that if in future we want to change the
way the Kernel handles printing of pointers we will have to grep through
the already existent 50 000 %lx call sites. Let's add specifier %px as a
clear, opt-in, way to print a pointer and maintain some level of
isolation from all the other hex integer output within the Kernel.
Add printk specifier %px to print the actual unmodified address.
Signed-off-by: Tobin C. Harding <me@tobin.cc>
Diffstat (limited to 'scripts/checkpatch.pl')
-rwxr-xr-x | scripts/checkpatch.pl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 95cda3ecc66b..040aa79e1d9d 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -5753,7 +5753,7 @@ sub process { for (my $count = $linenr; $count <= $lc; $count++) { my $fmt = get_quoted_string($lines[$count - 1], raw_line($count, 0)); $fmt =~ s/%%//g; - if ($fmt =~ /(\%[\*\d\.]*p(?![\WFfSsBKRraEhMmIiUDdgVCbGNO]).)/) { + if ($fmt =~ /(\%[\*\d\.]*p(?![\WFfSsBKRraEhMmIiUDdgVCbGNOx]).)/) { $bad_extension = $1; last; } |