linux-kernel - Re: [PATCH 0/6] Additional documentation for nova-core

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-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <D9EH7958IAJZ.1R6EJXUR5H3NB@nvidia.com>
Date: Thu, 24 Apr 2025 10:25:42 +0900
From: "Alexandre Courbot" <acourbot@...dia.com>
To: "Joel Fernandes" <joelagnelf@...dia.com>, <linux-kernel@...r.kernel.org>
Cc: "Danilo Krummrich" <dakr@...nel.org>, "David Airlie"
 <airlied@...il.com>, "Simona Vetter" <simona@...ll.ch>,
 <nouveau@...ts.freedesktop.org>, <dri-devel@...ts.freedesktop.org>, "John
 Hubbard" <jhubbard@...dia.com>, "Shirish Baskaran" <sbaskaran@...dia.com>,
 "Alistair Popple" <apopple@...dia.com>, "Timur Tabi" <ttabi@...dia.com>,
 "Ben Skeggs" <bskeggs@...dia.com>
Subject: Re: [PATCH 0/6] Additional documentation for nova-core

Thanks a lot for doing this, this was severely missing from the
WPR2 patchset.

Due to the strong focus on documentation, and in order to ease merging,
I think it makes sense to keep this separate from the WPR2 patchset and
merge it on top of it. Danilo, would that work for you?

On Thu Apr 24, 2025 at 7:53 AM JST, Joel Fernandes wrote:
> Hello,
> Please find in this series, several clarifications, diagrams and code comments
> for various things in the nova-core driver. These are essential to develop an
> understanding how nova-core's boot initialization works and aid in development.
>
> These patches are on top of Alex's last posting for GSP WPR2 [1]
>
> [1] https://lore.kernel.org/all/20250420-nova-frts-v1-0-ecd1cca23963@nvidia.com/
>
> Joel Fernandes (6):
>   nova-core: doc: Add code comments related to devinit
>   nova-core: doc: Clarify sysmembar operations
>   nova-core: docs: Document vbios layout
>   nova-core: docs: Document fwsec operation and layout
>   gpu: nova-core: Clarify fields in FalconAppifHdrV1
>   nova-core: docs: Document devinit process
>
>  Documentation/gpu/nova/core/devinit.rst |  54 ++++++++
>  Documentation/gpu/nova/core/fwsec.rst   | 173 ++++++++++++++++++++++++
>  Documentation/gpu/nova/core/vbios.rst   | 155 +++++++++++++++++++++
>  Documentation/gpu/nova/index.rst        |   2 +
>  drivers/gpu/nova-core/devinit.rs        |  36 ++++-
>  drivers/gpu/nova-core/firmware/fwsec.rs |  17 ++-
>  drivers/gpu/nova-core/gpu.rs            |  11 +-
>  drivers/gpu/nova-core/regs.rs           |  17 ++-
>  8 files changed, 452 insertions(+), 13 deletions(-)
>  create mode 100644 Documentation/gpu/nova/core/devinit.rst
>  create mode 100644 Documentation/gpu/nova/core/fwsec.rst
>  create mode 100644 Documentation/gpu/nova/core/vbios.rst