[Devel] [PATCH vz9] Make failslab writable again, #PSBM-141315

[Alexander Atanasov]

In (060807f841ac mm, slub: make remaining slub_debug related attributes
read-only) failslab was made read-only.
I think it became a collateral victim to the two other options for which
the reasons are perfectly valid.

Here is why:
     - sanity_checks and trace are slab internal debug options,
       failslab is used for fault injection.
     - for fault injections, which by presumption are random, it
       does not matter if it is not set atomically. And you need to
       set atleast one more option to trigger fault injection.
     - in a testing scenario you may need to change it at runtime
       example: module loading - you test all allocations limited
       by the space option. Then you move to test only your module's
       own slabs.
     - when set by command line flags it effectively disables all
       cache merges.

https://jira.sw.ru/browse/PSBM-141315
Signed-off-by: Alexander Atanasov <alexander.atanasov at virtuozzo.com>
---
 mm/slub.c | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/mm/slub.c b/mm/slub.c
index da515f247553..bb1f47f3e803 100644
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -5595,7 +5595,21 @@ static ssize_t failslab_show(struct kmem_cache *s, char *buf)
 {
 	return sysfs_emit(buf, "%d\n", !!(s->flags & SLAB_FAILSLAB));
 }
-SLAB_ATTR_RO(failslab);
+
+static ssize_t failslab_store(struct kmem_cache *s, const char *buf,
+				size_t length)
+{
+	if (s->refcount > 1)
+		return -EINVAL;
+
+	if (buf[0] == '1')
+		WRITE_ONCE(s->flags, s->flags | SLAB_FAILSLAB);
+	else
+		WRITE_ONCE(s->flags, s->flags & ~SLAB_FAILSLAB);
+
+	return length;
+}
+SLAB_ATTR(failslab);
 #endif
 
 static ssize_t shrink_show(struct kmem_cache *s, char *buf)
-- 
2.31.1