Re: [PATCH v5] enable x2APIC without interrupt remapping under KVM

[Eric W. Biederman]

Gleb Natapov <gleb@xxxxxxxxxx> writes:

>> Therefore I don't see the point of supporting one without the other.
> x2apic provide us with other benefits as commit message explains, and
> doesn't add any problems that we don't have now already.

If this code has a legitimate place on real hardware I am all for it.

If this is just a hack to make virtualization faster I don't like the
extra code paths in the middle core architecture code.  That will
be a support burden for the foreseeable future.  More code to
test etc. 

Quickly skimming the patch it just appears to stir a mess.
Plus it adds weird paravirtualization checks, ???

If we are going to have a special code path for virtual hardware
can we do it right and have something nice to use that makes life
simpler?  For what we want to do with ioapics they suck and are
really not suitable.  The only thing that recommends them is that
they are standard.  But you are deviating from the standard so
what is the point.

Eric

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