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

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



>>> On 11.06.15 at 10:28, <wei.w.wang@xxxxxxxxx> wrote:
> cpufreq_cpu_policy is used in intel_pstate_set_pstate(), so we change
> to NULL it after the call of cpufreq_driver->exit. Otherwise, a
> calltrace will show up on your screen due to the reference of a NULL
> pointer when you power down the system.

Apart from what was already said on this patch, I think it is placed
too late in this series (should precede patch 6) or, even better, not
needed at all: ->exit() gets passed the policy being cleaned up, so
I don't follow why the driver needs to consult the per-CPU field to
obtain it.

Plus, if the patch is to be kept, the description suggesting this to be
a problem at system shutdown only is wrong - it can equally well
happen while offlining a CPU. Just saying that the pointer is still
needed would be sufficient.

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®.