Skip to contentSkip to conversation

hwmon: (nct6775) Add missing sysfs attibute initialization

PATCH In review Guenter Roeck 3 participants 4 messages May 28, 2015 – May 31, 2015 Hardware monitoring

lore
Share & tools
lore

Guenter Roeck

author maintainer
#0 lore raw

PATCH 1/2 hwmon: (nct6775) Add missing sysfs attibute initialization

The following error message is seen when loading the nct6775 driver with DEBUG_LOCK_ALLOC enabled.

BUG: key ffff88040b2f0030 not in .data!
------------[ cut here ]------------
WARNING: CPU: 0 PID: 186 at kernel/locking/lockdep.c:2988
				lockdep_init_map+0x469/0x630()

DEBUG_LOCKS_WARN_ON(1)

Caused by a missing call to sysfs_attr_init() when initializing sysfs attributes.

---
 drivers/hwmon/nct6775.c | 2 ++
 1 file changed, 2 insertions(+)
drivers/hwmon/nct6775.c +2 -0
index 4fcb48103299..bd1c99deac71 100644
@@ -995,6 +995,7 @@ nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg,
 				 (*t)->dev_attr.attr.name, tg->base + i);
 			if ((*t)->s2) {
 				a2 = &su->u.a2;
+				sysfs_attr_init(&a2->dev_attr.attr);
 				a2->dev_attr.attr.name = su->name;
 				a2->nr = (*t)->u.s.nr + i;
 				a2->index = (*t)->u.s.index;
@@ -1005,6 +1006,7 @@ nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg,
 				*attrs = &a2->dev_attr.attr;
 			} else {
 				a = &su->u.a1;
+				sysfs_attr_init(&a->dev_attr.attr);
 				a->dev_attr.attr.name = su->name;
 				a->index = (*t)->u.index + i;
 				a->dev_attr.attr.mode -- 

2.1.0

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Reviewed-by on v1 1/2
#1 lore raw
Quoted 31 lines · Guenter Roeck

The following error message is seen when loading the nct6775 driver with DEBUG_LOCK_ALLOC enabled.

BUG: key ffff88040b2f0030 not in .data!
------------[ cut here ]------------
WARNING: CPU: 0 PID: 186 at kernel/locking/lockdep.c:2988
				lockdep_init_map+0x469/0x630()

DEBUG_LOCKS_WARN_ON(1)

Caused by a missing call to sysfs_attr_init() when initializing sysfs attributes.

---
 drivers/hwmon/nct6775.c | 2 ++
 1 file changed, 2 insertions(+)
drivers/hwmon/nct6775.c +1 -0
index 4fcb48103299..bd1c99deac71 100644
@@ -995,6 +995,7 @@ nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg,
 				 (*t)->dev_attr.attr.name, tg->base + i);
 			if ((*t)->s2) {
 				a2 = &su->u.a2;
+				sysfs_attr_init(&a2->dev_attr.attr);
 				a2->dev_attr.attr.name = su->name;
 				a2->nr = (*t)->u.s.nr + i;
 				a2->index = (*t)->u.s.index;

drivers/hwmon/nct6775.c nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg, · line 1012

Quoted 7 lines
drivers/hwmon/nct6775.c +1 -0
@@ -1005,6 +1006,7 @@ nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg,
 				*attrs = &a2->dev_attr.attr;
 			} else {
 				a = &su->u.a1;
+				sysfs_attr_init(&a->dev_attr.attr);
 				a->dev_attr.attr.name = su->name;
 				a->index = (*t)->u.index + i;
 				a->dev_attr.attr.mode 
Signature · 7 lines
Jean Delvare
SUSE L3 Support

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
Tested-by on v1 1/2
#2 lore raw
In reply to Guenter Roeck
Quoted 32 lines · Jean Delvare
Quoted 31 lines · Guenter Roeck

The following error message is seen when loading the nct6775 driver with DEBUG_LOCK_ALLOC enabled.

BUG: key ffff88040b2f0030 not in .data!
------------[ cut here ]------------
WARNING: CPU: 0 PID: 186 at kernel/locking/lockdep.c:2988
                              lockdep_init_map+0x469/0x630()

DEBUG_LOCKS_WARN_ON(1)

Caused by a missing call to sysfs_attr_init() when initializing sysfs attributes.

---
 drivers/hwmon/nct6775.c | 2 ++
 1 file changed, 2 insertions(+)
drivers/hwmon/nct6775.c +1 -0
index 4fcb48103299..bd1c99deac71 100644
@@ -995,6 +995,7 @@ nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg,
                               (*t)->dev_attr.attr.name, tg->base + i);
                      if ((*t)->s2) {
                              a2 = &su->u.a2;
+                             sysfs_attr_init(&a2->dev_attr.attr);
                              a2->dev_attr.attr.name = su->name;
                              a2->nr = (*t)->u.s.nr + i;
                              a2->index = (*t)->u.s.index;
@@ -1005,6 +1006,7 @@ nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg,
Quoted 8 lines

drivers/hwmon/nct6775.c nct6775_create_attr_group(struct device *dev, struct sensor_template_group *tg,

Quoted 7 lines
drivers/hwmon/nct6775.c +1 -0
                              *attrs = &a2->dev_attr.attr;
                      } else {
                              a = &su->u.a1;
+                             sysfs_attr_init(&a->dev_attr.attr);
                              a->dev_attr.attr.name = su->name;
                              a->index = (*t)->u.index + i;
                              a->dev_attr.attr.mode >

Tested on ASRock E3C222D4U motherboard with kernel 4.0.3; reported problem is fixed.

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

author maintainer
#3 lore raw
In reply to Guenter Roeck
Quoted 16 lines · Alexey Orishko
Quoted 15 lines · Jean Delvare
Quoted 14 lines · Guenter Roeck

The following error message is seen when loading the nct6775 driver with DEBUG_LOCK_ALLOC enabled.

BUG: key ffff88040b2f0030 not in .data!
------------[ cut here ]------------
WARNING: CPU: 0 PID: 186 at kernel/locking/lockdep.c:2988
                               lockdep_init_map+0x469/0x630()

DEBUG_LOCKS_WARN_ON(1)

Caused by a missing call to sysfs_attr_init() when initializing sysfs attributes.

[ ... ]

Quoted 7 lines
Quoted 2 lines

Tested on ASRock E3C222D4U motherboard with kernel 4.0.3; reported problem is fixed.

Hi Alexey,

Great, thanks a lot for testing!

The fix is already available in mainline, and should eventually find its way into 4.0.x.

Guenter

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors