[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH] xen: CONFIG_PV_SHIM_EXCLUSIVE and CONFIG_HVM are mutually exclusive
On 08.12.2020 15:33, Andrew Cooper wrote: > On 08/12/2020 13:51, Juergen Gross wrote: >> With CONFIG_PV_SHIM_EXCLUSIVE some sources required for CONFIG_HVM are >> not built, so let CONFIG_HVM depend on !CONFIG_PV_SHIM_EXCLUSIVE. >> >> Let CONFIG_HVM default to !CONFIG_PV_SHIM instead. >> >> Signed-off-by: Juergen Gross <jgross@xxxxxxxx> > > So while this will fix the randconfig failure, the statement isn't > true. There are HVM codepaths which aren't even dead in shim-exclusive > mode. > > The problem here is the way CONFIG_PV_SHIM_EXCLUSIVE abuses the Kconfig > system. What is currently happening is that this option is trying to > enforce the pv shim defconfig in the dependency system. > > We already have a defconfig, which is used in appropriate locations. We > should not have two different things fighting over control. > > This is the fault of c/s 8b5b49ceb3d which went in despite my > objections. The change is not related to PV_SHIM_EXCLUSIVE - it is to > do with not supporting a control domain, which a) better describes what > it is actually doing, and b) has wider utility than PV Shim. Would you mind pointing me at where you had voiced objections to that change? I've just searched both my inbox and the list archives, without finding any. I only recall your objections to the patch I sent later which is similar to Jürgen's. And I'm quite certain I'd have stayed away from committing anything while aware of unresolved objections, even if - more often than not - this means waiting almost indefinitely, which I don't appreciate as a way to deal with disagreement. >From what you further state, I derive that you'd like to see e.g. !PV_SHIM_EXCLUSIVE be a dependency of a new CONTROL_DOMAIN Kconfig setting? I'm not sure though I see how this would help the situation (I'm not even sure what scope this control would have: just domctl, or also sysctl, or additionally platform-op). Nor do I see what's wrong with forcing HVM off in shim-exclusive builds - there can't be HVM domains in such a configuration (but I'm pretty sure I said so somewhere else already, without ever hearing back, albeit it apparently wasn't on either of the patches' threads according to my outbox). Jan
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |