On Wed, 19 Feb 2025 15:55:17 +0100
Thomas Monjalon <tho...@monjalon.net> wrote:

> 14/02/2025 18:20, Stephen Hemminger:
> > The Linux version of sys/queue.h is frozen at an older version
> > and is missing the _SAFE macro variants. Several drivers started
> > introducing the own workarounds for this. Should be handled in EAL.
> > 
> > Signed-off-by: Stephen Hemminger <step...@networkplumber.org>  
> 
> We may want to unify with lib/eal/windows/include/sys/queue.h
> 
> 

Not sure, we have several options here:
  1. Keep using Linux sys/queue.h and add enhancements in rte_queue.h
  2. Make rte_queue.h a copy of FreeBSD version of queue.h (that is what 
Windows did)
  3. Use the bsd version of queue.h.
     On Debian/Ubuntu this in libbsd-dev package and referred to as 
bsd/sys/queue.h

I chose #1 as simplest. But all of these could work. #3 means DPDK has least new
code, but adds another dependency to the build.

No matter what, should add to checkpatch to block any new files that include 
sys/queue.h
directly.

Reply via email to