Hello,

On Thu, Nov 24, 2022 at 1:31 PM Varghese, Vipin <vipin.vargh...@amd.com> wrote:
>
> Hi John and Thomas,
>
> Sharing an observation for compilation failure

Thanks for reporting.
I suspect you forgot to mention that this error is only seen with some
additional -DRTE_BBDEV_OFFLOAD_COST in cflags.

>
> Git head: 9cb8326456317264a7d085978634661c520af706
> Application – test-bbdev
> Error: ```
>
> ../app/test-bbdev/test_bbdev_perf.c: In function ‘get_bbdev_queue_stats’:
> ../app/test-bbdev/test_bbdev_perf.c:4936:16: error: ‘struct rte_bbdev_stats’ 
> has no member named ‘enqueue_warning_count’; did you mean 
> ‘enqueue_warn_count’?
> 4936 |         stats->enqueue_warning_count = q_stats->enqueue_warning_count;
>       |                ^~~~~~~~~~~~~~~~~~~~~
>       |                enqueue_warn_count
> ../app/test-bbdev/test_bbdev_perf.c:4936:49: error: ‘struct rte_bbdev_stats’ 
> has no member named ‘enqueue_warning_count’; did you mean 
> ‘enqueue_warn_count’?
> 4936 |         stats->enqueue_warning_count = q_stats->enqueue_warning_count;
>       |                                                 ^~~~~~~~~~~~~~~~~~~~~
>       |                                                 enqueue_warn_count
> ../app/test-bbdev/test_bbdev_perf.c:4937:16: error: ‘struct rte_bbdev_stats’ 
> has no member named ‘dequeue_warning_count’; did you mean 
> ‘dequeue_warn_count’?
> 4937 |         stats->dequeue_warning_count = q_stats->dequeue_warning_count;
>       |                ^~~~~~~~~~~~~~~~~~~~~
>       |                dequeue_warn_count
> ../app/test-bbdev/test_bbdev_perf.c:4937:49: error: ‘struct rte_bbdev_stats’ 
> has no member named ‘dequeue_warning_count’; did you mean 
> ‘dequeue_warn_count’?
> 4937 |         stats->dequeue_warning_count = q_stats->dequeue_warning_count;
>       |                                                 ^~~~~~~~~~~~~~~~~~~~~
>       |                                                 dequeue_warn_count
> ````
>
> Reason for error: rte_bbdev_stats  has dequeue_warn_count
> Fix: convert stats to warn rather than warning

The fix should be trivial, can you send it quickly?
Thanks.


-- 
David Marchand

Reply via email to