csa
[Top] [All Lists]

Re: [PATCH 2.6.9-rc2 2/2] enhanced MM accounting data collection

To: LKML <linux-kernel@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH 2.6.9-rc2 2/2] enhanced MM accounting data collection
From: Jay Lan <jlan@xxxxxxxxxxxx>
Date: Mon, 27 Sep 2004 15:50:15 -0700
Cc: lse-tech <lse-tech@xxxxxxxxxxxxxxxxxxxxx>, CSA-ML <csa@xxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxx>, Guillaume Thouvenin <guillaume.thouvenin@xxxxxxxx>, Tim Schmielau <tim@xxxxxxxxxxxxxxxxxxxxxx>, Arthur Corliss <corliss@xxxxxxxxxxxxxxxx>
In-reply-to: <4158956F.3030706@engr.sgi.com>
References: <4158956F.3030706@engr.sgi.com>
Sender: csa-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20030225
2/2: acct_mm

Enhanced MM accounting data collection.

Signed-off-by: Jay Lan <jlan@xxxxxxx>

Index: linux/fs/exec.c
===================================================================
--- linux.orig/fs/exec.c        2004-09-27 11:57:40.201435722 -0700
+++ linux/fs/exec.c     2004-09-27 14:05:41.266160725 -0700
@@ -47,6 +47,7 @@
 #include <linux/syscalls.h>
 #include <linux/rmap.h>
 #include <linux/pagg.h>
+#include <linux/csa_internal.h>
 
 #include <asm/uaccess.h>
 #include <asm/mmu_context.h>
@@ -1163,6 +1164,9 @@
 
                /* execve success */
                security_bprm_free(&bprm);
+               /* no-op if CONFIG_CSA not set */
+                csa_update_integrals();
+                update_mem_hiwater();
                return retval;
        }
 
Index: linux/include/linux/csa_internal.h
===================================================================
--- /dev/null   1970-01-01 00:00:00.000000000 +0000
+++ linux/include/linux/csa_internal.h  2004-09-27 14:05:41.279832688 -0700
@@ -0,0 +1,70 @@
+/*
+ * Copyright (c) 2000-2002 Silicon Graphics, Inc and LANL  All Rights Reserved.
+ * Copyright (c) 2004 Silicon Graphics, Inc All Rights Reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it would be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 59 Temple Place - Suite 330, Boston MA 02111-1307, USA.
+ *
+ * Contact information:  Silicon Graphics, Inc., 1500 Crittenden Lane,
+ * Mountain View, CA  94043, or:
+ *
+ * http://www.sgi.com
+ */
+
+/*
+ *  CSA (Comprehensive System Accounting)
+ *  Job Accounting for Linux
+ *
+ *  This header file contains the definitions needed for communication
+ *  between the kernel and the CSA module.
+ */
+
+#ifndef _LINUX_CSA_INTERNAL_H
+#define _LINUX_CSA_INTERNAL_H
+
+#include <linux/config.h>
+
+#if defined (CONFIG_CSA) || defined (CONFIG_CSA_MODULE)
+
+#include <linux/linkage.h>
+#include <linux/ptrace.h>
+
+static inline void csa_update_integrals(void)
+{
+       long delta;
+
+       if (current->mm) {
+               delta = current->stime - current->csa_stimexpd;
+               current->csa_stimexpd = current->stime;
+               current->csa_rss_mem1 += delta * current->mm->rss;
+               current->csa_vm_mem1 += delta * current->mm->total_vm;
+       }
+}
+
+static inline void csa_clear_integrals(struct task_struct *tsk)
+{
+       if (tsk) {
+               tsk->csa_stimexpd = 0;
+               tsk->csa_rss_mem1 = 0;
+               tsk->csa_vm_mem1 = 0;
+       }       
+}
+
+#else  /* CONFIG_CSA || CONFIG_CSA_MODULE */
+
+#define csa_update_integrals()         do { } while (0)
+#define csa_clear_integrals(task)      do { } while (0)
+#endif /* CONFIG_CSA || CONFIG_CSA_MODULE */
+
+#endif /* _LINUX_CSA_INTERNAL_H */
Index: linux/include/linux/sched.h
===================================================================
--- linux.orig/include/linux/sched.h    2004-09-27 14:04:52.905497872 -0700
+++ linux/include/linux/sched.h 2004-09-27 14:06:35.938387661 -0700
@@ -249,6 +249,8 @@
        struct kioctx           *ioctx_list;
 
        struct kioctx           default_kioctx;
+
+       unsigned long hiwater_rss, hiwater_vm;
 };
 
 extern int mmlist_nr;
@@ -593,6 +595,10 @@
 
 /* i/o counters(bytes read/written, #syscalls, waittime */
        unsigned long rchar, wchar, syscr, syscw, bwtime;
+#if defined(CONFIG_CSA) || defined(CONFIG_CSA_MODULE)
+       u64 csa_rss_mem1, csa_vm_mem1;
+       clock_t csa_stimexpd;
+#endif
 
 };
 
@@ -817,6 +823,19 @@
 /* Remove the current tasks stale references to the old mm_struct */
 extern void mm_release(struct task_struct *, struct mm_struct *);
 
+/* Update highwater values */
+static inline void update_mem_hiwater(void)
+{
+       if (current->mm) {
+               if (current->mm->hiwater_rss < current->mm->rss) {
+                       current->mm->hiwater_rss = current->mm->rss;
+               }
+               if (current->mm->hiwater_vm < current->mm->total_vm) {
+                       current->mm->hiwater_vm = current->mm->total_vm;
+               }
+       }
+}
+
 extern int  copy_thread(int, unsigned long, unsigned long, unsigned long, 
struct task_struct *, struct pt_regs *);
 extern void flush_thread(void);
 extern void exit_thread(void);
Index: linux/kernel/exit.c
===================================================================
--- linux.orig/kernel/exit.c    2004-09-27 11:57:40.247334460 -0700
+++ linux/kernel/exit.c 2004-09-27 14:05:41.292528082 -0700
@@ -25,6 +25,7 @@
 #include <linux/proc_fs.h>
 #include <linux/mempolicy.h>
 #include <linux/pagg.h>
+#include <linux/csa_internal.h>
 
 #include <asm/uaccess.h>
 #include <asm/unistd.h>
@@ -808,6 +809,9 @@
                ptrace_notify((PTRACE_EVENT_EXIT << 8) | SIGTRAP);
        }
 
+       /* no-op if CONFIG_CSA not set */
+       csa_update_integrals();
+       update_mem_hiwater();
        acct_process(code);
        __exit_mm(tsk);
 
Index: linux/kernel/fork.c
===================================================================
--- linux.orig/kernel/fork.c    2004-09-27 12:35:06.585377528 -0700
+++ linux/kernel/fork.c 2004-09-27 14:05:41.296434358 -0700
@@ -39,7 +39,7 @@
 #include <linux/profile.h>
 #include <linux/rmap.h>
 #include <linux/pagg.h>
-
+#include <linux/csa_internal.h>
 #include <asm/pgtable.h>
 #include <asm/pgalloc.h>
 #include <asm/uaccess.h>
@@ -607,6 +607,9 @@
        if (retval)
                goto free_pt;
 
+       mm->hiwater_rss = mm->rss;
+       mm->hiwater_vm = mm->total_vm;  
+
 good_mm:
        tsk->mm = mm;
        tsk->active_mm = mm;
@@ -995,6 +998,8 @@
        p->utime = p->stime = 0;
        p->rchar = p->wchar = p->syscr = p->syscw = 0;
        p->bwtime = 0;
+       /* no-op if CONFIG_CSA not set */
+       csa_clear_integrals(p);
        p->lock_depth = -1;             /* -1 = no lock */
        p->start_time = get_jiffies_64();
        p->security = NULL;
Index: linux/mm/memory.c
===================================================================
--- linux.orig/mm/memory.c      2004-09-12 22:32:26.000000000 -0700
+++ linux/mm/memory.c   2004-09-27 14:05:41.304246908 -0700
@@ -44,6 +44,7 @@
 #include <linux/highmem.h>
 #include <linux/pagemap.h>
 #include <linux/rmap.h>
+#include <linux/csa_internal.h>
 #include <linux/module.h>
 #include <linux/init.h>
 
@@ -605,6 +606,8 @@
        tlb = tlb_gather_mmu(mm, 0);
        unmap_vmas(&tlb, mm, vma, address, end, &nr_accounted, details);
        tlb_finish_mmu(tlb, address, end);
+       /* no-op unless CONFIG_CSA is set */
+        csa_update_integrals();
        spin_unlock(&mm->page_table_lock);
 }
 
@@ -1095,9 +1098,12 @@
        spin_lock(&mm->page_table_lock);
        page_table = pte_offset_map(pmd, address);
        if (likely(pte_same(*page_table, pte))) {
-               if (PageReserved(old_page))
+               if (PageReserved(old_page)) {
                        ++mm->rss;
-               else
+                       /* no-op if CONFIG_CSA not set */
+                       csa_update_integrals();
+                       update_mem_hiwater();
+               } else
                        page_remove_rmap(old_page);
                break_cow(vma, new_page, address, page_table);
                lru_cache_add_active(new_page);
@@ -1379,6 +1385,10 @@
                remove_exclusive_swap_page(page);
 
        mm->rss++;
+       /* no-op if CONFIG_CSA not set */
+       csa_update_integrals();
+       update_mem_hiwater();
+  
        pte = mk_pte(page, vma->vm_page_prot);
        if (write_access && can_share_swap_page(page)) {
                pte = maybe_mkwrite(pte_mkdirty(pte), vma);
@@ -1444,6 +1454,9 @@
                        goto out;
                }
                mm->rss++;
+               /* no-op if CONFIG_CSA not set */
+               csa_update_integrals();
+               update_mem_hiwater();
                entry = maybe_mkwrite(pte_mkdirty(mk_pte(page,
                                                         vma->vm_page_prot)),
                                      vma);
@@ -1553,6 +1566,10 @@
        if (pte_none(*page_table)) {
                if (!PageReserved(new_page))
                        ++mm->rss;
+               /* no-op if CONFIG_CSA not set */
+               csa_update_integrals();
+               update_mem_hiwater();
+
                flush_icache_page(vma, new_page);
                entry = mk_pte(new_page, vma->vm_page_prot);
                if (write_access)
Index: linux/mm/mmap.c
===================================================================
--- linux.orig/mm/mmap.c        2004-09-12 22:32:54.000000000 -0700
+++ linux/mm/mmap.c     2004-09-27 14:05:41.308153183 -0700
@@ -20,6 +20,7 @@
 #include <linux/hugetlb.h>
 #include <linux/profile.h>
 #include <linux/module.h>
+#include <linux/csa_internal.h>
 #include <linux/mount.h>
 #include <linux/mempolicy.h>
 #include <linux/rmap.h>
@@ -1014,6 +1015,9 @@
                down_write(&mm->mmap_sem);
        }
        __vm_stat_account(mm, vm_flags, file, len >> PAGE_SHIFT);
+       /* no-op if CONFIG_CSA not set */
+        csa_update_integrals();
+        update_mem_hiwater();
        return addr;
 
 unmap_and_free_vma:
@@ -1360,6 +1364,9 @@
        if (vma->vm_flags & VM_LOCKED)
                vma->vm_mm->locked_vm += grow;
        __vm_stat_account(vma->vm_mm, vma->vm_flags, vma->vm_file, grow);
+       /* no-op if CONFIG_CSA_JOB_ACCT not set */
+       csa_update_integrals();
+       update_mem_hiwater();
        anon_vma_unlock(vma);
        return 0;
 }
@@ -1816,6 +1823,9 @@
                mm->locked_vm += len >> PAGE_SHIFT;
                make_pages_present(addr, addr + len);
        }
+       /* no-op if CONFIG_CSA not set */
+       csa_update_integrals();
+       update_mem_hiwater();
        return addr;
 }
 
Index: linux/mm/mremap.c
===================================================================
--- linux.orig/mm/mremap.c      2004-09-12 22:32:48.000000000 -0700
+++ linux/mm/mremap.c   2004-09-27 14:05:41.312059458 -0700
@@ -16,6 +16,7 @@
 #include <linux/fs.h>
 #include <linux/highmem.h>
 #include <linux/security.h>
+#include <linux/csa_internal.h>
 
 #include <asm/uaccess.h>
 #include <asm/cacheflush.h>
@@ -232,6 +233,10 @@
                                           new_addr + new_len);
        }
 
+       /* no-op if CONFIG_CSA not set */
+       csa_update_integrals();
+       update_mem_hiwater();
+
        return new_addr;
 }
 
@@ -368,6 +373,9 @@
                                make_pages_present(addr + old_len,
                                                   addr + new_len);
                        }
+                       /* no-op if CONFIG_CSA not set */
+                       csa_update_integrals();
+                       update_mem_hiwater();
                        ret = addr;
                        goto out;
                }
Index: linux/mm/rmap.c
===================================================================
--- linux.orig/mm/rmap.c        2004-09-12 22:33:36.000000000 -0700
+++ linux/mm/rmap.c     2004-09-27 14:05:41.315965733 -0700
@@ -50,6 +50,7 @@
 #include <linux/swapops.h>
 #include <linux/slab.h>
 #include <linux/init.h>
+#include <linux/csa_internal.h>
 #include <linux/rmap.h>
 #include <linux/rcupdate.h>
 
@@ -580,6 +581,8 @@
        }
 
        mm->rss--;
+       /* no-op if CONFIG_CSA not set */
+       csa_update_integrals();
        page_remove_rmap(page);
        page_cache_release(page);
 
@@ -679,6 +682,8 @@
 
                page_remove_rmap(page);
                page_cache_release(page);
+               /* no-op if CONFIG_CSA not set */
+               csa_update_integrals();
                mm->rss--;
                (*mapcount)--;
        }
Index: linux/mm/swapfile.c
===================================================================
--- linux.orig/mm/swapfile.c    2004-09-12 22:31:57.000000000 -0700
+++ linux/mm/swapfile.c 2004-09-27 14:05:41.320848577 -0700
@@ -24,6 +24,7 @@
 #include <linux/module.h>
 #include <linux/rmap.h>
 #include <linux/security.h>
+#include <linux/csa_internal.h>
 #include <linux/backing-dev.h>
 
 #include <asm/pgtable.h>
@@ -435,6 +436,9 @@
        set_pte(dir, pte_mkold(mk_pte(page, vma->vm_page_prot)));
        page_add_anon_rmap(page, vma, address);
        swap_free(entry);
+       /* no-op if CONFIG_CSA not set */
+       csa_update_integrals();
+       update_mem_hiwater();
 }
 
 /* vma->vm_mm->page_table_lock is held */
<Prev in Thread] Current Thread [Next in Thread>