[v2,0/3] KVM: s390: Some gaccess cleanup

Message

Janis Schoetterl-Glausch Oct. 28, 2021, 1:55 p.m. UTC

Cleanup s390 guest access code a bit, getting rid of some code
duplication and improving readability.

v1 -> v2
	separate patch for renamed variable
		fragment_len instead of seg
	expand comment of guest_range_to_gpas
	fix nits

I did not pick up Janosch's Reviewed-by because of the split patch
and the changed variable name.

Janis Schoetterl-Glausch (3):
  KVM: s390: gaccess: Refactor gpa and length calculation
  KVM: s390: gaccess: Refactor access address range check
  KVM: s390: gaccess: Cleanup access to guest frames

 arch/s390/kvm/gaccess.c | 158 +++++++++++++++++++++++-----------------
 1 file changed, 92 insertions(+), 66 deletions(-)

Comments

Janosch Frank Nov. 23, 2021, 12:39 p.m. UTC | #1

On 10/28/21 15:55, Janis Schoetterl-Glausch wrote:
> Cleanup s390 guest access code a bit, getting rid of some code
> duplication and improving readability.
> 
> v1 -> v2
> 	separate patch for renamed variable
> 		fragment_len instead of seg
> 	expand comment of guest_range_to_gpas
> 	fix nits
> 
> I did not pick up Janosch's Reviewed-by because of the split patch
> and the changed variable name.
> 
> Janis Schoetterl-Glausch (3):
>    KVM: s390: gaccess: Refactor gpa and length calculation
>    KVM: s390: gaccess: Refactor access address range check
>    KVM: s390: gaccess: Cleanup access to guest frames
> 
>   arch/s390/kvm/gaccess.c | 158 +++++++++++++++++++++++-----------------
>   1 file changed, 92 insertions(+), 66 deletions(-)
> 

Could you please push this to devel so we get some CI coverage?