<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style> </head> <body dir="ltr"> <div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Hi Timothy,</div> <div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> <br> </div> <div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Thanks for taking care of this, patch applied to 23.11.3 LTS queue.</div> <div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> <br> </div> <div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Regards,</div> <div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Xueming</div> <div id="appendonsend"></div> <hr style="display:inline-block;width:98%" tabindex="-1"> <div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> David Marchand <david.marchand@redhat.com><br> <b>Sent:</b> Thursday, October 24, 2024 6:12 PM<br> <b>To:</b> Timothy Redaelli <tredaelli@redhat.com><br> <b>Cc:</b> stable@dpdk.org <stable@dpdk.org>; Andrew Boyer <andrew.boyer@amd.com>; Luca Boccassi <bluca@debian.org>; Kevin Traynor <ktraynor@redhat.com><br> <b>Subject:</b> Re: [PATCH 23.11] net/ionic: fix build with Fedora Rawhide</font> <div> </div> </div> <div class="BodyFragment"><font size="2"><span style="font-size:11pt;"> <div class="PlainText">On Thu, Oct 24, 2024 at 11:30 AM Timothy Redaelli <tredaelli@redhat.com> wrote:<br> ><br> > Currently, a number of integer types are typedef'd to their corresponding<br> > userspace or RTE values. This can be problematic if these types are<br> > already defined somewhere else, as it would cause type collisions.<br> > This patch changes the typedefs to #define macros which are only defined<br> > if the types are not defined already.<br> ><br> > Fixes: 5ef518098ec6 ("net/ionic: register and initialize adapter")<br> ><br> > Signed-off-by: Timothy Redaelli <tredaelli@redhat.com><br> <br> Copying driver maintainer.<br> The patch lgtm on the principle.<br> <br> <br> > ---<br> > Backport of f0d9e787747d ("net/gve/base: fix build with Fedora Rawhide")<br> > is also needed on stable branches. My commit is not needed on<br> > main since it was fixed silently in 484027bf9452 ("common/ionic: create common<br> > code library").<br> > ---<br> <br> Indeed, it is a pity this got mixed in the mentionned commit.<br> <br> This may be needed in others LTS.<br> Copying Luca and Kevin.<br> <br> <br> -- <br> David Marchand<br> <br> </div> </span></font></div> </body> </html>