From: Haiyue Wang Date: Sun, 14 Oct 2018 02:34:05 +0000 (+0800) Subject: net/avf: fix build with debug enabled X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=a710ceb1728497774408225d9327bbf512e9ba16;p=dpdk.git net/avf: fix build with debug enabled DEBUG_DUMP_DESC flag is enabled/disabled from Makefile and enabling it causing the build issue. Add the missed 'volatile' keyword to avoid the warning for type mismatch, which will be treated as compiler error if WERROR_FLAGS is enabled. Fixes: bfd38e4d708b ("net/avf: fix missing compiler error flags") CC: stable@dpdk.org Signed-off-by: Haiyue Wang Reviewed-by: Ferruh Yigit --- diff --git a/drivers/net/avf/avf_rxtx.h b/drivers/net/avf/avf_rxtx.h index c4120f8a4e..898d2f387f 100644 --- a/drivers/net/avf/avf_rxtx.h +++ b/drivers/net/avf/avf_rxtx.h @@ -201,17 +201,17 @@ int avf_txq_vec_setup(struct avf_tx_queue *txq); static inline void avf_dump_rx_descriptor(struct avf_rx_queue *rxq, - const void *desc, + const volatile void *desc, uint16_t rx_id) { #ifdef RTE_LIBRTE_AVF_16BYTE_RX_DESC - const union avf_16byte_rx_desc *rx_desc = desc; + const volatile union avf_16byte_rx_desc *rx_desc = desc; printf("Queue %d Rx_desc %d: QW0: 0x%016"PRIx64" QW1: 0x%016"PRIx64"\n", rxq->queue_id, rx_id, rx_desc->read.pkt_addr, rx_desc->read.hdr_addr); #else - const union avf_32byte_rx_desc *rx_desc = desc; + const volatile union avf_32byte_rx_desc *rx_desc = desc; printf("Queue %d Rx_desc %d: QW0: 0x%016"PRIx64" QW1: 0x%016"PRIx64 " QW2: 0x%016"PRIx64" QW3: 0x%016"PRIx64"\n", rxq->queue_id, @@ -225,10 +225,10 @@ void avf_dump_rx_descriptor(struct avf_rx_queue *rxq, */ static inline void avf_dump_tx_descriptor(const struct avf_tx_queue *txq, - const void *desc, uint16_t tx_id) + const volatile void *desc, uint16_t tx_id) { const char *name; - const struct avf_tx_desc *tx_desc = desc; + const volatile struct avf_tx_desc *tx_desc = desc; enum avf_tx_desc_dtype_value type; type = (enum avf_tx_desc_dtype_value)rte_le_to_cpu_64(