> Yes. It would be great, if Youquan can point out where is the intel doc > about the change. > > Also if the patch can move on, hypervisor_x2apic_available() related > declaration and define > could be dropped.
Hi Yinghai, Sorry I do not know the document change but I also do not find the words/description/explanation that x2APIC physical mode also need interrupt remapping support when CPU < 256. Of course, X2APIC cluster mode must has interrupt remapping support. I have tested many machines, both old and most recent machines and from desktop to server, x2APIC physical mode works without interrupt remapping when CPU < 256. In theory and real test, I do not find any issue about the patch. In order to make sure the patch without involving unexpected issues beyond I can understand, I will confirm with our expert about it. so please pend the patch going to mainline. If the patch can move on, I think I will also provide other patch changing, like direct EOI. Thanks -Youquan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/