Re: Dell XPS13: MCE (Hardware Error) reported

From: Raj, Ashok
Date: Fri Jan 06 2017 - 10:59:27 EST


Hi Boris

On Fri, Jan 06, 2017 at 12:16:17PM +0100, Borislav Petkov wrote:
> On Thu, Jan 05, 2017 at 05:26:17PM -0800, Raj, Ashok wrote:
> > Agree, since we have both a log and another agent to deal with it, it makes
> > no good reason to continue... Will pass this along, and have someone look at
> > cleaning this up.
>
> Like this?

That was quick :-).

> - if (therm_throt_process(msr_val & THERM_STATUS_PROCHOT,
> - THERMAL_THROTTLING_EVENT,
> - CORE_LEVEL) != 0)

Looks like we don't need a return value from therm_throt_process(),
we can fix that as void as well.

Otherwise it looks good.

> + therm_throt_process(msr_val & THERM_STATUS_PROCHOT,
> + THERMAL_THROTTLING_EVENT,
> + CORE_LEVEL);
>

Cheers,
Ashok