Thread (16 messages) flat view 16 messages, 5 authors, 5d ago

Re: [PATCH 1/2] gpu: nova-core: declare unlimited DMA max segment size

From: Matteo Kloiber <hidden>
Date: 2026-09-13 21:13:35
Also in: dri-devel, driver-core, lkml, nova-gpu

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!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help