| Message ID | 20220314070514.2602-1-jianchunfu@cmss.chinamobile.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 309b517276f21dc7e6315c6637792f8bbfdf7ec4 |
| Headers | show |
| Series | arch:x86:xen: Remove unnecessary assignment in xen_apic_read() | expand |
On 3/14/22 3:05 AM, jianchunfu wrote: > In the function xen_apic_read(), the initialized value of 'ret' is unused > because it will be assigned by the function HYPERVISOR_platform_op(), > thus remove it. > > Signed-off-by: jianchunfu <jianchunfu@cmss.chinamobile.com> Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com> Applied to for-linus-5.18 (both patches)
diff --git a/arch/x86/xen/apic.c b/arch/x86/xen/apic.c index 0d46cc283..62d34b661 100644 --- a/arch/x86/xen/apic.c +++ b/arch/x86/xen/apic.c @@ -51,7 +51,7 @@ static u32 xen_apic_read(u32 reg) .interface_version = XENPF_INTERFACE_VERSION, .u.pcpu_info.xen_cpuid = 0, }; - int ret = 0; + int ret; /* Shouldn't need this as APIC is turned off for PV, and we only * get called on the bootup processor. But just in case. */
In the function xen_apic_read(), the initialized value of 'ret' is unused because it will be assigned by the function HYPERVISOR_platform_op(), thus remove it. Signed-off-by: jianchunfu <jianchunfu@cmss.chinamobile.com> --- arch/x86/xen/apic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)