[PATCH v2 12/12] app: remove unnecessary include of atomic header file
Joyce Kong
Joyce.Kong at arm.com
Wed Nov 17 08:05:37 CET 2021
<snip>
> Subject: Re: [PATCH v2 12/12] app: remove unnecessary include of atomic
> header file
>
> On Tue, Nov 16, 2021 at 10:44 AM Joyce Kong <joyce.kong at arm.com> wrote:
> >
> > Remove the unnecessary rte_atomic.h included in app modules.
> >
> > Signed-off-by: Joyce Kong <joyce.kong at arm.com>
> > Reviewed-by: Ruifeng Wang <ruifeng.wang at arm.com>
>
> After patch, I still see:
>
> $ git grep rte_atomic.h app/
> app/test/commands.c:#include <rte_atomic.h>
> app/test/test_atomic.c:#include <rte_atomic.h>
> app/test/test_event_timer_adapter.c:#include <rte_atomic.h>
>
> I can undertand why the test_atomic would depend on rte_atomic.h :-) but
> not the rest.
> Is there a reason? or is it just a miss?
>
> --
> David Marchand
Hi David, I checked the rest and it was a miss. Thanks for the remind, would update in v3.
Joyce
More information about the dev
mailing list