path: root/MAINTAINERS
diff options
authorLinus Torvalds <torvalds@linux-foundation.org>2015-04-21 09:17:28 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2015-04-21 09:17:28 -0700
commit8f443e2372ba23d51ee365974f54507acd6f69d1 (patch)
tree45491c137f2d684a49a1f01046f1eda6398e2aac /MAINTAINERS
parent646da63172f660ba84f195c1165360a9b73583ee (diff)
Revert "ocfs2: incorrect check for debugfs returns"
This reverts commit e2ac55b6a8e337fac7cc59c6f452caac92ab5ee6. Huang Ying reports that this causes a hang at boot with debugfs disabled. It is true that the debugfs error checks are kind of confusing, and this code certainly merits more cleanup and thinking about it, but there's something wrong with the trivial "check not just for NULL, but for error pointers too" patch. Yes, with debugfs disabled, we will end up setting the o2hb_debug_dir pointer variable to an error pointer (-ENODEV), and then continue as if everything was fine. But since debugfs is disabled, all the _users_ of that pointer end up being compiled away, so even though the pointer can not be dereferenced, that's still fine. So it's confusing and somewhat questionable, but the "more correct" error checks end up causing more trouble than they fix. Reported-by: Huang Ying <ying.huang@intel.com> Acked-by: Andrew Morton <akpm@linux-foundation.org> Acked-by: Chengyu Song <csong84@gatech.edu> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'MAINTAINERS')
0 files changed, 0 insertions, 0 deletions

Privacy Policy