Re: [PATCH net-next V6 2/2] net/mlx4_en: Use affinity hint

From: David Miller
Date: Mon Jun 02 2014 - 00:56:58 EST


From: Eric Dumazet <eric.dumazet@xxxxxxxxx>
Date: Sun, 01 Jun 2014 21:37:11 -0700

> On Sun, 2014-06-01 at 21:16 -0700, Eric Dumazet wrote:
>> CC [M] drivers/net/ethernet/mellanox/mlx4/en_netdev.o
>> drivers/net/ethernet/mellanox/mlx4/en_netdev.c: In function ʽmlx4_en_init_affinity_hintʼ:
>> drivers/net/ethernet/mellanox/mlx4/en_netdev.c:1546:23: error: incompatible types when assigning to type ʽcpumask_var_tʼ from type ʽvoid *ʼ
>> drivers/net/ethernet/mellanox/mlx4/en_netdev.c: In function ʽmlx4_en_free_affinity_hintʼ:
>> drivers/net/ethernet/mellanox/mlx4/en_netdev.c:1553:41: error: incompatible types when assigning to type ʽcpumask_var_tʼ from type ʽvoid *ʼ
>
>
> And :
>
> ERROR: "cpumask_set_cpu_local_first" [drivers/net/ethernet/mellanox/mlx4/mlx4_en.ko] undefined!
>
>
> $ git grep -n cpumask_set_cpu_local_first
> drivers/net/ethernet/mellanox/mlx4/en_netdev.c:1542: if (cpumask_set_cpu_local_first(ring_idx, numa_node,
> include/linux/cpumask.h:260:int cpumask_set_cpu_local_first(int i, int numa_node, cpumask_t *dstp);
> lib/cpumask.c:168: * cpumask_set_cpu_local_first - set i'th cpu with local numa cpu's first
> lib/cpumask.c:182:int cpumask_set_cpu_local_first(int i, int numa_node, cpumask_t *dstp)
> lib/cpumask.c:228:EXPORT_SYMBOL(cpumask_set_cpu_local_first);
>
> Fixes are needed if CONFIG_CPUMASK_OFFSTACK is not used.
>
> $ grep CONFIG_CPUMASK_OFFSTACK .config
> $ echo $?
> 1
>

Indeed you have to provide a dummy version for a non-SMP build etc.

I'm reverting.

N?§²ζμrΈ?yϊθ?Ψb²X¬ΆΗ§vΨ^?)ήΊ{.nΗ+?·??{±?κηzX§Ά?ʽά¨}©?²Ζ zΪ&j:+v?¨Ύ«?κηzZ+?Κ+zf£ʼ·h??§~?­?Ϋi?ϋΰzΉ?w?ʼΈ??¨θ­Ϊ&ʼ)ίʼf?ω^jΗ«y§m?α@A«aΆΪ? 0Άμh?ε?i