[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v4 08/11] x86/intel_pstate: changes in cpufreq_del_cpu for CPU offline



>>> On 25.06.15 at 13:16, <wei.w.wang@xxxxxxxxx> wrote:
> We change to NULL the cpufreq_cpu_policy pointer after the call of
> cpufreq_driver->exit, because the pointer is still needed in
> intel_pstate_set_pstate().

Just to continue from where I finished on the previous patch: This
is an acceptable change if the goal is for intel_pstate.c to be
minimally different from its Linux original. As said in a reply to an
earlier version, since it can be done differently (i.e. without doing
what you do here), I don't view it as acceptable if the delta of the
two files is as big as it is right now.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.