Sven Auhagen <sven.auha...@voleatech.de> wrote:
>
> +       // Set affinity
> +       cpu = ring_id % num_online_cpus();
> +       irq_set_affinity_hint(irq, get_cpu_mask(cpu));
> +

This doesn't look right.  There is no guarantee that the online
CPUs are the lowest bits in the bitmask.  Also, what are you going
to do when the CPUs go down (or up)?

Cheers,
-- 
Email: Herbert Xu <herb...@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Reply via email to