Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1635855 > unrolled thread

Re: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state machine conversion

Started byTommi Rantala <tt.rantala@gmail.com>
First post2017-05-04 17:50 +0200
Last post2017-05-10 16:10 +0200
Articles 5 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state  machine conversion Tommi Rantala <tt.rantala@gmail.com> - 2017-05-04 17:50 +0200
    Re: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state  machine conversion Thomas Gleixner <tglx@linutronix.de> - 2017-05-09 09:20 +0200
      Re: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state  machine conversion Tommi Rantala <tt.rantala@gmail.com> - 2017-05-10 16:00 +0200
        Re: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state  machine conversion Thomas Gleixner <tglx@linutronix.de> - 2017-05-10 16:10 +0200
          Re: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state  machine conversion Tommi Rantala <tt.rantala@gmail.com> - 2017-05-10 16:10 +0200

#1635855 — Re: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state machine conversion

FromTommi Rantala <tt.rantala@gmail.com>
Date2017-05-04 17:50 +0200
SubjectRe: [patch 0/6] hwmon/coretemp: Hotplug fixes, cleanups and state machine conversion
Message-ID<tDraV-5t6-5@gated-at.bofh.it>
2017-04-23 18:01 GMT+03:00 Thomas Gleixner <tglx@linutronix.de>:
> On Sat, 15 Apr 2017, Tommi Rantala wrote:
>
>> Testing with 4.10.8-200.fc25.x86_64: freezer, devices and platform are
>> OK, it breaks at "processors".
>> The screen stays off, and the machine no longer answers to ping.
>>
>> (Without coretemp loaded, the machine survives all the states. There
>> are some graphics glitches and radeon error messages)
>
> That's odd. I tried on a similar machine (w/o a radeon card) and it just
> works with the coretemp module loaded.
>
> Can you please do a CPU hotplug cycle (just one CPU) with the cpuhp events
> in the tracer enabled. Send me the trace output so I might be able to spot
> whats different and what interdependencies between other callbacks might be
> there.

Hi,

Here's the trace output, does it help?

http://termbin.com/qugr

-Tommi

[toc] | [next] | [standalone]


#1637918

FromThomas Gleixner <tglx@linutronix.de>
Date2017-05-09 09:20 +0200
Message-ID<tF7B8-6TD-7@gated-at.bofh.it>
In reply to#1635855
On Thu, 4 May 2017, Tommi Rantala wrote:
> 2017-04-23 18:01 GMT+03:00 Thomas Gleixner <tglx@linutronix.de>:
> > On Sat, 15 Apr 2017, Tommi Rantala wrote:
> >
> >> Testing with 4.10.8-200.fc25.x86_64: freezer, devices and platform are
> >> OK, it breaks at "processors".
> >> The screen stays off, and the machine no longer answers to ping.
> >>
> >> (Without coretemp loaded, the machine survives all the states. There
> >> are some graphics glitches and radeon error messages)
> >
> > That's odd. I tried on a similar machine (w/o a radeon card) and it just
> > works with the coretemp module loaded.
> >
> > Can you please do a CPU hotplug cycle (just one CPU) with the cpuhp events
> > in the tracer enabled. Send me the trace output so I might be able to spot
> > whats different and what interdependencies between other callbacks might be
> > there.
> 
> Hi,
> 
> Here's the trace output, does it help?

Not much. Can you please try the following:

1) Offline all CPUs except CPU0 before suspend/resume

2) Offline all CPUs except CPU0 and CPU1 before suspend/resume

3) Offline all CPUs except CPU0 and CPU2 before suspend/resume

Thanks,

	tglx

[toc] | [prev] | [next] | [standalone]


#1638854

FromTommi Rantala <tt.rantala@gmail.com>
Date2017-05-10 16:00 +0200
Message-ID<tFAjL-1pF-1@gated-at.bofh.it>
In reply to#1637918
2017-05-09 10:16 GMT+03:00 Thomas Gleixner <tglx@linutronix.de>:
> On Thu, 4 May 2017, Tommi Rantala wrote:
>> Here's the trace output, does it help?
>
> Not much. Can you please try the following:
>
> 1) Offline all CPUs except CPU0 before suspend/resume

it works!

> 2) Offline all CPUs except CPU0 and CPU1 before suspend/resume

now it breaks.

> 3) Offline all CPUs except CPU0 and CPU2 before suspend/resume

works again!

(Also works with CPUs 0,2,4,6 onlined.)

-Tommi

[toc] | [prev] | [next] | [standalone]


#1638863

FromThomas Gleixner <tglx@linutronix.de>
Date2017-05-10 16:10 +0200
Message-ID<tFAts-1IC-17@gated-at.bofh.it>
In reply to#1638854
On Wed, 10 May 2017, Tommi Rantala wrote:
> 2017-05-09 10:16 GMT+03:00 Thomas Gleixner <tglx@linutronix.de>:
> > On Thu, 4 May 2017, Tommi Rantala wrote:
> >> Here's the trace output, does it help?
> >
> > Not much. Can you please try the following:
> >
> > 1) Offline all CPUs except CPU0 before suspend/resume
> 
> it works!
> 
> > 2) Offline all CPUs except CPU0 and CPU1 before suspend/resume
> 
> now it breaks.
> 
> > 3) Offline all CPUs except CPU0 and CPU2 before suspend/resume
> 
> works again!
> 
> (Also works with CPUs 0,2,4,6 onlined.)

Output from /proc/cpuinfo please.

[toc] | [prev] | [next] | [standalone]


#1638865

FromTommi Rantala <tt.rantala@gmail.com>
Date2017-05-10 16:10 +0200
Message-ID<tFAts-1IC-21@gated-at.bofh.it>
In reply to#1638863
2017-05-10 17:01 GMT+03:00 Thomas Gleixner <tglx@linutronix.de>:
> On Wed, 10 May 2017, Tommi Rantala wrote:
>> 2017-05-09 10:16 GMT+03:00 Thomas Gleixner <tglx@linutronix.de>:
>> > On Thu, 4 May 2017, Tommi Rantala wrote:
>> >> Here's the trace output, does it help?
>> >
>> > Not much. Can you please try the following:
>> >
>> > 1) Offline all CPUs except CPU0 before suspend/resume
>>
>> it works!
>>
>> > 2) Offline all CPUs except CPU0 and CPU1 before suspend/resume
>>
>> now it breaks.
>>
>> > 3) Offline all CPUs except CPU0 and CPU2 before suspend/resume
>>
>> works again!
>>
>> (Also works with CPUs 0,2,4,6 onlined.)
>
> Output from /proc/cpuinfo please.

http://termbin.com/vec2

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web