<!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>