On 24/07/15 16:27, Pavel Fedin wrote: >>> Ok, let's leave this API alone then for now... >>> Will then be a concensus if i tweak the thing a little bit and we just >>> enable KVM without both > vGIC >>> and vTimer ? It will be an emulator's problem how to handle them then. >> >> Well, let's see the patches first, and how invasive they are. > > Please, i've just posted an RFC and cc'ed you. > I would say the implementation is very small. However, perhaps, it's not > going to work with active > state switching patch, because as far as i understand the whole thing depends > on manipulations with > ARCH_TIMER_CTRL_IT_MASK. Without it i'm going to get multiple interrupts > instead of only one. It > would be possible to signal the interrupt as real level-sensitive (since we > will be tracking active > state), but... it's not possible to know when to deactivate the line, > hardware doesn't report that.
Then it is likely to be a lost cause. We need to get rid of the timer masking crap, as it prevents us from running perfectly valid guests (QNX being one). It also imposes the same hack on other guests (UEFI was forced to implement the same horror). As I said before, not having a VGIC pretty much puts you in an uncharted territory, and the amount of hacks you have to introduce are not really worth the effort in mainline. It is rather sad, but I hope this serves as a lesson for whoever decides to cut corner on HW implementation while thinking "SW will take care of it". They usually shoot themselves in the foot. M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html