<div dir="ltr"><div dir="auto"><div dir="auto"><br></div><div data-smartmail="gmail_signature" dir="auto"><div dir="ltr"><span style="color:rgb(0,0,0);font-family:Tinos;font-size:medium"> </span><br></div></div><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Wed, Sep 23, 2026, 00:35 Bruce Richardson <<a href="mailto:bruce.richardson@intel.com" target="_blank">bruce.richardson@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Sep 23, 2026 at 08:30:18AM +0100, Bruce Richardson wrote:<br>
> On Tue, Sep 22, 2026 at 06:07:25PM -0700, Joshua Washington wrote:<br>
> > The RTE_DIV_ROUND_UP in rte_common.h makes DIV_ROUND_UP redundant.<br>
> > <br>
> > Signed-off-by: Joshua Washington <<a href="mailto:joshwash@google.com" rel="noreferrer" target="_blank">joshwash@google.com</a>><br>
> > Reviewed-by: Stephen Hemminger <<a href="mailto:stephen@networkplumber.org" rel="noreferrer" target="_blank">stephen@networkplumber.org</a>><br>
> Acked-by: Bruce Richardson <<a href="mailto:bruce.richardson@intel.com" rel="noreferrer" target="_blank">bruce.richardson@intel.com</a>><br>
> > ---<br>
> >  drivers/net/intel/i40e/base/i40e_osdep.h | 3 +--<br>
> >  1 file changed, 1 insertion(+), 2 deletions(-)<br>
> > <br>
<br>
FYI: Just a note for i40e and the other intel drivers: the osdep.h files<br>
have moved in the repo in next-net-intel tree, so this patch will conflict<br>
once those changes are merged to main. Therefore, I'd suggest not merging<br>
this till after the pull, and then adjusting the paths in the patch on<br>
apply for simplicity of merge.<br>
<br>
Thanks,<br>
/Bruce<br>
<br>
> > diff --git a/drivers/net/intel/i40e/base/i40e_osdep.h b/drivers/net/intel/i40e/base/i40e_osdep.h<br>
> > index 9c5a94f055..93620f5c50 100644<br>
> > --- a/drivers/net/intel/i40e/base/i40e_osdep.h<br>
> > +++ b/drivers/net/intel/i40e/base/i40e_osdep.h<br>
> > @@ -227,12 +227,11 @@ struct i40e_spinlock {<br>
> >  #define i40e_memset(a, b, c, d) memset((a), (b), (c))<br>
> >  #define i40e_memcpy(a, b, c, d) memcpy((a), (b), (c))<br>
> >  <br>
> > -#define DIV_ROUND_UP(n,d) (((n) + (d) - 1) / (d))<br>
> >  #define DELAY(x) rte_delay_us_sleep(x)<br>
> >  #define i40e_usec_delay(x) DELAY(x)<br>
> >  #define i40e_msec_delay(x) DELAY(1000 * (x))<br>
> >  #define udelay(x) DELAY(x)<br>
> >  #define msleep(x) DELAY(1000*(x))<br>
> > -#define usleep_range(min, max) msleep(DIV_ROUND_UP(min, 1000))<br>
> > +#define usleep_range(min, max) msleep(RTE_DIV_ROUND_UP(min, 1000))<br>
> >  <br>
> >  #endif /* _I40E_OSDEP_H_ */<br>
> > -- <br>
> > 2.55.0.1082.g2b9226bbc0-goog<br>
> ></blockquote></div><div dir="auto"><br></div><div dir="auto">Ah, I missed this. Thanks for the update. I will hold off on a v3 if it is necessary until the main branch is updated.</div></div>
</div>