* Andi Kleen <[email protected]> wrote:
> > Had i ever noticed this hack in the first place i would have NAK-ed
> > it. There is a fundamental design friction of a high-level feature
> > like HOTPLUG_CPU /requiring/ a fundamental change to the lowlevel
> > IRQ delivery mode!
>
> Well to be honest masked mode isn't that useful anyways. It's only
> theoretical advantage would be a bit more performance for multicast
> IPIs, but Ashok did benchmarks and it didn't make any significant
> difference. [...]
this argument is false for at least two reasons. Firstly, i can show you
a 1000 small changes that wont be measurable individually but that as a
summary effect can degrade the kernel substantially.
Secondly, in the physical case, /all/ IPI sending goes through this
code:
for_each_cpu_mask(query_cpu, mask) {
yes, even the single-IPI calls which give the overwhelming majority of
the use of IPIs. Even on systems that have only 2 CPUs to begin with.
This should be measurable.
> [...] With that I prefer to use always the same mode for small and
> large systems. Ok should probably drop the ifdef and just always use
> physical mode.
you are still not getting it i think. The IO-APIC is still in logical
delivery mode on small systems, and we very much make use of this fact
by using LowestPrio messages (that current CPUs started to support
again). The switching to physical mode is dangerous because it creates
'mixed' APIC messages (physical and logical targeted messages as well) -
which 'mixed mode' is notorious for erratas both in the CPU and in the
chipset. (I strongly suspect that my eth timeouts are due to that.)
Combine this with the fact that /normally/ we default to logical mode,
the basis of your position is quite puzzling to me.
the right solution is to use pure physical mode (both local APIC and
IO-APIC) only on large systems, and to use pure logical mode on small
systems - maybe with the combination of clustered mode as well.
and that's precisely what my patch achieves ...
> > Such a requirement is broken and just serves to hide a flaw in the
> > hotplug design - which flaw would trigger on i386 /anyway/, because
> > i386 still uses logical delivery mode for APIC IPIs.
>
> i386 cpu hotplug is somewhat broken anyways, but it should be fixed
> there too eventually. But some very old chipsets don't seem to support
> physical properly so it wasn't changed there.
as i said it before, what you are suggesting is not a 'fix', it's a
workaround for a design flaw in the hotplug code which flaw is hitting
us in other places and architectures anyway, and which workaround makes
us use an inferior IRQ delivery method on small systems ...
Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
