While reviewing Claudio's patch set I found a problem that should've been caught by the diag10 test but wasn't. Not only does the test never check the good case but it also doesn't fence against environments where diag10 is not available. The part that makes this problematic is the fact that this test only tests priv and spec PGMs. These PGMs are presented even if no diag10 support is provided since they are also part of the base diagnose architecture. The tests currently succeed in TCG emulation and PV, both of which do not implement this specific diagnose. Therefore this series fences TCG & PV as well as adding a check if the page has really been cleared. Janosch Frank (2): s390x: diag10: Fence tcg and pv environments s390x: diag10: Check page clear s390x/diag10.c | 26 ++++++++++++++++++++++++++ s390x/unittests.cfg | 1 + 2 files changed, 27 insertions(+) -- 2.48.1
