Commit a3cadf38 authored by K. Y. Srinivasan's avatar K. Y. Srinivasan Committed by Greg Kroah-Hartman
Browse files

Drivers: hv: vmbus: Get rid of unnecessary state in hv_context 



Currently we are replicating state in struct hv_context that is unnecessary -
this state can be retrieved from the hypervisor. Furthermore, this is a per-cpu
state that is being maintained as a global state in struct hv_context.
Get rid of this state in struct hv_context.

Signed-off-by: default avatarK. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 78ef4193

drivers/hv/hv.c

+3 −7
Original line number Diff line number Diff line
@@ -33,9 +33,7 @@ 
#include "hyperv_vmbus.h"



/* The one and only */

struct hv_context hv_context = {

	.synic_initialized	= false,

};

struct hv_context hv_context;



/*

 * If false, we're using the old mechanism for stimer0 interrupts
@@ -326,8 +324,6 @@ int hv_synic_init(unsigned int cpu) 


	hv_set_synic_state(sctrl.as_uint64);



	hv_context.synic_initialized = true;



	/*

	 * Register the per-cpu clockevent source.

	 */
@@ -373,7 +369,8 @@ int hv_synic_cleanup(unsigned int cpu) 
	bool channel_found = false;

	unsigned long flags;



	if (!hv_context.synic_initialized)

	hv_get_synic_state(sctrl.as_uint64);

	if (sctrl.enable != 1)

		return -EFAULT;



	/*
@@ -435,7 +432,6 @@ int hv_synic_cleanup(unsigned int cpu) 
	hv_set_siefp(siefp.as_uint64);



	/* Disable the global synic bit */

	hv_get_synic_state(sctrl.as_uint64);

	sctrl.enable = 0;

	hv_set_synic_state(sctrl.as_uint64);

drivers/hv/hyperv_vmbus.h

+0 −2
Original line number Diff line number Diff line
@@ -230,8 +230,6 @@ struct hv_context { 


	void *tsc_page;



	bool synic_initialized;



	struct hv_per_cpu_context __percpu *cpu_context;



	/*

CRA Git | Maintained and supported by SUSTech CRA and CCSE