<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<p><br>
</p>
<div class="moz-cite-prefix">On 06-05-2025 23:43, Jason Gunthorpe
wrote:<br>
</div>
<blockquote type="cite" cite="mid:20250506181353.GA89958@nvidia.com">
<pre class="moz-quote-pre" wrap=""><blockquote type="cite" style="color: #007cff;"><pre wrap="" class="moz-quote-pre">+/**
+ * struct fwctl_rpc_xe_pcode - FWCTL Remote Procedure Calls for Xe PCODE
+ */
+struct fwctl_rpc_xe_pcode {
+ /** @command: The main Mailbox command */
+ __u8 command;
+ /** @param1: A subcommand or a parameter of the main command */
+ __u16 param1;
+ /** @param2: A parameter of a subcommand or a subsubcommand */
+ __u16 param2;
+ /** @data0: The first 32 bits of data. In general data-in as param */
+ __u32 data0;
+ /** @data1: The other 32 bits of data. In general data-out */
+ __u32 data1;
+ /** @pad: Padding the uAPI struct - Must be 0. Not sent to firmware */
+ __u8 pad[3];
+};
</pre></blockquote><pre wrap="" class="moz-quote-pre">This has implicit padding? Make the padding explicit or use packed..</pre></pre>
</blockquote>
<p><span data-teams="true">Thanks for pointing this out, was miss at
my end, will go with explicit padding in next version</span></p>
<p><span data-teams="true">Regards,<br>
Badal</span></p>
<br>
</body>
</html>