Hi, I got a lot of noise trying to debug an AIX guest in a pseries machine when running with '-d unimp'. The reason is that there is no distinction between features (in my case, hypercalls) that are unimplemented because we never considered, versus features that we made a design choice not to implement.
This series adds a new log type, LOG_UNSUPP, as a way to filter the second case. After changing the log level of existing unsupported pseries hypercalls, -d unimp was reporting just the ones that I need to worry about and decide whether we should implement it or mark as unsupported in our model. After this series there's still one hypercall thgat is being thrown by AIX. We'll deal with that another day. I am posting this a little too close of the soft freeze. If we manage to get it reviewed in time, since it's a debug QoL change that doesn't break anything, I'd say we take it. Also, I didn't find the maintainer for the util/log.c file. Let me know if I need to CC someone in special for this file. Daniel Henrique Barboza (9): util/log.c: add LOG_UNSUPP type hw/ppc/spapr_hcall.c: log h_clean_slb() as unsupported hw/ppc/spapr_hcall.c: log h_invalidate_pid() as unsupported hw/ppc/spapr_hcall.c: log h_copy_tofrom_guest() as unsupported hw/ppc/spapr_hcall.c: log H_GET_EM_PARMS as unsupported hw/ppc/spapr_hcall.c: log H_BEST_ENERGY as unsupported hw/ppc/spapr_hcall.c: log H_QUERY_VAS_CAPABILITIES as unsupported hw/ppc/spapr_hcall.c: log H_GET_PPP as unsupported hw/ppc/spapr_hcall.c: log H_VIOCTL as unsupported hw/ppc/spapr_hcall.c | 98 +++++++++++++++++++++++++++++++++++++++++- include/hw/ppc/spapr.h | 3 ++ include/qemu/log.h | 3 +- util/log.c | 2 + 4 files changed, 103 insertions(+), 3 deletions(-) -- 2.35.1
