> -----Original Message-----
> From: Intel-wired-lan <[email protected]> On Behalf Of
> Larysa Zaremba
> Sent: Friday, May 16, 2025 7:58 AM
> To: [email protected]; Nguyen, Anthony L
> <[email protected]>
> Cc: David S. Miller <[email protected]>; Eric Dumazet
> <[email protected]>; Jakub Kicinski <[email protected]>; Paolo Abeni
> <[email protected]>; Simon Horman <[email protected]>; Jonathan
> Corbet <[email protected]>; Kitszel, Przemyslaw
> <[email protected]>; Jiri Pirko <[email protected]>; Nikolova,
> Tatyana
> E <[email protected]>; Andrew Lunn <[email protected]>;
> Lobakin, Aleksander <[email protected]>; Michael Ellerman
> <[email protected]>; Fijalkowski, Maciej <[email protected]>;
> Lee Trager <[email protected]>; Madhavan Srinivasan <[email protected]>;
> Zaremba, Larysa <[email protected]>; Samudrala, Sridhar
> <[email protected]>; Keller, Jacob E <[email protected]>;
> Michal Swiatkowski <[email protected]>; Polchlopek,
> Mateusz <[email protected]>; Zaki, Ahmed
> <[email protected]>; [email protected]; linux-
> [email protected]; [email protected]; Karlsson, Magnus
> <[email protected]>; Tantilov, Emil S <[email protected]>;
> Chittim, Madhu <[email protected]>; Hay, Joshua A
> <[email protected]>; Olech, Milena <[email protected]>; Linga,
> Pavan Kumar <[email protected]>; Singhai, Anjali
> <[email protected]>; Kubiak, Michal <[email protected]>
> Subject: [Intel-wired-lan] [PATCH iwl-next v4 11/15] idpf: print a debug
> message and bail in case of non-event ctlq message
>
> Unlike previous internal idpf ctlq implementation, idpf calls the default
> message handler for all received messages that do not have a matching xn
> transaction, not only for VIRTCHNL2_OP_EVENT. This leads to many error
> messages printing garbage, because the parsing expected a valid event
> message, but got e.g. a delayed response for a timed-out transaction.
>
> The information about timed-out transactions and otherwise unhandleable
> messages can still be valuable for developers, so print the information with
> dynamic debug and exit the function, so the following functions can parse
> valid events in peace.
>
> Reviewed-by: Michal Kubiak <[email protected]>
> Signed-off-by: Larysa Zaremba <[email protected]>
> ---
> 2.47.0
Tested-by: Samuel Salin <[email protected]>
