[Bloat] Fwd: [RFC net-next 2/2] ipv6: ioam: Support for Buffer occupancy data field

[Bloat] Fwd: [RFC net-next 2/2] ipv6: ioam: Support for Buffer occupancy data field

[Dave Taht]

interesting thread, go respond there, tho.

---------- Forwarded message ---------
From: Justin Iurman <justin.iurman@uliege.be>
Date: Mon, Dec 6, 2021 at 9:56 PM
Subject: [RFC net-next 2/2] ipv6: ioam: Support for Buffer occupancy data field
To: <netdev@vger.kernel.org>
Cc: <davem@davemloft.net>, <kuba@kernel.org>, <dsahern@kernel.org>,
<yoshfuji@linux-ipv6.org>, <linux-mm@kvack.org>, <cl@linux.com>,
<penberg@kernel.org>, <rientjes@google.com>, <iamjoonsoo.kim@lge.com>,
<akpm@linux-foundation.org>, <vbabka@suse.cz>,
<justin.iurman@uliege.be>


This patch is an attempt to support the buffer occupancy in IOAM trace
data fields. Any feedback is appreciated, or any other idea if this one
is not correct.

The draft [1] says the following:

   The "buffer occupancy" field is a 4-octet unsigned integer field.
   This field indicates the current status of the occupancy of the
   common buffer pool used by a set of queues.  The units of this field
   are implementation specific.  Hence, the units are interpreted within
   the context of an IOAM-Namespace and/or node-id if used.  The authors
   acknowledge that in some operational cases there is a need for the
   units to be consistent across a packet path through the network,
   hence it is recommended for implementations to use standard units
   such as Bytes.

An existing function (i.e., get_slabinfo) is used to retrieve info about
skbuff_head_cache. For that, both the prototype of get_slabinfo and
struct definition of slabinfo were moved from mm/slab.h to
include/linux/slab.h. Any objection on this?

The function kmem_cache_size is used to retrieve the size of a slab
object. Note that it returns the "object_size" field, not the "size"
field. If needed, a new function (e.g., kmem_cache_full_size) could be
added to return the "size" field. To match the definition from the
draft, the number of bytes is computed as follows:

slabinfo.active_objs * size

Thoughts?

  [1] https://datatracker.ietf.org/doc/html/draft-ietf-ippm-ioam-data#section-5.4.2.12

Signed-off-by: Justin Iurman <justin.iurman@uliege.be>
---
 include/linux/slab.h | 15 +++++++++++++++
 mm/slab.h            | 14 --------------
 net/ipv6/ioam6.c     | 13 ++++++++++++-
 3 files changed, 27 insertions(+), 15 deletions(-)

diff --git a/include/linux/slab.h b/include/linux/slab.h
index 181045148b06..745790dbcbcb 100644
--- a/include/linux/slab.h
+++ b/include/linux/slab.h
@@ -19,6 +19,21 @@
 #include <linux/percpu-refcount.h>


+struct slabinfo {
+       unsigned long active_objs;
+       unsigned long num_objs;
+       unsigned long active_slabs;
+       unsigned long num_slabs;
+       unsigned long shared_avail;
+       unsigned int limit;
+       unsigned int batchcount;
+       unsigned int shared;
+       unsigned int objects_per_slab;
+       unsigned int cache_order;
+};
+
+void get_slabinfo(struct kmem_cache *s, struct slabinfo *sinfo);
+
 /*
  * Flags to pass to kmem_cache_create().
  * The ones marked DEBUG are only valid if CONFIG_DEBUG_SLAB is set.
diff --git a/mm/slab.h b/mm/slab.h
index 56ad7eea3ddf..cd6a8a2768e3 100644
--- a/mm/slab.h
+++ b/mm/slab.h
@@ -175,20 +175,6 @@ void slab_kmem_cache_release(struct kmem_cache *);
 struct seq_file;
 struct file;

-struct slabinfo {
-       unsigned long active_objs;
-       unsigned long num_objs;
-       unsigned long active_slabs;
-       unsigned long num_slabs;
-       unsigned long shared_avail;
-       unsigned int limit;
-       unsigned int batchcount;
-       unsigned int shared;
-       unsigned int objects_per_slab;
-       unsigned int cache_order;
-};
-
-void get_slabinfo(struct kmem_cache *s, struct slabinfo *sinfo);
 void slabinfo_show_stats(struct seq_file *m, struct kmem_cache *s);
 ssize_t slabinfo_write(struct file *file, const char __user *buffer,
                       size_t count, loff_t *ppos);
diff --git a/net/ipv6/ioam6.c b/net/ipv6/ioam6.c
index 088eb2f877bc..f0a44dc2a0df 100644
--- a/net/ipv6/ioam6.c
+++ b/net/ipv6/ioam6.c
@@ -14,6 +14,8 @@
 #include <linux/ioam6_genl.h>
 #include <linux/rhashtable.h>
 #include <linux/netdevice.h>
+#include <linux/slab.h>
+#include <linux/skbuff.h>

 #include <net/addrconf.h>
 #include <net/genetlink.h>
@@ -778,7 +780,16 @@ static void __ioam6_fill_trace_data(struct sk_buff *skb,

        /* buffer occupancy */
        if (trace->type.bit11) {
-               *(__be32 *)data = cpu_to_be32(IOAM6_U32_UNAVAILABLE);
+               struct slabinfo sinfo;
+               u32 size, bytes;
+
+               sinfo.active_objs = 0;
+               get_slabinfo(skbuff_head_cache, &sinfo);
+               size = kmem_cache_size(skbuff_head_cache);
+
+               bytes = min(sinfo.active_objs * size, (unsigned
long)(U32_MAX-1));
+
+               *(__be32 *)data = cpu_to_be32(bytes);
                data += sizeof(__be32);
        }

--
2.25.1



-- 
I tried to build a better future, a few times:
https://wayforward.archive.org/?site=https%3A%2F%2Fwww.icei.org

Dave Täht CEO, TekLibre, LLC