On 10/01/2020 10:37, Sergey Dyasli wrote: > Hide the following information that can help identify the running Xen > binary version: XENVER_extraversion, XENVER_compile_info, XENVER_changeset. > Add explicit cases for XENVER_commandline and XENVER_build_id as well. > > Introduce xsm_filter_denied() to hvmloader to remove "<denied>" string > from guest's DMI tables that otherwise would be shown in tools like > dmidecode. > > Signed-off-by: Sergey Dyasli <sergey.dya...@citrix.com> > --- > v1 --> v2: > - Added xsm_filter_denied() to hvmloader instead of modifying xen_deny() > - Made behaviour the same for both Release and Debug builds > - XENVER_capabilities is no longer hided > > CC: Andrew Cooper <andrew.coop...@citrix.com> > CC: George Dunlap <george.dun...@eu.citrix.com> > CC: Ian Jackson <ian.jack...@eu.citrix.com> > CC: Jan Beulich <jbeul...@suse.com> > CC: Julien Grall <jul...@xen.org> > CC: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> > CC: Stefano Stabellini <sstabell...@kernel.org> > CC: Wei Liu <w...@xen.org> > CC: Daniel De Graaf <dgde...@tycho.nsa.gov>
I realise there are arguments over how to fix this, but we (the Xen community) have already f*cked up once here, and this is doing so a second time. Nack. Fixing it anywhere other than Xen is simply not appropriate. The reason for this (which ought to be obvious, but I guess only to those who actually do customer support) is basic human physiology. "denied" means something has gone wrong. It scares people, and causes them to seek help to change fix whatever is broken. It is not appropriate for it to find its way into the guest in the first place, and that includes turning up in `dmesg` and other logs, and expecting guest runtime to filter for it is complete nonsense. As said several times before, the empty string is completely fine ABI wise, doesn't confuse customers, and really really does work in practice. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
