Re: [RFC][Patch] fix for irq_affinity_write_proc v2.5

Martin J. Bligh (mbligh@aracnet.com)
Tue, 06 May 2003 20:56:54 -0700


>> irq_affinity_write_proc currently directly calls set_ioapic_affinity
>> which writes to the ioapic. This undermines the work done by kirqd by
>> writing a cpu mask directly to the ioapic. I propose the following patch
>> to tie the /proc affinity writes into the same code path as kirqd.
>> Kirqd will enforce the affinity requested by the user.
>
> Why should the kernel be enforcing policy here. You have to be root to
> do this, and root should have the ability to configure apparently stupid
> things because they may find them useful.

Whilst in general, I'd agree, in this case it makes no sense - there are
two masks set: the irqbalance mask, and the apicid mask. If irqbalance
is on, setting the apicid mask doesn't really do anything, because
irqbalance is going to reset it dynamically very shortly afterwards
anyway. It's hard to imagine how that might be useful ;-)

It might conceivably be useful to disable irqbalance dynamically, if that's
what you mean, but that shouldn't be the default behaviour, I think.
Keith already a nice simple patch to make it a config option ... doing it
dynamically is a different project, IMHO, and one of questionable utility
to anyone not capable of coding it themselves ;-)

M.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/