On 15.03.2024 19:05, Oleksii Kurochko wrote: > --- a/xen/arch/riscv/configs/tiny64_defconfig > +++ b/xen/arch/riscv/configs/tiny64_defconfig > @@ -7,6 +7,23 @@ > # CONFIG_GRANT_TABLE is not set > # CONFIG_SPECULATIVE_HARDEN_ARRAY is not set > # CONFIG_MEM_ACCESS is not set > +# CONFIG_ARGO is not set > +# CONFIG_HYPFS_CONFIG is not set
What exactly is wrong with permitting this? > +# CONFIG_CORE_PARKING is not set At the example of this: It cannot be enabled by a user (or randconfig), it needs to be selected by an arch. Which RISC-V doesn't. Hence why specify (and not really override) the value here? This may apply to others as well, at the very least ... > +# CONFIG_DEBUG_TRACE is not set > +# CONFIG_IOREQ_SERVER is not set > +# CONFIG_CRASH_DEBUG is not setz > +# CONFIG_KEXEC is not set > +# CONFIG_LIVEPATCH is not set > +# CONFIG_NUMA is not set > +# CONFIG_PERF_COUNTERS is not set > +# CONFIG_HAS_PMAP is not set ... anything CONFIG_HAS_*. In summary - please limit overrides to what is minimally necessary. Jan > +# CONFIG_TRACEBUFFER is not set > +# CONFIG_XENOPROF is not set > +# CONFIG_COMPAT is not set > +# CONFIG_COVERAGE is not set > +# CONFIG_UBSAN is not set > +# CONFIG_NEEDS_LIBELF is not set > > CONFIG_RISCV_64=y > CONFIG_DEBUG=y
