The per-target token bucket has no interface, so every target is still
unlimited.

Expose the interval as ratelimit_interval_ms through configfs. It
defaults to zero, so a target stays unlimited until an administrator
sets one.

Signed-off-by: Breno Leitao <[email protected]>
---
 drivers/net/netconsole.c | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c
index 2ad514afa2b89..6f94495d0abc3 100644
--- a/drivers/net/netconsole.c
+++ b/drivers/net/netconsole.c
@@ -940,6 +940,14 @@ static ssize_t transmit_errors_show(struct config_item 
*item, char *buf)
        return sysfs_emit(buf, "%llu\n", xmit_drop_count + enomem_count);
 }
 
+static ssize_t ratelimit_interval_ms_show(struct config_item *item, char *buf)
+{
+       struct netconsole_target *nt = to_target(item);
+
+       return sysfs_emit(buf, "%u\n",
+                         jiffies_to_msecs(READ_ONCE(nt->ratelimit.interval)));
+}
+
 /* configfs helper to display if cpu_nr sysdata feature is enabled */
 static ssize_t sysdata_cpu_nr_enabled_show(struct config_item *item, char *buf)
 {
@@ -1335,6 +1343,29 @@ static ssize_t remote_mac_store(struct config_item 
*item, const char *buf,
        return ret;
 }
 
+static ssize_t ratelimit_interval_ms_store(struct config_item *item,
+                                          const char *buf, size_t count)
+{
+       struct netconsole_target *nt = to_target(item);
+       unsigned int interval;
+       unsigned long jifs;
+       ssize_t ret;
+
+       ret = kstrtouint(buf, 10, &interval);
+       if (ret)
+               return ret;
+
+       jifs = msecs_to_jiffies(interval);
+       if (jifs > INT_MAX)
+               return -ERANGE;
+
+       dynamic_netconsole_mutex_lock();
+       WRITE_ONCE(nt->ratelimit.interval, jifs);
+       dynamic_netconsole_mutex_unlock();
+
+       return count;
+}
+
 struct userdatum {
        struct config_item item;
        char value[MAX_EXTRADATA_VALUE_LEN];
@@ -1699,6 +1730,7 @@ CONFIGFS_ATTR_RO(, local_mac);
 CONFIGFS_ATTR(, remote_mac);
 CONFIGFS_ATTR(, release);
 CONFIGFS_ATTR_RO(, transmit_errors);
+CONFIGFS_ATTR(, ratelimit_interval_ms);
 
 static struct configfs_attribute *netconsole_target_attrs[] = {
        &attr_enabled,
@@ -1712,6 +1744,7 @@ static struct configfs_attribute 
*netconsole_target_attrs[] = {
        &attr_local_mac,
        &attr_remote_mac,
        &attr_transmit_errors,
+       &attr_ratelimit_interval_ms,
        NULL,
 };
 

-- 
2.53.0-Meta


Reply via email to