My attributes are way too racy, what should I do?
Asked Answered
V

1

25

In a linux device driver, creating sysfs attributes in probe is way too racy--specifically, it experiences a race condition with userspace. The recommended workaround is to add your attributes to various default attribute groups so they can be automatically created before probe. For a device driver, struct device_driver contains const struct attribute_group **groups for this purpose.

However, struct attribute_group only got a field for binary attributes in Linux 3.11. With older kernels (specifically, 3.4), how should a device driver create sysfs binary attributes before probe?

Velamen answered 27/7, 2016 at 18:35 Comment(5)
Aw, you should have given it a bit to see how much more eyeballs you got...Exedra
@Will restored...let's give it a try.Velamen
@RadLexus you might notice that that blog post is actually the first link in my question...Velamen
Maybe there is no way to do that, becase the merge comment itself says: "Also here is the ability to create binary files as attribute groups, to solve that race condition, which was impossible to do before this, so that's my fault the drivers were broken."Supraliminal
@RomanKhimov That seems like a fairly authoritative source to say "it's not possible." Submit it as an answer and I'll accept it.Velamen
S
5

Quoting (emphasis mine) Greg Kroah-Hartman from his comment to a merge request (that was merged by Linus as a part of 3.11 development cycle):

Here are some driver core patches for 3.11-rc2. They aren't really bugfixes, but a bunch of new helper macros for drivers to properly create attribute groups, which drivers and subsystems need to fix up a ton of race issues with incorrectly creating sysfs files (binary and normal) after userspace has been told that the device is present.

Also here is the ability to create binary files as attribute groups, to solve that race condition, which was impossible to do before this, so that's my fault the drivers were broken.

So it looks like there really is no way to solve this problem on old kernels.

Supraliminal answered 16/8, 2016 at 6:0 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.