OpenVZ Forum


Home » Mailing lists » Devel » Re: [ckrm-tech] [PATCH 3/3][RFC] Containers: Pagecache controller reclaim
Re: [ckrm-tech] [PATCH 3/3][RFC] Containers: Pagecache controller reclaim [message #10883] Tue, 06 March 2007 11:13 Go to next message
Vaidyanathan Srinivas is currently offline  Vaidyanathan Srinivas
Messages: 49
Registered: February 2007
Member
Shane wrote:
> Anyone else have trouble fitting this patch ???.
> I see a later version today, but not markedly different from this
> mornings (Aus time). Initially I thought I had the first version, prior
> to Balbir's RSS controller V2 re-write, but apparently not.
> Kernel 2.6.20.1

Hi Shane,

I did post the same patch again today since the previous post
yesterday did not showup on LKML. I have not changed the version
since it is the same patch.

Next time around i will explicitly mention that this is the same patch
posted again.

> Had to toss it away so I could do some base line testing - I'll redo the
> build and see where the mis-matches are.

Please let me know if so see any problem running the patch. The
patches are against 2.6.20 only since dependent patches are at that level.

--Vaidy
Re: [ckrm-tech] [PATCH 3/3][RFC] Containers: Pagecache controller reclaim [message #10946 is a reply to message #10883] Wed, 07 March 2007 09:03 Go to previous message
Shane is currently offline  Shane
Messages: 2
Registered: March 2007
Junior Member
On Tue, 2007-03-06 at 16:43 +0530, Vaidyanathan Srinivasan wrote:
>
> Please let me know if so see any problem running the patch. The
> patches are against 2.6.20 only since dependent patches are at that level.

My problem - a bad copy of the patch. It patches o.k.
However, it fails to compile vmscan. This looks a bit dodgy;

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ ++++++++++++

@@ -1470,11 +1494,13 @@ unsigned long shrink_all_memory(unsigned
int pass;
struct reclaim_state reclaim_state;
struct scan_control sc = {
- .gfp_mask = GFP_KERNEL,
+ .gfp_mask = GFdefined(CONFIG_CONTAINER_PAGECACHE_ACCT)
+P_KERNEL,

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ ++++++++++++

I deleted what looks like an over-enthusiastic "copy-and-paste", and it
compiled o.k.
Testing continues.

Shane ...
Previous Topic: [PATCH] Fix race between proc_get_inode() and remove_proc_entry()
Next Topic: utrace regressions (was: -mm merge plans for 2.6.21)
Goto Forum:
  


Current Time: Thu Jul 31 23:51:42 GMT 2025

Total time taken to generate the page: 0.28589 seconds