[LU-16314] Migrate from %p to %pK by default for printing pointers Created: 15/Nov/22  Updated: 06/Sep/23

Status: Open
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Minor
Reporter: Shaun Tancheff Assignee: Shaun Tancheff
Resolution: Unresolved Votes: 0
Labels: None

Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

Periodically when debugging a lustre issue on newer kernels developers run into the default behavior of %p to hash the address. While this safe default hinders bad actors it also frustrates developers.

Lets start by defaulting %p to %pK and slowly decorating pointers with the available pointer decorators. See: https://docs.kernel.org/core-api/printk-formats.html



 Comments   
Comment by Gerrit Updater [ 15/Nov/22 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49159
Subject: LU-16314 osc: Migrate %p to %pK
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f11bf9e08e6e45774f5cf6730cc53b9cb470e8fa

Comment by Gerrit Updater [ 29/Nov/22 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49271
Subject: LU-16314 osd-ldiskfs: Migrate %p to %pK
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 50d8c306fad1256e9590b025bb8afd385d6179bf

Comment by Gerrit Updater [ 14/Dec/22 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49405
Subject: LU-16314 obdclass: Migrate %p to $pK
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7b6ba64860ececf98b7d1cc8ea06f881c8581b44

Comment by Gerrit Updater [ 05/Jun/23 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51212
Subject: LU-16314 llite: Prefer %pK with seq_printf
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c40d20e2b0daa9f7da6802fa963f8d396cb25592

Comment by Gerrit Updater [ 05/Jun/23 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51213
Subject: LU-16314 llite: Migrate LASSERTF %p to %px
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bee0fdbf0e517fd7c203a8d3552dec9310183c35

Comment by Gerrit Updater [ 06/Jun/23 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51231
Subject: LU-16314 lnet: Migrate LASSERTF %p to %px
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3993c56b471324683dc884ed55788bc485796307

Comment by Gerrit Updater [ 06/Aug/23 ]

"Shaun Tancheff <shaun.tancheff@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51877
Subject: LU-16314 debug: Enable optional unhashed pointers
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5ca14e48ebbc564a123901d3d65253f6e12dc69d

Comment by Gerrit Updater [ 06/Sep/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51212/
Subject: LU-16314 llite: Prefer %pK with seq_printf
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: bdb6033bf7351ff763ceee8b5cfeb7f5e4425ada

Generated at Sat Feb 10 03:25:54 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.