linux-kernel - Re: [PATCH v2 2/2] gpu: nova-core: add initial documentation

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

Message-ID: <Z6jLaCiiuuE8UPEt@pollux>
Date: Sun, 9 Feb 2025 16:36:08 +0100
From: Danilo Krummrich <dakr@...nel.org>
To: Zhi Wang <zhiw@...dia.com>
Cc: "airlied@...il.com" <airlied@...il.com>,
	"simona@...ll.ch" <simona@...ll.ch>,
	"corbet@....net" <corbet@....net>,
	"maarten.lankhorst@...ux.intel.com" <maarten.lankhorst@...ux.intel.com>,
	"mripard@...nel.org" <mripard@...nel.org>,
	"tzimmermann@...e.de" <tzimmermann@...e.de>,
	"ajanulgu@...hat.com" <ajanulgu@...hat.com>,
	"lyude@...hat.com" <lyude@...hat.com>,
	"pstanner@...hat.com" <pstanner@...hat.com>,
	Neo Jia <cjia@...dia.com>, John Hubbard <jhubbard@...dia.com>,
	Ben Skeggs <bskeggs@...dia.com>, Andy Currid <ACurrid@...dia.com>,
	"ojeda@...nel.org" <ojeda@...nel.org>,
	"alex.gaynor@...il.com" <alex.gaynor@...il.com>,
	"boqun.feng@...il.com" <boqun.feng@...il.com>,
	"gary@...yguo.net" <gary@...yguo.net>,
	"bjorn3_gh@...tonmail.com" <bjorn3_gh@...tonmail.com>,
	"benno.lossin@...ton.me" <benno.lossin@...ton.me>,
	"a.hindborg@...nel.org" <a.hindborg@...nel.org>,
	"aliceryhl@...gle.com" <aliceryhl@...gle.com>,
	"tmgross@...ch.edu" <tmgross@...ch.edu>,
	"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
	"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"nouveau@...ts.freedesktop.org" <nouveau@...ts.freedesktop.org>,
	"rust-for-linux@...r.kernel.org" <rust-for-linux@...r.kernel.org>
Subject: Re: [PATCH v2 2/2] gpu: nova-core: add initial documentation

On Wed, Feb 05, 2025 at 07:44:19PM +0000, Zhi Wang wrote:
> On 05/02/2025 18.10, Danilo Krummrich wrote:
> > On Wed, Feb 05, 2025 at 03:56:46PM +0200, Zhi Wang wrote:
> >> On Tue,  4 Feb 2025 20:03:12 +0100
> >> Danilo Krummrich <dakr@...nel.org> wrote:
> >>> diff --git a/Documentation/gpu/nova/guidelines.rst b/Documentation/gpu/nova/guidelines.rst
> >>> new file mode 100644
> >>> index 000000000000..28a959f51c2c
> >>> --- /dev/null
> >>> +++ b/Documentation/gpu/nova/guidelines.rst
> >>> @@ -0,0 +1,73 @@
> >>> +.. SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> >>> +
> >>
> >> I think this will develop into a maintainer handbook in the future which
> >> reflects maintainer's requirements, thoughts, tips...maybe we can make it
> >> explicit? I think it is rules of book that we agree to follow.
> >>
> >> A similar one can be found here.
> >> https://lore.kernel.org/kvm/20230411171651.1067966-1-seanjc@google.com/
> > 
> > Yes, that's the idea. Is there anything additional you'd like to see added for
> > this initial commit?
> >
> 
> I am mostly thinking of testing of vGPU, we can conclude that more 
> clearly when we have basic stuff working. IMO, maybe some brief 
> introduction about maintainer branches, requirements of patch comments, 
> e.g. prefix of patch titles to different components, expectation of the 
> patch comment body beside those already in submitting_patches.rst. I 
> think those would be helpful.

I agree, but I don't think we need those things right from the get-go. Let's add
those things subsequently.