On 25/01/2023 22.26, Janis Schoetterl-Glausch wrote:
Instead of having one function covering all mem_op operations, have a function implementing absolute access and dispatch to that function in its caller, based on the operation code. This way additional future operations can be implemented by adding an implementing function without changing existing operations. Suggested-by: Janosch Frank <frankja@xxxxxxxxxxxxx> Signed-off-by: Janis Schoetterl-Glausch <scgl@xxxxxxxxxxxxx> --- arch/s390/kvm/kvm-s390.c | 38 ++++++++++++++++++++++++-------------- 1 file changed, 24 insertions(+), 14 deletions(-)
Reviewed-by: Thomas Huth <thuth@xxxxxxxxxx>