| Message ID | 20210629133322.19193-3-frankja@linux.ibm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | s390x: sie and uv cleanups | expand |
On Tue, Jun 29 2021, Janosch Frank <frankja@linux.ibm.com> wrote: > Let's only use the uint*_t types. > > Signed-off-by: Janosch Frank <frankja@linux.ibm.com> > --- > lib/s390x/sie.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Cornelia Huck <cohuck@redhat.com>
On 29/06/2021 15.33, Janosch Frank wrote: > Let's only use the uint*_t types. > > Signed-off-by: Janosch Frank <frankja@linux.ibm.com> > --- > lib/s390x/sie.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/lib/s390x/sie.h b/lib/s390x/sie.h > index b4bb78c..6ba858a 100644 > --- a/lib/s390x/sie.h > +++ b/lib/s390x/sie.h > @@ -173,9 +173,9 @@ struct kvm_s390_sie_block { > } __attribute__((packed)); > > struct vm_save_regs { > - u64 grs[16]; > - u64 fprs[16]; > - u32 fpc; > + uint64_t grs[16]; > + uint64_t fprs[16]; > + uint32_t fpc; > }; > > /* We might be able to nestle all of this into the stack frame. But > @@ -191,7 +191,7 @@ struct vm { > struct kvm_s390_sie_block *sblk; > struct vm_save_area save_area; > /* Ptr to first guest page */ > - u8 *guest_mem; > + uint8_t *guest_mem; > }; Yes, that's more consistent in this file. Reviewed-by: Thomas Huth <thuth@redhat.com>
On Tue, 29 Jun 2021 13:33:19 +0000 Janosch Frank <frankja@linux.ibm.com> wrote: > Let's only use the uint*_t types. > > Signed-off-by: Janosch Frank <frankja@linux.ibm.com> Reviewed-by: Claudio Imbrenda <imbrenda@linux.ibm.com> > --- > lib/s390x/sie.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/lib/s390x/sie.h b/lib/s390x/sie.h > index b4bb78c..6ba858a 100644 > --- a/lib/s390x/sie.h > +++ b/lib/s390x/sie.h > @@ -173,9 +173,9 @@ struct kvm_s390_sie_block { > } __attribute__((packed)); > > struct vm_save_regs { > - u64 grs[16]; > - u64 fprs[16]; > - u32 fpc; > + uint64_t grs[16]; > + uint64_t fprs[16]; > + uint32_t fpc; > }; > > /* We might be able to nestle all of this into the stack frame. But > @@ -191,7 +191,7 @@ struct vm { > struct kvm_s390_sie_block *sblk; > struct vm_save_area save_area; > /* Ptr to first guest page */ > - u8 *guest_mem; > + uint8_t *guest_mem; > }; > > extern void sie_entry(void);
diff --git a/lib/s390x/sie.h b/lib/s390x/sie.h index b4bb78c..6ba858a 100644 --- a/lib/s390x/sie.h +++ b/lib/s390x/sie.h @@ -173,9 +173,9 @@ struct kvm_s390_sie_block { } __attribute__((packed)); struct vm_save_regs { - u64 grs[16]; - u64 fprs[16]; - u32 fpc; + uint64_t grs[16]; + uint64_t fprs[16]; + uint32_t fpc; }; /* We might be able to nestle all of this into the stack frame. But @@ -191,7 +191,7 @@ struct vm { struct kvm_s390_sie_block *sblk; struct vm_save_area save_area; /* Ptr to first guest page */ - u8 *guest_mem; + uint8_t *guest_mem; }; extern void sie_entry(void);
Let's only use the uint*_t types. Signed-off-by: Janosch Frank <frankja@linux.ibm.com> --- lib/s390x/sie.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)