HI Matan,
> > On Wed, 24 Jul 2019 15:37:40 +0000
> > Matan Azrad <ma...@mellanox.com> wrote:
> >
> > > Hi Hemant, Sachin
> > >
> > > Looks like issue in dpaa.
> > > Please check.
> > >
> > > > -----Original Message-----
> > > > From: dev <dev-boun...@dpdk.org> On Behalf Of bugzi...@dpdk.org
> > > > Sent: Wednesday, July 24, 2019 6:35 PM
> > > > To: dev@dpdk.org
> > > > Subject: [dpdk-dev] [Bug 335] Compilation fails when HEADROOM is 0
> > > >
> > > > https://bugs.
> > > >
> >
> dpdk.org%2Fshow_bug.cgi%3Fid%3D335&amp;data=02%7C01%7Cmatan%40
> > > >
> >
> mellanox.com%7Cef4c519dd7b1495178f808d7104c7400%7Ca652971c7d2e4d
> 9
> > > >
> >
> ba6a4d149256f461b%7C0%7C0%7C636995792862104273&amp;sdata=DrdcC
> J
> > > >
> >
> mYUMZxk3j%2F%2BxPlqXMOAb9GDNBPzSNzpifzcMU%3D&amp;reserved=0
> > > >
> > > >             Bug ID: 335
> > > >            Summary: Compilation fails when HEADROOM is 0
> > > >            Product: DPDK
> > > >            Version: unspecified
> > > >           Hardware: All
> > > >                 OS: All
> > > >             Status: UNCONFIRMED
> > > >           Severity: normal
> > > >           Priority: Normal
> > > >          Component: ethdev
> > > >           Assignee: dev@dpdk.org
> > > >           Reporter: ma...@mellanox.com
> > > >   Target Milestone: ---
> > > >
> > > > The compilation error:
> > > > In file included from
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/po
> > > > rtal/dpaa2_hw_dpbp.c:28:0:
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/po
> > > > rtal/dpaa2_hw_pvt.h:76:2:
> > > > error: #error "Annotation requirement is more than
> > > > RTE_PKTMBUF_HEADROOM"
> > > >  #error "Annotation requirement is more than
> > RTE_PKTMBUF_HEADROOM"
> > > >   ^
> > > > cc1: error: unrecognized command line option
> > > > "-Wno-address-of-packed- member"
> > > > [-Werror]
> > > > cc1: all warnings being treated as errors
> > > > make[6]: *** [portal/dpaa2_hw_dpbp.o] Error 1
> > > > make[6]: *** Waiting for unfinished jobs....
> > > > In file included from
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/po
> > > > rtal/dpaa2_hw_dpio.c:37:0:
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/po
> > > > rtal/dpaa2_hw_pvt.h:76:2:
> > > > error: #error "Annotation requirement is more than
> > > > RTE_PKTMBUF_HEADROOM"
> > > >  #error "Annotation requirement is more than
> > RTE_PKTMBUF_HEADROOM"
> > > >   ^
> > > > cc1: error: unrecognized command line option
> > > > "-Wno-address-of-packed- member"
> > > > [-Werror]
> > > > cc1: all warnings being treated as errors
> > > > make[6]: *** [portal/dpaa2_hw_dpio.o] Error 1 In file included
> > > > from
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/po
> > > > rtal/dpaa2_hw_dpci.c:26:0:
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/po
> > > > rtal/dpaa2_hw_pvt.h:76:2:
> > > > error: #error "Annotation requirement is more than
> > > > RTE_PKTMBUF_HEADROOM"
> > > >  #error "Annotation requirement is more than
> > RTE_PKTMBUF_HEADROOM"
> > > >   ^
> > > > cc1: error: unrecognized command line option
> > > > "-Wno-address-of-packed- member"
> > > > [-Werror]
> > > > cc1: all warnings being treated as errors
> > > > make[6]: *** [portal/dpaa2_hw_dpci.o] Error 1 In file included
> > > > from
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/fsl
> > > > mc_vfio.c:40:0:
> > > >
> >
> /auto/mtrswgwork/matan/reps/matan_rep/dpdk.org/drivers/bus/fslmc/po
> > > > rtal/dpaa2_hw_pvt.h:76:2:
> > > > error: #error "Annotation requirement is more than
> > > > RTE_PKTMBUF_HEADROOM"
> > > >  #error "Annotation requirement is more than
> > RTE_PKTMBUF_HEADROOM"
> > > >   ^
> > > >
> > > > --
> > > > You are receiving this mail because:
> > > > You are the assignee for the bug.
> >
> > It seems reasonable for a driver to require headroom to store extra
> > data in receive buffer. It should be documented though.
> 
> Yes.
> 
> But if so, this driver should not be compiled by default, or change to raise
> run-time error instead.
> 
> What do you think?
[Hemant] We will change it to run-time error.

Reply via email to