On Wed, Oct 14, 2026 at 12:24:18PM +0000, Dawid Wesierski wrote:
> The VF allocates qos_cap and initializes the TM lists only when the PF
> grants VIRTCHNL_VF_OFFLOAD_QOS. The in-tree ice PF does not grant it.
> On such a VF, node add, shaper profile add, capabilities get, and node
> capabilities get dereference a NULL qos_cap, or insert into a list that
> has no initialization. The application gets a segmentation fault from a
> call such as rte_tm_capabilities_get().
>
> Return -ENOTSUP from these operations when qos_cap is NULL. The other
> TM operations only search the lists, or check VIRTCHNL_VF_OFFLOAD_QOS
> already, so they do not need the check.
>
> Fixes: 44d0a720a538 ("net/iavf: query QoS capabilities and set queue TC
> mapping")
> Cc: [email protected]
>
> Signed-off-by: Dawid Wesierski <[email protected]>
> ---
Acked-by: Bruce Richardson <[email protected]>
- [PATCH 0/3] net/iavf: fix TM hierarchy commit and TM ops ... Dawid Wesierski
- [PATCH 2/3] net/iavf: fix bandwidth config of unshap... Dawid Wesierski
- Re: [PATCH 2/3] net/iavf: fix bandwidth config o... Bruce Richardson
- Re: [PATCH 2/3] net/iavf: fix bandwidth conf... Dawid Wesierski
- [PATCH 3/3] net/iavf: fix crash in TM ops without Qo... Dawid Wesierski
- Re: [PATCH 3/3] net/iavf: fix crash in TM ops wi... Bruce Richardson
- [PATCH 1/3] net/iavf: fix invalid free of queue TC m... Dawid Wesierski
- Re: [PATCH 1/3] net/iavf: fix invalid free of qu... Bruce Richardson
- Re: [PATCH 1/3] net/iavf: fix invalid free o... Bruce Richardson
- Re: [PATCH 1/3] net/iavf: fix invalid fr... Dawid Wesierski
- [PATCH v2 0/3] net/iavf: fix TM hierarchy commit and... Dawid Wesierski
- [PATCH v2 1/3] net/iavf: fix invalid free of que... Dawid Wesierski
- [PATCH v2 3/3] net/iavf: fix crash in TM ops wit... Dawid Wesierski
- [PATCH v2 2/3] net/iavf: fix bandwidth config of... Dawid Wesierski
- Re: [PATCH v2 0/3] net/iavf: fix TM hierarchy co... Bruce Richardson

