public inbox for rust-for-linux@vger.kernel.org
 help / color / mirror / Atom feed
From: Timur Tabi <ttabi@nvidia.com>
To: Gary Guo <gary@garyguo.net>, Alice Ryhl <aliceryhl@google.com>,
	<mmaurer@google.com>, Danilo Krummrich <dakr@kernel.org>,
	Alexandre Courbot <acourbot@nvidia.com>,
	John Hubbard <jhubbard@nvidia.com>,
	Joel Fernandes <joelagnelf@nvidia.com>,
	<nouveau@lists.freedesktop.org>, <rust-for-linux@vger.kernel.org>
Subject: [PATCH v6 0/7] gpu: nova-core: expose the logging buffers via debugfs
Date: Wed, 28 Jan 2026 20:28:30 -0600	[thread overview]
Message-ID: <20260129022837.4133832-1-ttabi@nvidia.com> (raw)

GSP-RM writes its printf message to "logging buffers", which are blocks
memory allocated by the driver.  The messages are encoded, so exposing
the buffers as debugfs entries allows the buffers to be extracted and
decoded by a special application.

When the driver loads, a /sys/kernel/debug/nova_core root entry is
created.  To do this, the normal module_pci_driver! macro call is
replaced with an explicit initialization function, as this allows
that debugfs entry to be created once for all GPUs.

Then in each GPU's initialization, a subdirectory based on the PCI
BDF name is created, and the logging buffer entries are created under
that.

These entries are gated on a CONFIG_NOVA_CORE_DEBUGFS Kconfig option.

Note: the debugfs entry has a file size of 0, because debugfs defaults
a 0 size and the Rust abstractions do not adjust it for the same of
the object.  Nouveau makes this adjustment manually in the driver.

Changes since v5:
1. Replaced the debugfs::BinaryWriter implementation for LogBuffer with
one for CoherentAllocation<u8>, and moved it from gsp.rs to dma.rs.
2. Updated the DEBUGFS_ROOT drop to ensure the devices are unregistered (
and thus all the device-specific debugfs entries are removed) before the
driver itself is unregistered (which removes the top-level nova_core entry).
3. Updated several SAFETY comments.
4. Removed all references to PCI in device::name
5. Removed debugfs::Dir::empty()
6. Replaced UserSliceWriter::write_buffer with write_dma
7. Replaced BinaryWriter for LogBuffer with BinaryWriter for CoherentAllocation<u8>

Timur Tabi (7):
  rust: device: add device name method
  rust: uaccess: add write_dma() for copying from DMA buffers to
    userspace
  gpu: nova-core: implement BinaryWriter for CoherentAllocation<u8>
  gpu: nova-core: Replace module_pci_driver! with explicit module init
  gpu: nova-core: use pin projection in method boot()
  gpu: nova-core: create debugfs root in module init
  gpu: nova-core: create GSP-RM logging buffers debugfs entries

 drivers/gpu/nova-core/Kconfig      | 13 ++++++++
 drivers/gpu/nova-core/gsp.rs       | 46 +++++++++++++++++++++-----
 drivers/gpu/nova-core/gsp/boot.rs  | 14 ++++----
 drivers/gpu/nova-core/nova_core.rs | 53 ++++++++++++++++++++++++++++--
 rust/helpers/device.c              |  5 +++
 rust/kernel/device.rs              | 10 ++++++
 rust/kernel/dma.rs                 | 36 +++++++++++++++++++-
 rust/kernel/uaccess.rs             | 41 +++++++++++++++++++++++
 8 files changed, 201 insertions(+), 17 deletions(-)


base-commit: cea7b66a80412e2a5b74627b89ae25f1d0110a4b
-- 
2.52.0


             reply	other threads:[~2026-01-29  2:29 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-29  2:28 Timur Tabi [this message]
2026-01-29  2:28 ` [PATCH v6 1/7] rust: device: add device name method Timur Tabi
2026-01-29  2:28 ` [PATCH v6 2/7] rust: uaccess: add write_dma() for copying from DMA buffers to userspace Timur Tabi
2026-01-29 17:24   ` Gary Guo
2026-01-31  0:14   ` Danilo Krummrich
2026-01-29  2:28 ` [PATCH v6 3/7] gpu: nova-core: implement BinaryWriter for CoherentAllocation<u8> Timur Tabi
2026-01-30  8:31   ` Alice Ryhl
2026-01-30 18:53     ` Timur Tabi
2026-01-29  2:28 ` [PATCH v6 4/7] gpu: nova-core: Replace module_pci_driver! with explicit module init Timur Tabi
2026-01-29  2:28 ` [PATCH v6 5/7] gpu: nova-core: use pin projection in method boot() Timur Tabi
2026-01-29  2:28 ` [PATCH v6 6/7] gpu: nova-core: create debugfs root in module init Timur Tabi
2026-01-30  8:34   ` Alice Ryhl
2026-01-30 14:59     ` Timur Tabi
2026-01-29  2:28 ` [PATCH v6 7/7] gpu: nova-core: create GSP-RM logging buffers debugfs entries Timur Tabi
2026-01-30 23:36   ` Timur Tabi
2026-01-30 23:58     ` Gary Guo
2026-01-31  0:07       ` John Hubbard
2026-01-31  0:10       ` Danilo Krummrich
2026-01-31  0:16         ` John Hubbard
2026-01-31  0:20           ` Danilo Krummrich
2026-01-31  0:24             ` John Hubbard
2026-01-31  4:11           ` Timur Tabi
2026-01-31  0:26         ` Timur Tabi
2026-01-31  0:35           ` Danilo Krummrich
2026-01-31  0:51             ` Timur Tabi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260129022837.4133832-1-ttabi@nvidia.com \
    --to=ttabi@nvidia.com \
    --cc=acourbot@nvidia.com \
    --cc=aliceryhl@google.com \
    --cc=dakr@kernel.org \
    --cc=gary@garyguo.net \
    --cc=jhubbard@nvidia.com \
    --cc=joelagnelf@nvidia.com \
    --cc=mmaurer@google.com \
    --cc=nouveau@lists.freedesktop.org \
    --cc=rust-for-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox