Hi. On Thursday 09 February 2006 17:00, Magnus Damm wrote: > Hi Nigel, > > On 2/2/06, Nigel Cunningham <[email protected]> wrote: > > Hi everyone. > > > > This is my latest revision of the dynamically allocated pageflags patch. > > > > The patch is useful for kernel space applications that sometimes need to flag > > pages for some purpose, but don't otherwise need the retain the state. A prime > > example is suspend-to-disk, which needs to flag pages as unsaveable, allocated > > by suspend-to-disk and the like while it is working, but doesn't need to > > retain any of this state between cycles. > > > > Since the last revision, I have switched to using per-zone bitmaps within each > > bitmap. > > > > I know that I could still add hotplug memory support. Is there anything else > > missing? > > I like the idea of the patch, but the code looks a bit too complicated > IMO. What is wrong with using vmalloc() to allocate a virtual > contiguous range of 0-order pages (one bit per page), and then use the > functions in linux/bitmap.h...? Or maybe I'm misunderstanding. > > A system that has 2 GB RAM and 4 KB pages would use 64 KB per bitmap > (one bitmap per node), which is not so bad memory wise if you plan to > use all bits. > > OTOH, if your plan is to use a single bit here and there, and leave > most of the bits unused then some kind of tree is probably better. > > Or does the kernel already implement some kind of data structure that > never consumes _that_ much more space than a bitmap when fully used, > and saves a lot of memory when just sparsely populated? Thanks for the suggestion - I'll look into it too. Part of my reason for implementing them in this way was to make serialising the data in an image header easy, when suspending to disk. I use the bitmaps to record which pages were atomically copied, and generate a similar bitmap for addresses of the freshly loaded pages prior to restoring the image at resume time. Of course the routines that does the atomic restore also iterates through these bitmaps. I'm not sure whether that changes anything, but thought it was worth mentioning. Regards, Nigel
Attachment:
pgpFvXm1v4dgL.pgp
Description: PGP signature
- References:
- [PATCH] Dynamically allocated pageflags
- From: Nigel Cunningham <[email protected]>
- Re: [PATCH] Dynamically allocated pageflags
- From: Magnus Damm <[email protected]>
- [PATCH] Dynamically allocated pageflags
- Prev by Date: [RFC:PATCH(003/003)] Memory add to onlined node. (ver. 2) (For x86_64)
- Next by Date: Re: msync() behaviour broken for MS_ASYNC, revert patch?
- Previous by thread: Re: [PATCH] Dynamically allocated pageflags
- Next by thread: [ANNOUNCE] OpenVZ patch for 2.6.15 kernel
- Index(es):
 |