WARNING: CPU: 5 PID: 69 at drivers/gpu/drm/ttm/ttm_bo.c:139 ttm_bo_move_to_lru_tail+0x376/0x500 [ttm]

2021-03-04 Thread Jeff Layton
q_exit_rcu+0x13e/0x190 [ 147.868011] ---[ end trace 595bd7e9298cfe26 ]--- Is this a known bug? Is there a patch that fixes it? I'd be happy to test it since it's making testing in VMs difficult right now. Thanks, -- Jeff Layton ___ dri-de

softlockup in v5.15.12 in dcn20_post_unlock_program_front_end

2022-01-02 Thread Jeff Layton
ules: amdgpu I'm able to test patches if it helps. Let me know if you want other info as well. Thanks! -- Jeff Layton

Re: softlockup in v5.15.12 in dcn20_post_unlock_program_front_end

2022-01-03 Thread Jeff Layton
On Sun, 2022-01-02 at 09:30 -0500, Jeff Layton wrote: > I'm seeing a reproducible softlockup on amdgpu on v5.15.12: > > [ 861.656146] [drm:dc_dmub_srv_wait_idle [amdgpu]] *ERROR* Error waiting for > DMUB idle: status=3 > [ 861.914848] [drm:dc_dmub_srv_wait_idle [amdgpu]] *

Re: [PATCH 03/34] net/ceph: convert put_page() to put_user_page*()

2019-08-02 Thread Jeff Layton
t; - } > + put_user_pages_dirty_lock(pages, num_pages, dirty); > kvfree(pages); > } > EXPORT_SYMBOL(ceph_put_page_vector); This patch looks sane enough. Assuming that the earlier patches are OK: Acked-by: Jeff Layton ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [fs/lock] 0064b3d9f9: BUG:kernel_NULL_pointer_dereference,address

2022-07-26 Thread Jeff Layton
9f96f .config > make HOSTCC=clang-15 CC=clang-15 ARCH=i386 olddefconfig prepare > modules_prepare bzImage modules > make HOSTCC=clang-15 CC=clang-15 ARCH=i386 > INSTALL_MOD_PATH= modules_install > cd > find lib/ | cpio -o -H newc --quiet | gzip > modules.cgz > > > git clone https://github.com/intel/lkp-tests.git > cd lkp-tests > bin/lkp qemu -k -m modules.cgz job-script # job-script is > attached in this email > > # if come across any failure that blocks the test, > # please remove ~/.lkp and /lkp dir to run from a clean state. > > > -- Jeff Layton

Re: [PATCH] drm: make drm_dp_add_payload_part2 gracefully handle NULL state pointer

2023-04-17 Thread Jeff Layton
On Mon, 2023-04-17 at 11:44 +0300, Jani Nikula wrote: > On Fri, 14 Apr 2023, Lyude Paul wrote: > > On Fri, 2023-04-14 at 13:35 +0300, Jani Nikula wrote: > > > On Fri, 14 Apr 2023, Jeff Layton wrote: > > > > On Fri, 2023-04-14 at 04:40 +, Lin,

Re: [PATCH] drm: make drm_dp_add_payload_part2 gracefully handle NULL state pointer

2023-04-17 Thread Jeff Layton
On Mon, 2023-04-17 at 13:29 +0300, Jani Nikula wrote: > On Mon, 17 Apr 2023, Jeff Layton wrote: > > On Mon, 2023-04-17 at 11:44 +0300, Jani Nikula wrote: > > > On Fri, 14 Apr 2023, Lyude Paul wrote: > > > > On Fri, 2023-04-14 at 13:35 +0300, Jani Nikula wrote: >

Re: [PATCH] drm: make drm_dp_add_payload_part2 gracefully handle NULL state pointer

2023-04-17 Thread Jeff Layton
On Mon, 2023-04-17 at 10:58 +, Lin, Wayne wrote: > [AMD Official Use Only - General] > > > > > -Original Message- > > From: Jani Nikula > > Sent: Monday, April 17, 2023 6:30 PM > > To: Jeff Layton ; Lyude Paul ; Lin, > > Wayne ; Alex Deu

[PATCH v2] drm: use mgr->dev in drm_dbg_kms in drm_dp_add_payload_part2

2023-04-19 Thread Jeff Layton
ch to using mgr->dev instead of state->dev since "state" can be NULL in some cases. Link: https://bugzilla.redhat.com/show_bug.cgi?id=2184855 Suggested-by: Jani Nikula Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_mst_topology.c | 2 +- 1 file changed, 1 ins

Re: [PATCH v2] drm: use mgr->dev in drm_dbg_kms in drm_dp_add_payload_part2

2023-04-22 Thread Jeff Layton
On Wed, 2023-04-19 at 16:21 +0300, Jani Nikula wrote: > On Wed, 19 Apr 2023, Jeff Layton wrote: > > I've been experiencing some intermittent crashes down in the display > > driver code. The symptoms are ususally a line like this in dmesg: > > > > amdgpu

Re: [PATCH v2] drm: use mgr->dev in drm_dbg_kms in drm_dp_add_payload_part2

2023-04-27 Thread Jeff Layton
On Wed, 2023-04-19 at 16:54 -0400, Lyude Paul wrote: > Reviewed-by: Lyude Paul > > Thanks! > > On Wed, 2023-04-19 at 07:24 -0400, Jeff Layton wrote: > > I've been experiencing some intermittent crashes down in the display > > driver code. The symptoms are us

NULL pointer dereference in drm_dp_add_payload_part2+0xca/0x100

2023-04-08 Thread Jeff Layton
ng like this? drm_dbg_kms(state ? state->dev : NULL, "Part 1 of payload creation for %s failed, skipping part 2\n", I think that would at least prevent this problem from crashing the machine. Thanks, -- Jeff Layton

Re: NULL pointer dereference in drm_dp_add_payload_part2+0xca/0x100

2023-04-12 Thread Jeff Layton
On Sat, 2023-04-08 at 07:46 -0400, Jeff Layton wrote: > I've hit some repeated crashes in drm_dp_add_payload_part2. Here's one > from this morning that occurred not long after booting the machine. I > hadn't even logged in yet -- it was still at a gdm prompt: > >

[PATCH] drm: make drm_dp_add_payload_part2 gracefully handle NULL state pointer

2023-04-13 Thread Jeff Layton
real bug is probably in the caller of this function, which is passing it a NULL state pointer, but this patch at least keeps my machine from oopsing when this occurs. Link: https://bugzilla.redhat.com/show_bug.cgi?id=2184855 Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_mst_topol

Re: [PATCH] drm: make drm_dp_add_payload_part2 gracefully handle NULL state pointer

2023-04-13 Thread Jeff Layton
On Thu, 2023-04-13 at 15:31 +0300, Jani Nikula wrote: > On Thu, 13 Apr 2023, Jeff Layton wrote: > > I've been experiencing some intermittent crashes down in the display > > driver code. The symptoms are ususally a line like this in dmesg: > > > > amdgpu

Re: [PATCH] drm: make drm_dp_add_payload_part2 gracefully handle NULL state pointer

2023-04-14 Thread Jeff Layton
essage, my proposed patch did prevent the box from oopsing, so I'd really like to see it go in unless it's just categorically wrong for the caller to pass down a NULL state pointer to drm_dp_add_payload_part2. > > -Original Message- > > From: Alex Deucher > > Se

Re: [PATCH v2] drm: use mgr->dev in drm_dbg_kms in drm_dp_add_payload_part2

2023-06-20 Thread Jeff Layton
. Thanks, Jeff On Wed, 2023-04-19 at 16:54 -0400, Lyude Paul wrote: > Reviewed-by: Lyude Paul > > Thanks! > > On Wed, 2023-04-19 at 07:24 -0400, Jeff Layton wrote: > > I've been experiencing some intermittent crashes down in the display > > driver code. The

Re: [PATCH v2 34/47] nfsd: dynamically allocate the nfsd-client shrinker

2023-07-24 Thread Jeff Layton
+ shrinker_register(nn->nfsd_client_shrinker); > + > return 0; > > err_shrinker: > @@ -8228,7 +8232,7 @@ nfs4_state_shutdown_net(struct net *net) > struct list_head *pos, *next, reaplist; > struct nfsd_net *nn = net_generic(net, nfsd_net_id); > > - unregister_shrinker(&nn->nfsd_client_shrinker); > + shrinker_unregister(nn->nfsd_client_shrinker); > cancel_work(&nn->nfsd_shrinker_work); > cancel_delayed_work_sync(&nn->laundromat_work); > locks_end_grace(&nn->nfsd4_manager); Acked-by: Jeff Layton

Re: [PATCH v2 35/47] nfsd: dynamically allocate the nfsd-reply shrinker

2023-07-24 Thread Jeff Layton
ntrol *sc) > { > - struct nfsd_net *nn = container_of(shrink, > - struct nfsd_net, nfsd_reply_cache_shrinker); > + struct nfsd_net *nn = shrink->private_data; > > return atomic_read(&nn->num_drc_entries); > } > @@ -342,8 +344,

DPMS problems with radeon card and dual monitor setup

2023-10-02 Thread Jeff Layton
00011 R11: 0246 R12: 7ffe0a955040 [ 4125.634141] R13: c03864bc R14: 000a R15: 56075466b960 [ 4125.634156] [ 4125.634159] ---[ end trace 0000 ]--- Thanks, -- Jeff Layton

WARNING in allocate_mst_payload

2023-11-21 Thread Jeff Layton
I have a recurring problem where my workstation tries to put the monitor to sleep, which triggers a warning down in the depths of the video card driver. When I return to the machine the monitor is black, but not in powersave mode and all of the windows on my desktop have been shuffled off to the se

Re: WARNING in allocate_mst_payload

2023-11-21 Thread Jeff Layton
gt; connection and maybe individually connect them for a test if possible. > > Regards, > Christian. > > Am 21.11.23 um 14:01 schrieb Jeff Layton: > > I have a recurring problem where my workstation tries to put the monitor > > to sleep, which triggers a warning down in t

amdgpu UBSAN warnings in 6.10.0-rc5

2024-06-30 Thread Jeff Layton
bff63d0 [8.774859] [8.774864] ---[ end trace ]--- It looks like "count" probably needs to be clamped to ARRAY_SIZE(header->gpio_pin) in bios_parser_get_gpio_pin_info ? dmesg is attached. There are couple of other warnings in there too after the UBSAN one, but this one

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
above should be > converted to the new workqueue? > > Thanks, > > Alan Stern > > -- > To unsubscribe from this list: send the line "unsubscribe linux-cifs" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Jeff Layton ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
On Thu, 16 Feb 2012 08:29:51 -0800 Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > > These should all be freezable and we might even be able to get away > > with WQ_UNBOUND for some of these. > > In general, I would

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
ove should be > converted to the new workqueue? > > Thanks, > > Alan Stern > > -- > To unsubscribe from this list: send the line "unsubscribe linux-cifs" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Jeff Layton

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
On Thu, 16 Feb 2012 08:29:51 -0800 Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > > These should all be freezable and we might even be able to get away > > with WQ_UNBOUND for some of these. > > In general, I would

[PATCH v5 00/10] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-04-28 Thread Jeff Layton
re some display port and i915 changes in here too. Note that those are untested, mostly because I don't have the necessary hardware handy. Signed-off-by: Jeff Layton --- Changes in v5: - add class string to each ref_tracker_dir - auto-register debugfs file for every tracker in ref_tracker_d

[PATCH v5 05/10] ref_tracker: add a static classname string to each ref_tracker_dir

2025-04-28 Thread Jeff Layton
urpose. Add a new "class" string to the ref_tracker dir that describes the the type of object (sans any individual info for that object). Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_tunnel.c | 2 +- drivers/gpu/drm/i915/intel_runtime_pm.c | 3 ++- drivers/gpu/drm/i915

[PATCH v5 09/10] i915: add ref_tracker_dir symlinks for each tracker

2025-04-28 Thread Jeff Layton
Now that there is the ability to create a symlink for each tracker, do so for the i915 entries. Signed-off-by: Jeff Layton --- drivers/gpu/drm/i915/intel_runtime_pm.c | 1 + drivers/gpu/drm/i915/intel_wakeref.c| 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/gpu/drm/i915

[PATCH v5 06/10] ref_tracker: automatically register a file in debugfs for a ref_tracker_dir

2025-04-28 Thread Jeff Layton
ld be safe since this directory is only accessible by root If debugfs file creation fails, a pr_warn will be isssued. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 14 + lib/ref_tracker.c | 72 + 2 files changed, 86

[PATCH v5 08/10] net: add symlinks to ref_tracker_dir for netns

2025-04-28 Thread Jeff Layton
files and symlinks. Signed-off-by: Jeff Layton --- net/core/net_namespace.c | 28 +++- 1 file changed, 27 insertions(+), 1 deletion(-) diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 008de9675ea98fa8c18628b2f1c3aee7f3ebc9c6

[PATCH v5 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-04-28 Thread Jeff Layton
Now that we have dentries and the ability to create meaningful symlinks to them, don't keep a name string in each tracker. Switch the output format to print "class@address", and drop the name field. Also, add a kerneldoc header for ref_tracker_dir_init(). Signed-off-by: Jeff Layto

[PATCH v5 04/10] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-04-28 Thread Jeff Layton
Allow pr_ostream to also output directly to a seq_file without an intermediate buffer. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 25 + 1 file changed, 25 insertions(+) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index

[PATCH v5 07/10] ref_tracker: add a way to create a symlink to the ref_tracker_dir debugfs file

2025-04-28 Thread Jeff Layton
Add the ability for a subsystem to add a user-friendly symlink that points to a ref_tracker_dir's debugfs file. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 7 +++ lib/ref_tracker.c | 28 2 files changed, 35 insertions(+) diff --

[PATCH v5 02/10] ref_tracker: add a top level debugfs directory for ref_tracker

2025-04-28 Thread Jeff Layton
Add a new "ref_tracker" directory in debugfs. Each individual refcount tracker can register files under there to display info about currently-held references. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 16 1 file changed, 16 insertion

[PATCH v5 03/10] ref_tracker: have callers pass output function to pr_ostream()

2025-04-28 Thread Jeff Layton
_fmt macro. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 53 ++--- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index a66cde325920780cfe7529ea9d827d

[PATCH v5 01/10] ref_tracker: don't use %pK in pr_ostream() output

2025-04-28 Thread Jeff Layton
/20250414-restricted-pointers-net-v1-0-12af0ce46...@linutronix.de/ Cc: Thomas Weißschuh Reviewed-by: Thomas Weißschuh Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index

[PATCH v6 02/10] ref_tracker: add a top level debugfs directory for ref_tracker

2025-04-30 Thread Jeff Layton
Add a new "ref_tracker" directory in debugfs. Each individual refcount tracker can register files under there to display info about currently-held references. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 16 1 file changed, 16 insertion

[PATCH v6 07/10] ref_tracker: add a way to create a symlink to the ref_tracker_dir debugfs file

2025-04-30 Thread Jeff Layton
Add the ability for a subsystem to add a user-friendly symlink that points to a ref_tracker_dir's debugfs file. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 10 ++ lib/ref_tracker.c | 28 2 files changed, 38 insertions(+)

[PATCH v6 04/10] ref_tracker: add a static classname string to each ref_tracker_dir

2025-04-30 Thread Jeff Layton
urpose. Add a new "class" string to the ref_tracker dir that describes the the type of object (sans any individual info for that object). Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_tunnel.c | 2 +- drivers/gpu/drm/i915/intel_runtime_pm.c | 3 ++- drivers/gpu/drm/i915

[PATCH v6 06/10] ref_tracker: automatically register a file in debugfs for a ref_tracker_dir

2025-04-30 Thread Jeff Layton
ld be safe since this directory is only accessible by root If debugfs file creation fails, a pr_warn will be isssued. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 14 + lib/ref_tracker.c | 73 +++-- 2 files changed, 85

[PATCH v6 03/10] ref_tracker: have callers pass output function to pr_ostream()

2025-04-30 Thread Jeff Layton
_fmt macro. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 53 ++--- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index a66cde325920780cfe7529ea9d827d

[PATCH v6 08/10] net: add symlinks to ref_tracker_dir for netns

2025-04-30 Thread Jeff Layton
files and symlinks. Signed-off-by: Jeff Layton --- net/core/net_namespace.c | 28 +++- 1 file changed, 27 insertions(+), 1 deletion(-) diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 008de9675ea98fa8c18628b2f1c3aee7f3ebc9c6

[PATCH v6 00/10] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-04-30 Thread Jeff Layton
re some display port and i915 changes in here too. Note that those are untested, mostly because I don't have the necessary hardware handy. Signed-off-by: Jeff Layton --- Changes in v6: - clean up kerneldoc comment for ref_tracker_dir_debugfs() - add missing stub function for ref_tracker_dir_

[PATCH v6 05/10] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-04-30 Thread Jeff Layton
Allow pr_ostream to also output directly to a seq_file without an intermediate buffer. The first caller of +ref_tracker_dir_seq_print() will come in a later patch, so mark that __maybe_unused for now. That designation will be removed once it is used. Reviewed-by: Andrew Lunn Signed-off-by: Jeff

[PATCH v6 01/10] ref_tracker: don't use %pK in pr_ostream() output

2025-04-30 Thread Jeff Layton
/20250414-restricted-pointers-net-v1-0-12af0ce46...@linutronix.de/ Cc: Thomas Weißschuh Reviewed-by: Thomas Weißschuh Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index

[PATCH v6 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-04-30 Thread Jeff Layton
Now that we have dentries and the ability to create meaningful symlinks to them, don't keep a name string in each tracker. Switch the output format to print "class@address", and drop the name field. Also, add a kerneldoc header for ref_tracker_dir_init(). Signed-off-by: Jeff Layto

[PATCH v6 09/10] i915: add ref_tracker_dir symlinks for each tracker

2025-04-30 Thread Jeff Layton
Now that there is the ability to create a symlink for each tracker, do so for the i915 entries. Signed-off-by: Jeff Layton --- drivers/gpu/drm/i915/intel_runtime_pm.c | 1 + drivers/gpu/drm/i915/intel_wakeref.c| 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/gpu/drm/i915

Re: [PATCH v5 04/10] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-04-29 Thread Jeff Layton
On Tue, 2025-04-29 at 16:27 -0700, Jakub Kicinski wrote: > On Mon, 28 Apr 2025 11:26:27 -0700 Jeff Layton wrote: > > Allow pr_ostream to also output directly to a seq_file without an > > intermediate buffer. > > > > Reviewed-by: Andrew Lunn > > Signed-off-by: Je

Re: [PATCH v6 08/10] net: add symlinks to ref_tracker_dir for netns

2025-04-30 Thread Jeff Layton
On Wed, 2025-04-30 at 20:50 -0700, Kuniyuki Iwashima wrote: > From: Jeff Layton > Date: Wed, 30 Apr 2025 20:42:40 -0700 > > On Wed, 2025-04-30 at 20:07 -0700, Kuniyuki Iwashima wrote: > > > From: Jeff Layton > > > Date: Wed, 30 Apr 2025 19:59:23 -0700 > > &

Re: [PATCH v6 08/10] net: add symlinks to ref_tracker_dir for netns

2025-04-30 Thread Jeff Layton
On Wed, 2025-04-30 at 14:29 -0700, Kuniyuki Iwashima wrote: > From: Jeff Layton > Date: Wed, 30 Apr 2025 08:06:54 -0700 > > After assigning the inode number to the namespace, use it to create a > > unique name for each netns refcount tracker with the ns.inum value in >

Re: [PATCH v6 08/10] net: add symlinks to ref_tracker_dir for netns

2025-04-30 Thread Jeff Layton
On Wed, 2025-04-30 at 20:07 -0700, Kuniyuki Iwashima wrote: > From: Jeff Layton > Date: Wed, 30 Apr 2025 19:59:23 -0700 > > On Wed, 2025-04-30 at 14:29 -0700, Kuniyuki Iwashima wrote: > > > From: Jeff Layton > > > Date: Wed, 30 Apr 2025 08:06:54 -0700 > > >

[PATCH v7 06/10] ref_tracker: automatically register a file in debugfs for a ref_tracker_dir

2025-05-05 Thread Jeff Layton
ld be safe since this directory is only accessible by root If debugfs file creation fails, a pr_warn will be isssued. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 13 lib/ref_tracker.c | 73 +++-- 2 files changed, 84

[PATCH v7 07/10] ref_tracker: add a way to create a symlink to the ref_tracker_dir debugfs file

2025-05-05 Thread Jeff Layton
Add the ability for a subsystem to add a user-friendly symlink that points to a ref_tracker_dir's debugfs file. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 11 +++ lib/ref_tracker.c | 28 2 files changed, 39 insertions(+)

[PATCH v7 08/10] net: add symlinks to ref_tracker_dir for netns

2025-05-05 Thread Jeff Layton
() to register its files and symlinks. Signed-off-by: Jeff Layton --- net/core/net_namespace.c | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 008de9675ea98fa8c18628b2f1c3aee7f3ebc9c6

[PATCH v7 09/10] i915: add ref_tracker_dir symlinks for each tracker

2025-05-05 Thread Jeff Layton
Now that there is the ability to create a symlink for each tracker, do so for the i915 entries. Signed-off-by: Jeff Layton --- drivers/gpu/drm/i915/intel_runtime_pm.c | 1 + drivers/gpu/drm/i915/intel_wakeref.c| 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/gpu/drm/i915

[PATCH v7 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-05 Thread Jeff Layton
Now that we have dentries and the ability to create meaningful symlinks to them, don't keep a name string in each tracker. Switch the output format to print "class@address", and drop the name field. Also, add a kerneldoc header for ref_tracker_dir_init(). Signed-off-by: Jeff Layto

[PATCH v7 00/10] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-05-05 Thread Jeff Layton
re some display port and i915 changes in here too. Note that those are untested, mostly because I don't have the necessary hardware handy. Signed-off-by: Jeff Layton --- Changes in v7: - include net->net_cookie in netns symlink name - add __ostream_printf to ref_tracker_dir_symlink() stub fu

[PATCH v7 02/10] ref_tracker: add a top level debugfs directory for ref_tracker

2025-05-05 Thread Jeff Layton
Add a new "ref_tracker" directory in debugfs. Each individual refcount tracker can register files under there to display info about currently-held references. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 16 1 file changed, 16 insertion

[PATCH v7 03/10] ref_tracker: have callers pass output function to pr_ostream()

2025-05-05 Thread Jeff Layton
_fmt macro. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 2 ++ lib/ref_tracker.c | 51 - 2 files changed, 38 insertions(+), 15 deletions(-) diff --git a/include/linux/ref_tracker.h b/include/linux/ref

[PATCH v7 04/10] ref_tracker: add a static classname string to each ref_tracker_dir

2025-05-05 Thread Jeff Layton
urpose. Add a new "class" string to the ref_tracker dir that describes the the type of object (sans any individual info for that object). Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_tunnel.c | 2 +- drivers/gpu/drm/i915/intel_runtime_pm.c | 3 ++- drivers/gpu/drm/i915

[PATCH v7 05/10] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-05-05 Thread Jeff Layton
Allow pr_ostream to also output directly to a seq_file without an intermediate buffer. The first caller of +ref_tracker_dir_seq_print() will come in a later patch, so mark that __maybe_unused for now. That designation will be removed once it is used. Reviewed-by: Andrew Lunn Signed-off-by: Jeff

[PATCH v7 01/10] ref_tracker: don't use %pK in pr_ostream() output

2025-05-05 Thread Jeff Layton
/20250414-restricted-pointers-net-v1-0-12af0ce46...@linutronix.de/ Cc: Thomas Weißschuh Reviewed-by: Thomas Weißschuh Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index

[PATCH v8 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-07 Thread Jeff Layton
Now that we have dentries and the ability to create meaningful symlinks to them, don't keep a name string in each tracker. Switch the output format to print "class@address", and drop the name field. Also, add a kerneldoc header for ref_tracker_dir_init(). Signed-off-by: Jeff Layto

[PATCH v8 08/10] net: add symlinks to ref_tracker_dir for netns

2025-05-07 Thread Jeff Layton
() to register its files and symlinks. Signed-off-by: Jeff Layton --- net/core/net_namespace.c | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 008de9675ea98fa8c18628b2f1c3aee7f3ebc9c6

[PATCH v8 01/10] ref_tracker: don't use %pK in pr_ostream() output

2025-05-07 Thread Jeff Layton
/20250414-restricted-pointers-net-v1-0-12af0ce46...@linutronix.de/ Cc: Thomas Weißschuh Reviewed-by: Thomas Weißschuh Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index

[PATCH v8 04/10] ref_tracker: add a static classname string to each ref_tracker_dir

2025-05-07 Thread Jeff Layton
urpose. Add a new "class" string to the ref_tracker dir that describes the the type of object (sans any individual info for that object). Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_tunnel.c | 2 +- drivers/gpu/drm/i915/intel_runtime_pm.c | 3 ++- drivers/gpu/drm/i915

[PATCH v8 05/10] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-05-07 Thread Jeff Layton
Allow pr_ostream to also output directly to a seq_file without an intermediate buffer. The first caller of +ref_tracker_dir_seq_print() will come in a later patch, so mark that __maybe_unused for now. That designation will be removed once it is used. Reviewed-by: Andrew Lunn Signed-off-by: Jeff

[PATCH v8 06/10] ref_tracker: automatically register a file in debugfs for a ref_tracker_dir

2025-05-07 Thread Jeff Layton
ld be safe since this directory is only accessible by root If debugfs file creation fails, a pr_warn will be isssued. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 23 ++ lib/ref_tracker.c | 73 +++-- 2 files c

[PATCH v8 07/10] ref_tracker: add a way to create a symlink to the ref_tracker_dir debugfs file

2025-05-07 Thread Jeff Layton
Add the ability for a subsystem to add a user-friendly symlink that points to a ref_tracker_dir's debugfs file. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 13 + lib/ref_tracker.c | 28 2 files changed, 41 insertions(+)

[PATCH v8 09/10] i915: add ref_tracker_dir symlinks for each tracker

2025-05-07 Thread Jeff Layton
Now that there is the ability to create a symlink for each tracker, do so for the i915 entries. Acked-by: Jani Nikula Signed-off-by: Jeff Layton --- drivers/gpu/drm/i915/intel_runtime_pm.c | 1 + drivers/gpu/drm/i915/intel_wakeref.c| 1 + 2 files changed, 2 insertions(+) diff --git a

[PATCH v8 00/10] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-05-07 Thread Jeff Layton
Minor update to fix up the build and some compiler warnings with more esoteric Kconfigs. Signed-off-by: Jeff Layton --- Changes in v8: - fix up compiler warnings that the KTR warned about - ensure builds with CONFIG_DEBUG_FS=n and CONFIG_REF_TRACKER=y work - Link to v7: https://lore.kernel.org

[PATCH v8 03/10] ref_tracker: have callers pass output function to pr_ostream()

2025-05-07 Thread Jeff Layton
_fmt macro. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 2 ++ lib/ref_tracker.c | 51 - 2 files changed, 38 insertions(+), 15 deletions(-) diff --git a/include/linux/ref_tracker.h b/include/linux/ref

[PATCH v8 02/10] ref_tracker: add a top level debugfs directory for ref_tracker

2025-05-07 Thread Jeff Layton
Add a new "ref_tracker" directory in debugfs. Each individual refcount tracker can register files under there to display info about currently-held references. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 16 1 file changed, 16 insertion

Re: [PATCH v9 00/10] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-05-10 Thread Jeff Layton
On Fri, 2025-05-09 at 16:00 -0700, Jakub Kicinski wrote: > On Fri, 09 May 2025 11:53:36 -0400 Jeff Layton wrote: > > This one just fixes a typo in the ref_tracker_dir_init() kerneldoc > > header. I'm only resending so the CI will pick it up. > > We got a bunch of:

[PATCH v9 09/10] i915: add ref_tracker_dir symlinks for each tracker

2025-05-09 Thread Jeff Layton
Now that there is the ability to create a symlink for each tracker, do so for the i915 entries. Acked-by: Jani Nikula Signed-off-by: Jeff Layton --- drivers/gpu/drm/i915/intel_runtime_pm.c | 1 + drivers/gpu/drm/i915/intel_wakeref.c| 1 + 2 files changed, 2 insertions(+) diff --git a

Re: [PATCH v8 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-09 Thread Jeff Layton
On Wed, 2025-05-07 at 20:01 -0700, Jakub Kicinski wrote: > On Wed, 07 May 2025 09:06:35 -0400 Jeff Layton wrote: > > + * @quarantime_count: max number of entries to be tracked > > quarantime > ^ > :( Sorry, I thought I had fixed that. Do you need me to resend? The pi

[PATCH v9 00/10] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-05-09 Thread Jeff Layton
This one just fixes a typo in the ref_tracker_dir_init() kerneldoc header. I'm only resending so the CI will pick it up. Signed-off-by: Jeff Layton --- Changes in v9: - fix typo in ref_tracker_dir_init() kerneldoc header - Link to v8: https://lore.kernel.org/r/20250507-reftrack-dbgfs

[PATCH v9 03/10] ref_tracker: have callers pass output function to pr_ostream()

2025-05-09 Thread Jeff Layton
_fmt macro. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 2 ++ lib/ref_tracker.c | 51 - 2 files changed, 38 insertions(+), 15 deletions(-) diff --git a/include/linux/ref_tracker.h b/include/linux/ref

[PATCH v9 06/10] ref_tracker: automatically register a file in debugfs for a ref_tracker_dir

2025-05-09 Thread Jeff Layton
ld be safe since this directory is only accessible by root If debugfs file creation fails, a pr_warn will be isssued. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 23 ++ lib/ref_tracker.c | 73 +++-- 2 files c

[PATCH v9 08/10] net: add symlinks to ref_tracker_dir for netns

2025-05-09 Thread Jeff Layton
() to register its files and symlinks. Signed-off-by: Jeff Layton --- net/core/net_namespace.c | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 008de9675ea98fa8c18628b2f1c3aee7f3ebc9c6

[PATCH v9 01/10] ref_tracker: don't use %pK in pr_ostream() output

2025-05-09 Thread Jeff Layton
/20250414-restricted-pointers-net-v1-0-12af0ce46...@linutronix.de/ Cc: Thomas Weißschuh Reviewed-by: Thomas Weißschuh Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index

[PATCH v9 04/10] ref_tracker: add a static classname string to each ref_tracker_dir

2025-05-09 Thread Jeff Layton
urpose. Add a new "class" string to the ref_tracker dir that describes the the type of object (sans any individual info for that object). Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_tunnel.c | 2 +- drivers/gpu/drm/i915/intel_runtime_pm.c | 3 ++- drivers/gpu/drm/i915

[PATCH v9 07/10] ref_tracker: add a way to create a symlink to the ref_tracker_dir debugfs file

2025-05-09 Thread Jeff Layton
Add the ability for a subsystem to add a user-friendly symlink that points to a ref_tracker_dir's debugfs file. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 13 + lib/ref_tracker.c | 28 2 files changed, 41 insertions(+)

[PATCH v9 02/10] ref_tracker: add a top level debugfs directory for ref_tracker

2025-05-09 Thread Jeff Layton
Add a new "ref_tracker" directory in debugfs. Each individual refcount tracker can register files under there to display info about currently-held references. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 16 1 file changed, 16 insertion

[PATCH v9 05/10] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-05-09 Thread Jeff Layton
Allow pr_ostream to also output directly to a seq_file without an intermediate buffer. The first caller of +ref_tracker_dir_seq_print() will come in a later patch, so mark that __maybe_unused for now. That designation will be removed once it is used. Reviewed-by: Andrew Lunn Signed-off-by: Jeff

[PATCH v9 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-09 Thread Jeff Layton
Now that we have dentries and the ability to create meaningful symlinks to them, don't keep a name string in each tracker. Switch the output format to print "class@address", and drop the name field. Also, add a kerneldoc header for ref_tracker_dir_init(). Signed-off-by: Jeff Layto

[PATCH v10 6/9] ref_tracker: automatically register a file in debugfs for a ref_tracker_dir

2025-05-27 Thread Jeff Layton
ld be safe since this directory is only accessible by root If debugfs file creation fails, a pr_warn will be isssued. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 23 ++ lib/ref_tracker.c | 73 +++-- 2 files c

[PATCH v10 8/9] net: add symlinks to ref_tracker_dir for netns

2025-05-27 Thread Jeff Layton
() to register its files and symlinks. Signed-off-by: Jeff Layton --- net/core/net_namespace.c | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 8708eb975295ffb78de35fcf4abef7cc281f5a51

[PATCH v10 3/9] ref_tracker: have callers pass output function to pr_ostream()

2025-05-27 Thread Jeff Layton
_fmt macro. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 2 ++ lib/ref_tracker.c | 51 - 2 files changed, 38 insertions(+), 15 deletions(-) diff --git a/include/linux/ref_tracker.h b/include/linux/ref

[PATCH v10 0/9] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-05-27 Thread Jeff Layton
not truly unique. Since I don't know much about the i915 driver, I'll leave that as an exercise for that team. It should be fairly simple to add if they come up with a scheme that gives these object unique names. [1]: https://patchwork.freedesktop.org/series/148490/ Signed-off-by: J

[PATCH v10 1/9] ref_tracker: don't use %pK in pr_ostream() output

2025-05-27 Thread Jeff Layton
/20250414-restricted-pointers-net-v1-0-12af0ce46...@linutronix.de/ Cc: Thomas Weißschuh Reviewed-by: Thomas Weißschuh Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ref_tracker.c b/lib/ref_tracker.c index

[PATCH v10 9/9] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-27 Thread Jeff Layton
Now that we have dentries and the ability to create meaningful symlinks to them, don't keep a name string in each tracker. Switch the output format to print "class@address", and drop the name field. Also, add a kerneldoc header for ref_tracker_dir_init(). Signed-off-by: Jeff Layto

[PATCH v10 7/9] ref_tracker: add a way to create a symlink to the ref_tracker_dir debugfs file

2025-05-27 Thread Jeff Layton
Add the ability for a subsystem to add a user-friendly symlink that points to a ref_tracker_dir's debugfs file. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 13 + lib/ref_tracker.c | 28 2 files changed, 41 insertions(+)

[PATCH v10 2/9] ref_tracker: add a top level debugfs directory for ref_tracker

2025-05-27 Thread Jeff Layton
Add a new "ref_tracker" directory in debugfs. Each individual refcount tracker can register files under there to display info about currently-held references. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 16 1 file changed, 16 insertion

[PATCH v10 4/9] ref_tracker: add a static classname string to each ref_tracker_dir

2025-05-27 Thread Jeff Layton
urpose. Add a new "class" string to the ref_tracker dir that describes the the type of object (sans any individual info for that object). Signed-off-by: Jeff Layton --- drivers/gpu/drm/display/drm_dp_tunnel.c | 2 +- drivers/gpu/drm/i915/intel_runtime_pm.c | 3 ++- drivers/gpu/drm/i915

[PATCH v10 5/9] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-05-27 Thread Jeff Layton
Allow pr_ostream to also output directly to a seq_file without an intermediate buffer. The first caller of +ref_tracker_dir_seq_print() will come in a later patch, so mark that __maybe_unused for now. That designation will be removed once it is used. Reviewed-by: Andrew Lunn Signed-off-by: Jeff

Re: [PATCH v10 8/9] net: add symlinks to ref_tracker_dir for netns

2025-05-27 Thread Jeff Layton
On Tue, 2025-05-27 at 06:37 -0700, Eric Dumazet wrote: > On Tue, May 27, 2025 at 6:35 AM Eric Dumazet wrote: > > > > On Tue, May 27, 2025 at 4:34 AM Jeff Layton wrote: > > > > > > After assigning the inode number to the namespace, use it to create a > >

Re: [PATCH v10 9/9] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-27 Thread Jeff Layton
On Tue, 2025-05-27 at 07:33 -0400, Jeff Layton wrote: > Now that we have dentries and the ability to create meaningful symlinks > to them, don't keep a name string in each tracker. Switch the output > format to print "class@address", and drop the name field. > > Al

  1   2   >