Re: [PATCH 1/2] gpu: nova-core: declare unlimited DMA max segment size
From: Matteo Kloiber
Date: Sun Sep 13 2026 - 17:17:50 EST
On Mon Sep 7, 2026 at 11:10 AM JST, Alexandre Courbot wrote:
> Note: new kernel policy [1] asks that the specific model is not named, so
> this should just be `Assisted-by: LLM`.
Got it, thanks! I was checking what other people did and noticed a lot of
mentions of Claude, etc., so I went with it. Changed for v2.
> Thanks, this patch looks correct and I would like to merge it early, but
> one thing about the comment: it carries way more context than needed and
> reads heavily, as is often the case when AI-generated. For instance,
> "declare that to the DMA layer" is obvious from the method we are
> calling. Make sure to give a human pass to such comments as they can
> make the code tedious when they accumulate.
Thanks! I agree. I was unsure about that and decided to leave a bit more
in for the review. I'll just use your comment verbatim.
> Since this seems to be your first patch, this is a good opportunity to
> practice sending a v2. :)
Yes, submitting patches to the mailing list was certainly a new experience.
Thank you for your help and time so far!