mbox series

[V2,0/4] ethdev: change the queue ID type

Message ID 1599219135-53194-1-git-send-email-humin29@huawei.com (mailing list archive)
Headers
Series ethdev: change the queue ID type |

Message

humin (Q) Sept. 4, 2020, 11:32 a.m. UTC 
  This series are minor change the queue ID type from
uint8_t to uint16_t.

Huisong Li (2):
  ethdev: fix compiling errors for per-queue statistics
  ethdev: change data type in TC rxq and TC txq

Min Hu (Connor) (2):
  doc: announce modified field in DCB TC queue mapping
  doc: announce modified field in ethdev API

 app/proc-info/main.c                   |  2 +-
 app/test-pmd/cmdline.c                 |  2 +-
 app/test-pmd/config.c                  |  4 ++--
 app/test-pmd/testpmd.c                 |  2 +-
 app/test-pmd/testpmd.h                 |  5 +++--
 doc/guides/rel_notes/release_20_11.rst |  7 +++++++
 drivers/net/igc/igc_ethdev.c           |  4 ++--
 drivers/net/ixgbe/ixgbe_ethdev.c       |  4 ++--
 drivers/net/memif/rte_eth_memif.c      |  2 +-
 drivers/net/octeontx2/otx2_ethdev.h    |  2 +-
 drivers/net/octeontx2/otx2_stats.c     |  2 +-
 drivers/net/virtio/virtio_ethdev.c     |  4 ++--
 lib/librte_ethdev/rte_ethdev.c         |  6 +++---
 lib/librte_ethdev/rte_ethdev.h         | 12 ++++++------
 lib/librte_ethdev/rte_ethdev_core.h    |  2 +-
 15 files changed, 34 insertions(+), 26 deletions(-)

Comments

Ferruh Yigit Sept. 4, 2020, 2:33 p.m. UTC | #1 
On 9/4/2020 12:32 PM, Min Hu (Connor) wrote:
> This series are minor change the queue ID type from
> uint8_t to uint16_t.

Hi Min Hu,

For the next time, a change log to summarize what has been updated in new
version can help to the reviewers, please add it either to commit log or cover
letter.

> 
> Huisong Li (2):
>   ethdev: fix compiling errors for per-queue statistics
>   ethdev: change data type in TC rxq and TC txq
> 
> Min Hu (Connor) (2):
>   doc: announce modified field in DCB TC queue mapping
>   doc: announce modified field in ethdev API

As said previously, you can drop the deprecation notice patches from the series.
If everything else is good I can drop while merging.

Thanks,
ferruh