I believe Jeff @ Intel is going to submit a patch soon.
On 2/18/15, 11:18 AM, "Thomas Monjalon" <thomas.monjalon at 6wind.com> wrote: >Hi Keith, > >2015-02-18 17:01, Wiles, Keith: >> I just pulled the code and found a unused function error ?dump_rxd()? >> >> I had to add the ifdef around the function and remove the ifdef inside >>the function: >> >> #ifdef RTE_LIBRTE_FM10K_DEBUG_RX >> static inline void dump_rxd(union fm10k_rx_desc *rxd) >> { >> RTE_SET_USED(rxd); >> PMD_RX_LOG(DEBUG, "+----------------|----------------+"); >> PMD_RX_LOG(DEBUG, "| GLORT | PKT HDR & TYPE |"); >> PMD_RX_LOG(DEBUG, "| 0x%08x | 0x%08x |", rxd->d.glort, >> rxd->d.data); >> PMD_RX_LOG(DEBUG, "+----------------|----------------+"); >> PMD_RX_LOG(DEBUG, "| VLAN & LEN | STATUS |"); >> PMD_RX_LOG(DEBUG, "| 0x%08x | 0x%08x |", rxd->d.vlan_len, >> rxd->d.staterr); >> PMD_RX_LOG(DEBUG, "+----------------|----------------+"); >> PMD_RX_LOG(DEBUG, "| RESERVED | RSS_HASH |"); >> PMD_RX_LOG(DEBUG, "| 0x%08x | 0x%08x |", 0, rxd->d.rss); >> PMD_RX_LOG(DEBUG, "+----------------|----------------+"); >> PMD_RX_LOG(DEBUG, "| TIME TAG |"); >> PMD_RX_LOG(DEBUG, "| 0x%016lx |", rxd->q.timestamp); >> PMD_RX_LOG(DEBUG, "+----------------|----------------+"); >> } >> #endif >> >> Also clang on Ubuntu 14.04 does not like the option: >> CC fm10k_pf.o >> error: unknown warning option '-Wno-unused-but-set-variable'; did you >>mean '-Wno-unused-const-variable'? [-Werror,-Wunknown-warning-option] > >Thanks for reporting. >I have no error with GCC 4.9.2. >Please could you try to send a patch?