| Message ID | 1479150525-13225-1-git-send-email-jacob.jun.pan@linux.intel.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Zhang Rui |
| Headers | show |
On Mon, Nov 14, 2016 at 11:08:45AM -0800, Jacob Pan wrote: > Commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 replaced module > cpu id table with a cpu feature check, which is logically correct. > But we need the module device table to allow module auto loading. > > Fixes:3105f234 thermal/powerclamp: correct cpu support check > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > --- > drivers/thermal/intel_powerclamp.c | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) <formletter> This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read Documentation/stable_kernel_rules.txt for how to do this properly. </formletter> -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, 15 Nov 2016 08:03:32 +0100 Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > On Mon, Nov 14, 2016 at 11:08:45AM -0800, Jacob Pan wrote: > > Commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 replaced module > > cpu id table with a cpu feature check, which is logically correct. > > But we need the module device table to allow module auto loading. > > > > Fixes:3105f234 thermal/powerclamp: correct cpu support check > > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > > --- > > drivers/thermal/intel_powerclamp.c | 9 ++++++++- > > 1 file changed, 8 insertions(+), 1 deletion(-) > > <formletter> > > This is not the correct way to submit patches for inclusion in the > stable kernel tree. Please read Documentation/stable_kernel_rules.txt > for how to do this properly. > > </formletter> Good to know, thanks. Rui will take care of it this time. Per Rui "I will apply patch 1 and queue up for next -rc and 4.8 stable." Thanks, Jacob -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, 2016-11-17 at 11:42 -0800, Jacob Pan wrote: > On Tue, 15 Nov 2016 08:03:32 +0100 > Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > > > > > On Mon, Nov 14, 2016 at 11:08:45AM -0800, Jacob Pan wrote: > > > > > > Commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 replaced module > > > cpu id table with a cpu feature check, which is logically > > > correct. > > > But we need the module device table to allow module auto loading. > > > > > > Fixes:3105f234 thermal/powerclamp: correct cpu support check > > > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > > > --- > > > drivers/thermal/intel_powerclamp.c | 9 ++++++++- > > > 1 file changed, 8 insertions(+), 1 deletion(-) > > <formletter> > > > > This is not the correct way to submit patches for inclusion in the > > stable kernel tree. Please read > > Documentation/stable_kernel_rules.txt > > for how to do this properly. > > > > </formletter> > Good to know, thanks. Rui will take care of it this time. Per Rui > "I will apply patch 1 and queue up for next -rc and 4.8 stable." > Just find another problem. We're still missing this upstream commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 (thermal/powerclamp: correct cpu support check) for 4.7 stable, and in this case, we can not queue this patch for both 4.7 and 4.8 stable at the moment because it does not apply to 4.7 stable. I will send this patch out asap to catch 4.9, and then send a note to stable kernel with the following Option 2 in Documentation/stable_kernel_rules.txt after it's merged. thanks, rui -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Nov 21, 2016 at 11:43:10AM +0800, Zhang Rui wrote: > On Thu, 2016-11-17 at 11:42 -0800, Jacob Pan wrote: > > On Tue, 15 Nov 2016 08:03:32 +0100 > > Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > > > > > > > > On Mon, Nov 14, 2016 at 11:08:45AM -0800, Jacob Pan wrote: > > > > > > > > Commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 replaced module > > > > cpu id table with a cpu feature check, which is logically > > > > correct. > > > > But we need the module device table to allow module auto loading. > > > > > > > > Fixes:3105f234 thermal/powerclamp: correct cpu support check > > > > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > > > > --- > > > > drivers/thermal/intel_powerclamp.c | 9 ++++++++- > > > > 1 file changed, 8 insertions(+), 1 deletion(-) > > > <formletter> > > > > > > This is not the correct way to submit patches for inclusion in the > > > stable kernel tree. Please read > > > Documentation/stable_kernel_rules.txt > > > for how to do this properly. > > > > > > </formletter> > > Good to know, thanks. Rui will take care of it this time. Per Rui > > "I will apply patch 1 and queue up for next -rc and 4.8 stable." > > > > Just find another problem. > We're still missing this upstream > commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 (thermal/powerclamp: > correct cpu support check) for 4.7 stable, and in this case, we can not > queue this patch for both 4.7 and 4.8 stable at the moment because it > does not apply to 4.7 stable. I don't understand, 4.7 is end-of-life, no one cares about it anymore, why are you worrying about that kernel version? confused, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 2016-11-21 at 08:13 +0100, Greg Kroah-Hartman wrote: > On Mon, Nov 21, 2016 at 11:43:10AM +0800, Zhang Rui wrote: > > > > On Thu, 2016-11-17 at 11:42 -0800, Jacob Pan wrote: > > > > > > On Tue, 15 Nov 2016 08:03:32 +0100 > > > Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > > > > > > > > > > > > > > > On Mon, Nov 14, 2016 at 11:08:45AM -0800, Jacob Pan wrote: > > > > > > > > > > > > > > > Commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 replaced > > > > > module > > > > > cpu id table with a cpu feature check, which is logically > > > > > correct. > > > > > But we need the module device table to allow module auto > > > > > loading. > > > > > > > > > > Fixes:3105f234 thermal/powerclamp: correct cpu support check > > > > > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > > > > > --- > > > > > drivers/thermal/intel_powerclamp.c | 9 ++++++++- > > > > > 1 file changed, 8 insertions(+), 1 deletion(-) > > > > <formletter> > > > > > > > > This is not the correct way to submit patches for inclusion in > > > > the > > > > stable kernel tree. Please read > > > > Documentation/stable_kernel_rules.txt > > > > for how to do this properly. > > > > > > > > </formletter> > > > Good to know, thanks. Rui will take care of it this time. Per Rui > > > "I will apply patch 1 and queue up for next -rc and 4.8 stable." > > > > > Just find another problem. > > We're still missing this upstream > > commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 > > (thermal/powerclamp: > > correct cpu support check) for 4.7 stable, and in this case, we can > > not > > queue this patch for both 4.7 and 4.8 stable at the moment because > > it > > does not apply to 4.7 stable. > I don't understand, 4.7 is end-of-life, no one cares about it > anymore, sorry, I was not aware of that. will send the patch out tomorrow. thanks, rui -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c index bd4914e..745fcec 100644 --- a/drivers/thermal/intel_powerclamp.c +++ b/drivers/thermal/intel_powerclamp.c @@ -704,9 +704,16 @@ static int powerclamp_set_cur_state(struct thermal_cooling_device *cdev, .set_cur_state = powerclamp_set_cur_state, }; +static const struct x86_cpu_id __initconst intel_powerclamp_ids[] = { + { X86_VENDOR_INTEL, X86_FAMILY_ANY, X86_MODEL_ANY, X86_FEATURE_MWAIT }, + {} +}; +MODULE_DEVICE_TABLE(x86cpu, intel_powerclamp_ids); + static int __init powerclamp_probe(void) { - if (!boot_cpu_has(X86_FEATURE_MWAIT)) { + + if (!x86_match_cpu(intel_powerclamp_ids)) { pr_err("CPU does not support MWAIT"); return -ENODEV; }
Commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 replaced module cpu id table with a cpu feature check, which is logically correct. But we need the module device table to allow module auto loading. Fixes:3105f234 thermal/powerclamp: correct cpu support check Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> --- drivers/thermal/intel_powerclamp.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)