Re: [PATCH] Provide refrigerator support for irda

From: Martin Diehl (lists@mdiehl.de)
Date: Wed Jun 25 2003 - 01:18:31 EST


On Tue, 24 Jun 2003, Jean Tourrilhes wrote:

> On Wed, Jun 25, 2003 at 11:56:53AM +1000, Neil Brown wrote:
> >
> > Without this patch, kIrDAd prevents my notebook from entering suspend
> > mode.

Are you talking about normal apm or acpi suspend or this swsusp thing?

> Wow ! I knew about microwave for 802.11b, but not about
> refrigerator for IrDA.

Inclined to say "me too".

> The man for sir_kthread is Martin Diehl. He is much more
> intimate with kernel tasks than me, and he has other sir_dev updates
> in the pipeline.

Admittedly I didn't care about swsusp so far. Given the big fat warning on
top of Documentation/swsusp.txt I would not even try and i personally
don't see what I would miss without swsusp.

But of course, if we can make all parties happy, why not.

> > + if (current->flags & PF_FREEZE)
> > + refrigerator(PF_IOTHREAD);
> > +

I've tried to find some documentation about this - without success. So I
have no idea why this might be needed and what it will do. Grepping for
other kernel threads it looks like most of them use the same two lines.
OTOH the irda thread is very similar to keventd's workqueue which do not
use this. Not sure about the reason.

Well, I'm thinking about making the irda thread using workqueue anyway, in
which case the issue would either disappear or get shifted to
kernel/workqueue.c.

For the meantime, I think we should apply this if somebody would explain
what's going on here.

Martin

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



This archive was generated by hypermail 2b29 : Mon Jun 30 2003 - 22:00:18 EST