Your message dated Fri, 11 Mar 2022 23:26:05 +0100
with message-id <[email protected]>
and subject line Re: Bug#1006813: reportbug: lm-sensors.service executes 
sensors twice
has caused the Debian Bug report #1006813,
regarding reportbug: lm-sensors.service executes sensors twice
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
1006813: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1006813
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: lm-sensors
Version: 1:3.6.0-7
Severity: normal

In lm-sensors.service, the sensors program is executed twice.
The second execution happens without the -s flag, causing
sensors to read all sensors and print the results to the
system log.
On a Dell Inspiron 3505 however, this results in a freeze for
~4 seconds during boot, caused by the second (and pointless)
execution of sensors coupled with very slow sensor reads.

In order to fix this issue, the second (and pointless) execution
of sensors should be omited.

-- System Information:
Debian Release: 11.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable-security'), (500, 'stable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 5.17.0-rc6-hwmon-next+ (SMP w/4 CPU threads; PREEMPT)
Kernel taint flags: TAINT_UNSIGNED_MODULE
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages lm-sensors depends on:
ii  libc6        2.31-13+deb11u2
ii  libsensors5  1:3.6.0-7
ii  lsb-base     11.1.0
ii  perl         5.32.1-4+deb11u2
ii  sed          4.7-1

lm-sensors recommends no packages.

Versions of packages lm-sensors suggests:
pn  fancontrol  <none>
ii  i2c-tools   4.2-1+b1
ii  read-edid   3.0.2-1.1

-- no debconf information

--- End Message ---
--- Begin Message ---
On 2022-03-06 20:25, Armin Wolf wrote:
> Am 06.03.22 um 15:18 schrieb Aurelien Jarno:
> 
> > Hi,
> > 
> > On 2022-03-05 18:21, Armin Wolf wrote:
> > > Package: lm-sensors
> > > Version: 1:3.6.0-7
> > > Severity: normal
> > > 
> > > In lm-sensors.service, the sensors program is executed twice.
> > > The second execution happens without the -s flag, causing
> > > sensors to read all sensors and print the results to the
> > > system log.
> > Yes, the second call is actually done on purpose due to the way the
> > alarms work. The "sensors -s" sets the minimum and maximum limits,
> > however the ALARM flags are still latched until the next time the
> > sensors values are read. This is the purpose of the "sensors" call just
> > after, in other words it makes sure to clear the ALARM flags that might
> > have been triggered by different limits.
> > 
> > See /usr/share/doc/lm-sensors/README.Debian.gz to see how the ALARM
> > flags work.
> > 
> > > On a Dell Inspiron 3505 however, this results in a freeze for
> > > ~4 seconds during boot, caused by the second (and pointless)
> > > execution of sensors coupled with very slow sensor reads.
> > As said above the second call to sensors is not pointless. The bug is
> > actually on the kernel module side. You should get it fixed instead, and
> > in the meantime you can probably blacklist the module.
> > 
> > > In order to fix this issue, the second (and pointless) execution
> > > of sensors should be omited.
> > Not it shouldn't as it is not pointless. The kernel should be fixed
> > instead.
> > 
> > Regards,
> > Aurelien
> > 
> I understand, sorry for bothering you with this pointless bugreport then.
> I will try to introduce a fix inside the dell_smm_hwmon driver for my
> notebook model. Thank you for maintaining the lm-sensors package :).

No worry. Closing the bug with this mail.

Regards,
Aurelien

-- 
Aurelien Jarno                          GPG: 4096R/1DDD8C9B
[email protected]                 http://www.aurel32.net

--- End Message ---

Reply via email to