public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Nigel Cunningham <nigel@suspend2.net>
To: Hugh Dickins <hugh@veritas.com>
Cc: Pavel Machek <pavel@ucw.cz>, linux-kernel@vger.kernel.org
Subject: Re: [Suspend2][ 15/20] [Suspend2] Attempt to freeze processes.
Date: Thu, 29 Jun 2006 08:10:43 +1000	[thread overview]
Message-ID: <200606290810.47025.nigel@suspend2.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0606281943010.24170@blonde.wat.veritas.com>

[-- Attachment #1: Type: text/plain, Size: 1650 bytes --]

Hi Hugh.

On Thursday 29 June 2006 04:59, Hugh Dickins wrote:
> On Wed, 28 Jun 2006, Nigel Cunningham wrote:
> > On Tuesday 27 June 2006 23:45, Pavel Machek wrote:
> > > Current code seems to free memory without need to thaw/re-freeze
> > > kernel threads. Have you found bugs in that, or is this unneccessary?
> >
> > Did you read my other email? Try it with a swap file on a journalled
> > filesystem, in a situation where freeing memory will force the swap file
> > to be used.
>
> Hi Nigel,
>
> I may have missed your "other email" in the avalanche ;)

:)

> That particular example sounds dubious to me: it may well have been
> a problem on 2.4, but are you sure that it's still a problem on 2.6?

Yes, I am sure but I'll double check again. What I recall at the moment is a 
deadlock in actually writing the page.

> Andrew very nicely rewrote the swapfile handling, to bmap the whole
> file at swapon time (see setup_swap_extents), and thereafter the only
> difference between using a swapfile and using a disk partition is that
> the swapfile blocks may be fragmented into many extents where the disk
> partition is contiguous.  Much more reliable.
>
> I don't see how your "journalled filesystem" would affect it at all.

Ok. I'll reproduce it and post the trace. Of course it may be that my 
examination was too superficial and the cause is more subtle.

I'm not sure if I'll have time to do this during this week, but I'll leave 
your message marked Todo so I don't forget.

Thanks for the reply!

Nigel

-- 
See http://www.suspend2.net for Howtos, FAQs, mailing
lists, wiki and bugzilla info.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

  reply	other threads:[~2006-06-28 22:10 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-26 22:34 [Suspend2][ 00/20] Prepare image Nigel Cunningham
2006-06-26 22:34 ` [Suspend2][ 01/20] [Suspend2] Prepare_image.c header Nigel Cunningham
2006-06-26 22:34 ` [Suspend2][ 02/20] [Suspend2] Get number of pcp pages Nigel Cunningham
2006-06-26 22:34 ` [Suspend2][ 03/20] [Suspend2] Get the real number of free pages (incl. pcp) Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 04/20] [Suspend2] Calculate pagedir1 growth allowance Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 05/20] [Suspend2] Get the amount of storage needed for the image proper Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 06/20] [Suspend2] Calculate header storage needed Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 07/20] [Suspend2] Display image vital statistics Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 08/20] [Suspend2] Generate free page bitmap Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 09/20] [Suspend2] Get size of a free region Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 10/20] [Suspend2] Count pages in image parts Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 11/20] [Suspend2] Amount of memory still to be freed Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 12/20] [Suspend2] Recalculate image contents Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 13/20] [Suspend2] Try to refreeze processes Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 14/20] [Suspend2] Update the image Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 15/20] [Suspend2] Attempt to freeze processes Nigel Cunningham
2006-06-27 13:45   ` Pavel Machek
2006-06-27 23:38     ` Nigel Cunningham
2006-06-28 18:59       ` Hugh Dickins
2006-06-28 22:10         ` Nigel Cunningham [this message]
2006-06-26 22:35 ` [Suspend2][ 16/20] [Suspend2] Calculate storage needed for an image Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 17/20] [Suspend2] Calculate the amount of free memory needed Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 18/20] [Suspend2] Free up memory if necessary Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 19/20] [Suspend2] Prepare an image Nigel Cunningham
2006-06-26 22:35 ` [Suspend2][ 20/20] [Suspend2] Prepare image header file Nigel Cunningham

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200606290810.47025.nigel@suspend2.net \
    --to=nigel@suspend2.net \
    --cc=hugh@veritas.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@ucw.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox