Hi!

> This patch adds a freezer call to the slow path in __alloc_pages. It
> thus avoids freezing failures in low memory situations. Like the other
> patches, it has been in Suspend2 for longer than I can remember.

This one seems wrong.

What if someone does

        down(&some_lock_needed_during_suspend);
        kmalloc()

? If you freeze him during that allocation, you'll deadlock later...

                                                                Pavel


> Signed-of-by: Nigel Cunningham <[EMAIL PROTECTED]>
> 
> diff -ruNp 213-missing-refrigerator-calls-old/mm/page_alloc.c 
> 213-missing-refrigerator-calls-new/mm/page_alloc.c
> --- 213-missing-refrigerator-calls-old/mm/page_alloc.c        2005-02-03 
> 22:33:50.000000000 +1100
> +++ 213-missing-refrigerator-calls-new/mm/page_alloc.c        2005-03-16 
> 09:01:28.000000000 +1100
> @@ -838,6 +838,7 @@ rebalance:
>                       do_retry = 1;
>       }
>       if (do_retry) {
> +             try_to_freeze(0);
>               blk_congestion_wait(WRITE, HZ/50);
>               goto rebalance;
>       }


-- 
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to