Thread (1 message) 1 message, 1 author, 2018-05-17

Re: [PATCH 34/40] atm: simplify procfs code

From: Eric W. Biederman <hidden>
Date: 2018-05-17 01:15:21
Also in: linux-acpi, linux-ext4, linux-ide, linux-scsi, lkml, netdev, netfilter-devel

Possibly related (same subject, not in this thread)

Christoph Hellwig [off-list ref] writes:
On Sat, May 05, 2018 at 07:51:18AM -0500, Eric W. Biederman wrote:
quoted
Christoph Hellwig [off-list ref] writes:
quoted
Use remove_proc_subtree to remove the whole subtree on cleanup, and
unwind the registration loop into individual calls.  Switch to use
proc_create_seq where applicable.
Can you please explain why you are removing the error handling when
you are unwinding the registration loop?
Because there is no point in handling these errors.  The code work
perfectly fine without procfs, or without given proc files and the
removal works just fine if they don't exist either.  This is a very
common patter in various parts of the kernel already.

I'll document it better in the changelog.
Thank you.  That is the kind of thing that could be a signal of
inattentiveness and problems, especially when it is not documented.

Eric
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help