Thread (3 messages) 3 messages, 3 authors, 2021-02-12

Re: [PATCH] gpiolib: cdev: convert stream-like files from

From: Linus Walleij <hidden>
Date: 2021-02-12 09:01:54
Also in: lkml

On Sun, Feb 7, 2021 at 10:00 AM Yang Li [off-list ref] wrote:
Eliminate the following coccicheck warning:
./drivers/gpio/gpiolib-cdev.c:2307:7-23: WARNING: gpio_fileops: .read()
has stream semantic; safe to change nonseekable_open -> stream_open.

Reported-by: Abaci Robot <redacted>
Signed-off-by: Yang Li <redacted>
This doesn't make any sense to me.

It is pretty clear from context that this file should *not* be
seeked and it seems just dangerous to randomly allow that.
Better safe than sorry.

I don't know if the function nonseekable_open() has an unintuitive
name if it means anything else than that. I burnt myself on the FS
before.

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