Skip to contentSkip to conversation

Commit 'synchronize incompat feature bits with sysfs files' still missing in for-next?

DISCUSSION Holger Hoffstätte 2 participants 2 messages Sep 13, 2016 – Oct 3, 2016 Btrfs file system

lore
Share & tools
lore
#0 lore raw

I've noticed that the 4.5-rc commit 14e46e04:
'btrfs: synchronize incompat feature bits with sysfs files' [1] was reverted later in [2], but despite fixes to protect sysfs with locks & exorcise GFP_NOFS in favor of GFP_KERNEL it was never reinstated - neither for 4.5-final, nor later..and it's been quite a while since then. Is this still valid?

I ask because I've just noticed I've had this in my tree since forever, but have never encountered a problem during balance - probably because of all the other fixes.

thanks,
Holger

[1] goo.gl/2DBMSe
[2] goo.gl/cIKgv5

#1 lore raw
Quoted 11 lines · Holger Hoffstätte

I've noticed that the 4.5-rc commit 14e46e04:
'btrfs: synchronize incompat feature bits with sysfs files' [1] was reverted later in [2], but despite fixes to protect sysfs with locks & exorcise GFP_NOFS in favor of GFP_KERNEL it was never reinstated - neither for 4.5-final, nor later..and it's been quite a while since then. Is this still valid?

I ask because I've just noticed I've had this in my tree since forever, but have never encountered a problem during balance - probably because of all the other fixes.

I spent sime time trying to fix it but got lost in sysfs. Supposedly, the attribute group functions should provide all what's needed, ie. define a set of attributes, visibility callback. Calling an update will, in theory and my expectations, refresh the values and files. It does not. IIRC it was never meant to work like that, so it would have to be worked around or new sysfs helpers adde or I don't know what else. Working with sysfs code is sometimes painful, but I'm about to return to adding the free-space-tree bits someday.