[dpdk-dev] [PATCH] net/af_packet: fix build failure because of unused parameter
Andrew Rybchenko
arybchenko at solarflare.com
Mon Sep 25 11:04:20 CEST 2017
On 09/25/2017 11:42 AM, Ferruh Yigit wrote:
> On 9/25/2017 7:50 AM, Andrew Rybchenko wrote:
>> Failure happens on build using:
>> gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-4)
> Yes, that case is missed. What do you think about following one:
>
> @@ -561,7 +561,7 @@ rte_pmd_init_internals(struct rte_vdev_device *dev,
> unsigned int blockcnt,
> unsigned int framesize,
> unsigned int framecnt,
> - unsigned int qdisc_bypass,
> + unsigned int qdisc_bypass __rte_unused,
> struct pmd_internals **internals,
> struct rte_eth_dev **eth_dev,
> struct rte_kvargs *kvlist)
It is OK for me as well, but I've not chosen it since it could be treated as
always unused.
>> Fixes: 0d16c17ae7a4 ("net/af_packet: make qdisc bypass configurable")
>>
>> Signed-off-by: Andrew Rybchenko <arybchenko at solarflare.com>
>> ---
>> May be the right solution in fact remove PACKET_QDISC_BYPASS conditional
>> completely. If below solution is accepted, feel free to squash it into
>> the original patch.
> It is a little to late for this, I already sent a pull-request with this
> patch. So fix will need to be a separate patch.
Sad, if it means that the build will be broken in main DPDK repo as well.
Not critical, but a bit inconvenient.
> <...>
More information about the dev
mailing list