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.