Re: [PATCH v3 12/33] gpu: nova-core: gsp: add GMC dispatch on receive

From: Timur Tabi

Date: Tue Sep 22 2026 - 16:12:15 EST


On Mon, 2026-09-21 at 19:50 -0700, John Hubbard wrote:
> How about this:
>
>         return Err(self.poison(fmt!(
>             "GMC element seq# {} has payload size {} in its GMC API header but {} in \
>                 its queue element header",
>             header.gmc.sequence,
>             header.gmc.size,
>             payload_length
>         )));
>
> It names the element, states the fault as one sentence, and puts each number
> next to the header that it came from.
>
> Sample output:
>
>   GSP receive: queue poisoned: GMC element seq# 7 has payload size 96 in its GMC API header but 80 in its queue element header

Sure, that's fine. My only concern was that the original text did not make it clear that the
two numbers are supposed to be the same.