[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start inLmSensorsPlugin::update()

2021-05-27 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=437523

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||5.22

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start inLmSensorsPlugin::update()

2021-05-27 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=437523

David Redondo  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
   |ma/ksystemstats/commit/6be9 |ma/ksystemstats/commit/be6a
   |54add10b934f3683bc599849e0f |4b3aa80c67aacf61618293b9ec2
   |ee1da140d   |aacf374d9

--- Comment #8 from David Redondo  ---
Git commit be6a4b3aa80c67aacf61618293b9ec2aacf374d9 by David Redondo.
Committed on 27/05/2021 at 10:06.
Pushed by davidre into branch 'Plasma/5.22'.

Init lib sensors centrally in the daemon

Initing muliple times in the same process leads to crashes.


(cherry picked from commit 6be954add10b934f3683bc599849e0fee1da140d)

M  +1-1autotests/CMakeLists.txt
M  +0-1plugins/cpu/linuxcpuplugin.cpp
M  +0-4plugins/lmsensors/lmsensors.cpp
M  +6-0src/CMakeLists.txt
M  +10   -0src/daemon.cpp

https://invent.kde.org/plasma/ksystemstats/commit/be6a4b3aa80c67aacf61618293b9ec2aacf374d9

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start inLmSensorsPlugin::update()

2021-05-27 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=437523

David Redondo  changed:

   What|Removed |Added

 Resolution|--- |FIXED
  Latest Commit||https://invent.kde.org/plas
   ||ma/ksystemstats/commit/6be9
   ||54add10b934f3683bc599849e0f
   ||ee1da140d
 Status|CONFIRMED   |RESOLVED

--- Comment #7 from David Redondo  ---
Git commit 6be954add10b934f3683bc599849e0fee1da140d by David Redondo.
Committed on 25/05/2021 at 07:43.
Pushed by davidre into branch 'master'.

Init lib sensors centrally in the daemon

Initing muliple times in the same process leads to crashes.

M  +1-1autotests/CMakeLists.txt
M  +0-1plugins/cpu/linuxcpuplugin.cpp
M  +0-4plugins/lmsensors/lmsensors.cpp
M  +6-0src/CMakeLists.txt
M  +10   -0src/daemon.cpp

https://invent.kde.org/plasma/ksystemstats/commit/6be954add10b934f3683bc599849e0fee1da140d

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start inLmSensorsPlugin::update()

2021-05-26 Thread Maximiliano Bertacchini
https://bugs.kde.org/show_bug.cgi?id=437523

--- Comment #6 from Maximiliano Bertacchini  ---
(In reply to Nate Graham from comment #5)
> Nice work! Would you like to submit a merge request to fix it?

I'd love to, but... I don't yet understand the real issue, let alone how to fix
it :) (other than the hack of just ignoring those sensor types). Will keep
looking at libsensors internals for now.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start inLmSensorsPlugin::update()

2021-05-26 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=437523

Nate Graham  changed:

   What|Removed |Added

 Status|REPORTED|CONFIRMED
   Severity|normal  |crash
 CC||[email protected]
 Ever confirmed|0   |1
Summary|ksystemstats segfaults on   |ksystemstats segfaults on
   |start   |start
   ||inLmSensorsPlugin::update()

--- Comment #5 from Nate Graham  ---
Nice work! Would you like to submit a merge request to fix it?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start

2021-05-25 Thread Maximiliano Bertacchini
https://bugs.kde.org/show_bug.cgi?id=437523

--- Comment #4 from Maximiliano Bertacchini  ---
Just in case, the output of `sensors` on my laptop:

$ sensors
ucsi_source_psy_USBC000:001-isa-
Adapter: ISA adapter
in0:   0.00 V  (min =  +0.00 V, max =  +0.00 V)
curr1: 3.00 A  (max =  +0.00 A)

thinkpad-isa-
Adapter: ISA adapter
fan1:2477 RPM
fan2:2188 RPM
temp1:+41.0°C  
temp2:+41.0°C  
temp3: +0.0°C  
temp4: +0.0°C  
temp5: +0.0°C  
temp6: +0.0°C  
temp7: +0.0°C  
temp8: +0.0°C  
temp9: +0.0°C  
temp10:+0.0°C  
temp11:+1.0°C  
temp12:+0.0°C  
temp13:   +13.0°C  
temp14:+0.0°C  
temp15:+0.0°C  
temp16:+0.0°C  

coretemp-isa-
Adapter: ISA adapter
Package id 0:  +43.0°C  (high = +100.0°C, crit = +100.0°C)
Core 0:+42.0°C  (high = +100.0°C, crit = +100.0°C)
Core 1:+41.0°C  (high = +100.0°C, crit = +100.0°C)
Core 2:+42.0°C  (high = +100.0°C, crit = +100.0°C)
Core 3:+42.0°C  (high = +100.0°C, crit = +100.0°C)
Core 4:+41.0°C  (high = +100.0°C, crit = +100.0°C)
Core 5:+41.0°C  (high = +100.0°C, crit = +100.0°C)

BAT0-acpi-0
Adapter: ACPI interface
in0:  16.14 V  

ucsi_source_psy_USBC000:002-isa-
Adapter: ISA adapter
in0:   0.00 V  (min =  +0.00 V, max =  +0.00 V)
curr1: 0.00 A  (max =  +0.00 A)

iwlwifi_1-virtual-0
Adapter: Virtual device
temp1:+32.0°C  

pch_cannonlake-virtual-0
Adapter: Virtual device
temp1:+46.0°C  

nvme-pci-0200
Adapter: PCI adapter
Composite:+44.9°C  (low  =  -5.2°C, high = +79.8°C)
   (crit = +84.8°C)

acpitz-acpi-0
Adapter: ACPI interface
temp1:+41.0°C  (crit = +128.0°C)

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start

2021-05-25 Thread Maximiliano Bertacchini
https://bugs.kde.org/show_bug.cgi?id=437523

--- Comment #3 from Maximiliano Bertacchini  ---
Managed to debug a locally compiled build and pinpointed the crash to sensors
of type SENSORS_FEATURE_IN and SENSORS_FEATURE_CURR. So that if
makeSensorsFeatureSensor() returns nullptr on those types, ksystemstats works
fine with all the other sensors in my system (2 fans and a lot of
temperatures). Alternatively, it also works if SensorsFeatureSensor::update()
returns immediately on feature names "in0_input" or "curr1_input".

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start

2021-05-24 Thread Maximiliano Bertacchini
https://bugs.kde.org/show_bug.cgi?id=437523

--- Comment #2 from Maximiliano Bertacchini  ---
(In reply to Marco Martin from comment #1)
> seems specifoc to lmsensors

Indeed. Removing the lmsensors plugin from
"/usr/lib/x86_64-linux-gnu/qt5/plugins/ksystemstats" unlocks both
plasma-systemmonitor and kstatsviewer.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start

2021-05-24 Thread Marco Martin
https://bugs.kde.org/show_bug.cgi?id=437523

Marco Martin  changed:

   What|Removed |Added

 CC||[email protected]

--- Comment #1 from Marco Martin  ---
seems specifoc to lmsensors

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasma-systemmonitor] [Bug 437523] ksystemstats segfaults on start

2021-05-22 Thread Maximiliano Bertacchini
https://bugs.kde.org/show_bug.cgi?id=437523

Maximiliano Bertacchini  changed:

   What|Removed |Added

Version|unspecified |5.21.90

-- 
You are receiving this mail because:
You are watching all bug changes.