summaryrefslogtreecommitdiffstats
path: root/Documentation
diff options
context:
space:
mode:
authorEric W. Biederman <ebiederm@xmission.com>2012-01-13 21:32:59 -0800
committerGreg Kroah-Hartman <gregkh@suse.de>2012-01-24 12:12:32 -0800
commitce597919361dcec97341151690e780eade2a9cf4 (patch)
tree7734c3d6a07173cb509be7a2ac7318fd389510bb /Documentation
parent8381b5e88a19b780f19fc52b7dd67f2b05d07dca (diff)
downloadop-kernel-dev-ce597919361dcec97341151690e780eade2a9cf4.zip
op-kernel-dev-ce597919361dcec97341151690e780eade2a9cf4.tar.gz
sysfs: Complain bitterly about attempts to remove files from nonexistent directories.
Recently an OOPS was observed from the usb serial io_ti driver when it tried to remove sysfs directories. Upon investigation it turns out this driver was always buggy and that a recent sysfs change had stopped guarding itself against removing attributes from sysfs directories that had already been removed. :( Historically we have been silent about attempting to files from nonexistent sysfs directories and have politely returned error codes. That has resulted in people writing broken code that ignores the error codes. Issue a kernel WARNING and a stack backtrace to make it clear in no uncertain terms that abusing sysfs is not ok, and the callers need to fix their code. This change transforms the io_ti OOPS into a more comprehensible error message and stack backtrace. Signed-off-by: Eric W. Biederman <ebiederm@xmission.com> Reported-by: Wolfgang Frisch <wfpub@roembden.net> Cc: stable <stable@vger.kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud