Re: [PATCH 1/2] KVM: s390: selftests: Refactor memop test

From: Shuah Khan
Date: Thu Feb 17 2022 - 12:36:24 EST


On 2/17/22 7:53 AM, Janis Schoetterl-Glausch wrote:
Introduce macro for performing MEM_OP ioctls in a concise way.

How does this help? What is the value in re-writing existing
code and turning it into a macro?


Split test cases into multiple host/guest pairs making them independent.

This is a good change.

Make various minor improvements.
All in all this lays the groundwork for future extensions.

Also good if these changes make it easier to add test. Would be helpful
to know the details of the groundwork.

thanks,
-- Shuah