Hello,

I would like to update this patch since there were redundant extern 
declarations.

Regards,
Hideo

---

This patch includes changes in network core sub system for memory
accounting.

Memory scheduling, charging, uncharging and reclaiming functions are
added. These functions use sk_forward_alloc to store socket local
accounting. They currently support only datagram protocols.

sk_datagram_rfree() is a receive buffer detractor for datagram
protocols which are capable of protocol specific memory accounting.

Cc: Satoshi Oshima <[EMAIL PROTECTED]>
signed-off-by: Takahiro Yasui <[EMAIL PROTECTED]>
signed-off-by: Masami Hiramatsu <[EMAIL PROTECTED]>
signed-off-by: Hideo Aoki <[EMAIL PROTECTED]>
---

 include/net/sock.h  |   78 +++++++++++++++++++++++++++++++++++++++++++++++
 net/core/datagram.c |   85 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 163 insertions(+)

diff -pruN net-2.6-udp-take11a1-p1/include/net/sock.h 
net-2.6-udp-take11a1-p2/include/net/sock.h
--- net-2.6-udp-take11a1-p1/include/net/sock.h  2007-12-11 10:54:53.000000000 
-0500
+++ net-2.6-udp-take11a1-p2/include/net/sock.h  2007-12-18 18:57:35.000000000 
-0500
@@ -750,6 +750,9 @@ static inline struct inode *SOCK_INODE(s
        return &container_of(socket, struct socket_alloc, socket)->vfs_inode;
 }

+/*
+ * Functions for memory accounting
+ */
 extern void __sk_stream_mem_reclaim(struct sock *sk);
 extern int sk_stream_mem_schedule(struct sock *sk, int size, int kind);

@@ -778,6 +781,79 @@ static inline int sk_stream_wmem_schedul
               sk_stream_mem_schedule(sk, size, 0);
 }

+extern int __sk_datagram_account_charge(struct sock *sk, int size, int kind);
+extern void __sk_datagram_mem_reclaim(struct sock *sk);
+extern int sk_datagram_mem_schedule(struct sock *sk, int size, int kind);
+
+#define SK_DATAGRAM_MEM_QUANTUM ((unsigned int)PAGE_SIZE)
+
+static inline int sk_datagram_pages(int amt)
+{
+       /* Cast to unsigned as an optimization, since amt is always positive. */
+       return DIV_ROUND_UP((unsigned int)amt, SK_DATAGRAM_MEM_QUANTUM);
+}
+
+static inline void sk_datagram_mem_reclaim(struct sock *sk)
+{
+       if (!sk->sk_prot->memory_allocated)
+               return;
+
+       __sk_datagram_mem_reclaim(sk);
+}
+
+static inline int sk_datagram_rmem_schedule(struct sock *sk, int size)
+{
+       return size <= sk->sk_forward_alloc ||
+               sk_datagram_mem_schedule(sk, size, 1);
+}
+
+static inline int sk_datagram_wmem_schedule(struct sock *sk, int size)
+{
+       return size <= sk->sk_forward_alloc ||
+               sk_datagram_mem_schedule(sk, size, 0);
+}
+
+static inline void sk_mem_reclaim(struct sock *sk)
+{
+       if (sk->sk_type == SOCK_DGRAM)
+               sk_datagram_mem_reclaim(sk);
+}
+
+static inline int sk_wmem_schedule(struct sock *sk, int size)
+{
+       if (sk->sk_type == SOCK_DGRAM)
+               return sk_datagram_wmem_schedule(sk, size);
+       else
+               return 1;
+}
+
+static inline int sk_account_rmem_charge(struct sock *sk, int size)
+{
+       /* account if protocol supports memory accounting. */
+       if (!sk->sk_prot->memory_allocated || sk->sk_type != SOCK_DGRAM)
+               return 1;
+
+       return __sk_datagram_account_charge(sk, size, 1);
+}
+
+static inline int sk_account_wmem_charge(struct sock *sk, int size)
+{
+       /* account if protocol supports memory accounting. */
+       if (!sk->sk_prot->memory_allocated || sk->sk_type != SOCK_DGRAM)
+               return 1;
+
+       return __sk_datagram_account_charge(sk, size, 0);
+}
+
+static inline void sk_account_uncharge(struct sock *sk, int size)
+{
+       /* account if protocol supports memory accounting. */
+       if (!sk->sk_prot->memory_allocated || sk->sk_type != SOCK_DGRAM)
+               return;
+
+       sk->sk_forward_alloc += size;
+}
+
 /* Used by processes to "lock" a socket state, so that
  * interrupts and bottom half handlers won't change it
  * from under us. It essentially blocks any incoming
@@ -1166,6 +1242,8 @@ static inline void skb_set_owner_r(struc
        atomic_add(skb->truesize, &sk->sk_rmem_alloc);
 }

+extern void sk_datagram_rfree(struct sk_buff *skb);
+
 extern void sk_reset_timer(struct sock *sk, struct timer_list* timer,
                           unsigned long expires);

diff -pruN net-2.6-udp-take11a1-p1/net/core/datagram.c 
net-2.6-udp-take11a1-p2/net/core/datagram.c
--- net-2.6-udp-take11a1-p1/net/core/datagram.c 2007-12-11 10:54:55.000000000 
-0500
+++ net-2.6-udp-take11a1-p2/net/core/datagram.c 2007-12-18 18:49:39.000000000 
-0500
@@ -484,6 +484,91 @@ fault:
 }

 /**
+ *     sk_datagram_rfree - receive buffer detractor for datagram protocls
+ *     @skb: skbuff
+ */
+void sk_datagram_rfree(struct sk_buff *skb)
+{
+       struct sock *sk = skb->sk;
+
+       skb_truesize_check(skb);
+       atomic_sub(skb->truesize, &sk->sk_rmem_alloc);
+       sk_account_uncharge(sk, skb->truesize);
+       sk_datagram_mem_reclaim(sk);
+}
+EXPORT_SYMBOL(sk_datagram_rfree);
+
+
+/**
+ *     __sk_datagram_account_charge - send buffer for datagram protocls
+ *     @sk: socket
+ *     @size: memory size to charge
+ *     @kind: charge type
+ *
+ *     If kind is 0, it means wmem allocation. Otherwise it means rmem
+ *     allocation.
+ */
+int __sk_datagram_account_charge(struct sock *sk, int size, int kind)
+{
+       if ((kind && sk_datagram_rmem_schedule(sk, size)) ||
+           (!kind && sk_datagram_wmem_schedule(sk, size))) {
+               sk->sk_forward_alloc -= size;
+               return 1;
+       }
+       return 0;
+}
+EXPORT_SYMBOL(__sk_datagram_account_charge);
+
+/**
+ *     __sk_datagram_mem_reclaim - send buffer for datagram protocls
+ *     @sk: socket
+ */
+void __sk_datagram_mem_reclaim(struct sock *sk)
+{
+       if (sk->sk_forward_alloc < SK_DATAGRAM_MEM_QUANTUM)
+               return;
+
+       atomic_sub(sk->sk_forward_alloc / SK_DATAGRAM_MEM_QUANTUM,
+                  sk->sk_prot->memory_allocated);
+       sk->sk_forward_alloc &= SK_DATAGRAM_MEM_QUANTUM - 1;
+}
+EXPORT_SYMBOL(__sk_datagram_mem_reclaim);
+
+/**
+ *     sk_datagram_mem_schedule - memory accounting for datagram protocls
+ *     @sk: socket
+ *     @size: memory size to allocate
+ *     @kind: allocation type
+ *
+ *     If kind is 0, it means wmem allocation. Otherwise it means rmem
+ *     allocation.
+ */
+int sk_datagram_mem_schedule(struct sock *sk, int size, int kind)
+{
+       int amt;
+       struct proto *prot = sk->sk_prot;
+
+       /* Don't account and limit memory if protocol doesn't support. */
+       if (!prot->memory_allocated)
+               return 1;
+
+       amt = sk_datagram_pages(size);
+       if (atomic_add_return(amt, prot->memory_allocated) >
+           prot->sysctl_mem[0])
+               if ((kind && atomic_read(&sk->sk_rmem_alloc) + size >=
+                    prot->sysctl_rmem[0]) ||
+                   (!kind && atomic_read(&sk->sk_wmem_alloc) + size >=
+                    prot->sysctl_wmem[0])) {
+                       /* Undo changes. */
+                       atomic_sub(amt, prot->memory_allocated);
+                       return 0;
+               }
+       sk->sk_forward_alloc += amt * SK_DATAGRAM_MEM_QUANTUM;
+       return 1;
+}
+EXPORT_SYMBOL(sk_datagram_mem_schedule);
+
+/**
  *     datagram_poll - generic datagram poll
  *     @file: file struct
  *     @sock: socket
-- 
Hitachi Computer Products (America) Inc.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to