Re: [PATCH] w1: w1_process() is not freezable kthread

From: Jiri Kosina
Date: Wed Oct 28 2015 - 01:27:03 EST


On Tue, 27 Oct 2015, Evgeniy Polyakov wrote:

> > w1_process() calls try_to_freeze(), but the thread doesn't mark itself
> > freezable through set_freezable(), so the try_to_freeze() call is useless.
>
> I believe it is better to mark it freezable, what do you think? Its task
> is useless if anyone else goes sleeping, it should freeze too.

I fail to see why this kthread should be freezable at all. There is no way
for w1 device to generate new I/O requests that should be written out to
filesystem, is it?

Thanks,

--
Jiri Kosina
SUSE Labs

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/