| Message ID | 20211116193955.2793171-1-philmd@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | [PATCH-for-7.0] target/i386/kvm: Replace use of __u32 type | expand |
On 11/16/21 8:39 PM, Philippe Mathieu-Daudé wrote: > QEMU coding style mandates to not use Linux kernel internal > types for scalars types. Replace __u32 by uint32_t. > > Signed-off-by: Philippe Mathieu-Daudé<philmd@redhat.com> > --- > target/i386/kvm/kvm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~
Hi Laurent, This patch is reviewed, can it go via your trivial tree? On 11/16/21 20:39, Philippe Mathieu-Daudé wrote: > QEMU coding style mandates to not use Linux kernel internal > types for scalars types. Replace __u32 by uint32_t. > > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> > --- > target/i386/kvm/kvm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/i386/kvm/kvm.c b/target/i386/kvm/kvm.c > index 5a698bde19a..13f8e30c2a5 100644 > --- a/target/i386/kvm/kvm.c > +++ b/target/i386/kvm/kvm.c > @@ -1406,7 +1406,7 @@ static int hyperv_fill_cpuids(CPUState *cs, > c->edx = cpu->hyperv_limits[2]; > > if (hyperv_feat_enabled(cpu, HYPERV_FEAT_EVMCS)) { > - __u32 function; > + uint32_t function; > > /* Create zeroed 0x40000006..0x40000009 leaves */ > for (function = HV_CPUID_IMPLEMENT_LIMITS + 1; >
Le 16/11/2021 à 20:39, Philippe Mathieu-Daudé a écrit : > QEMU coding style mandates to not use Linux kernel internal > types for scalars types. Replace __u32 by uint32_t. > > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> > --- > target/i386/kvm/kvm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/i386/kvm/kvm.c b/target/i386/kvm/kvm.c > index 5a698bde19a..13f8e30c2a5 100644 > --- a/target/i386/kvm/kvm.c > +++ b/target/i386/kvm/kvm.c > @@ -1406,7 +1406,7 @@ static int hyperv_fill_cpuids(CPUState *cs, > c->edx = cpu->hyperv_limits[2]; > > if (hyperv_feat_enabled(cpu, HYPERV_FEAT_EVMCS)) { > - __u32 function; > + uint32_t function; > > /* Create zeroed 0x40000006..0x40000009 leaves */ > for (function = HV_CPUID_IMPLEMENT_LIMITS + 1; > Applied to my trivial-patches branch. Thanks, Laurent
diff --git a/target/i386/kvm/kvm.c b/target/i386/kvm/kvm.c index 5a698bde19a..13f8e30c2a5 100644 --- a/target/i386/kvm/kvm.c +++ b/target/i386/kvm/kvm.c @@ -1406,7 +1406,7 @@ static int hyperv_fill_cpuids(CPUState *cs, c->edx = cpu->hyperv_limits[2]; if (hyperv_feat_enabled(cpu, HYPERV_FEAT_EVMCS)) { - __u32 function; + uint32_t function; /* Create zeroed 0x40000006..0x40000009 leaves */ for (function = HV_CPUID_IMPLEMENT_LIMITS + 1;
QEMU coding style mandates to not use Linux kernel internal types for scalars types. Replace __u32 by uint32_t. Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> --- target/i386/kvm/kvm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)