On 2/28/25 22:15, Eric Woudstra wrote:
> In case of using mediatek wireless, in nft_dev_fill_forward_path(), the
> forward path is filled, ending with mediatek wlan1.
> 
> Because DEV_PATH_MTK_WDMA is unknown inside nft_dev_path_info() it returns
> with info.indev = NULL. Then nft_dev_forward_path() returns without
> setting the direct transmit parameters.
> 
> This results in a neighbor transmit, and direct transmit not possible.
> But we want to use it for flow between bridged interfaces.
> 
> So this patch adds DEV_PATH_MTK_WDMA to nft_dev_path_info() and makes
> direct transmission possible.
> 
> Signed-off-by: Eric Woudstra <[email protected]>
> ---
>  net/netfilter/nft_flow_offload.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/net/netfilter/nft_flow_offload.c 
> b/net/netfilter/nft_flow_offload.c
> index 323c531c7046..b9e6d9e6df66 100644
> --- a/net/netfilter/nft_flow_offload.c
> +++ b/net/netfilter/nft_flow_offload.c
> @@ -105,6 +105,7 @@ static void nft_dev_path_info(const struct 
> net_device_path_stack *stack,
>               switch (path->type) {
>               case DEV_PATH_ETHERNET:
>               case DEV_PATH_DSA:
> +             case DEV_PATH_MTK_WDMA:
>               case DEV_PATH_VLAN:
>               case DEV_PATH_PPPOE:
>                       info->indev = path->dev;
> @@ -117,6 +118,10 @@ static void nft_dev_path_info(const struct 
> net_device_path_stack *stack,
>                               i = stack->num_paths;
>                               break;
>                       }
> +                     if (path->type == DEV_PATH_MTK_WDMA) {
> +                             i = stack->num_paths;
> +                             break;
> +                     }
>  
>                       /* DEV_PATH_VLAN and DEV_PATH_PPPOE */
>                       if (info->num_encaps >= NF_FLOW_TABLE_ENCAP_MAX) {

Reviewed-by: Nikolay Aleksandrov <[email protected]>


Reply via email to