Browse Source

Merge pull request #15235 from TDT-AG/pr/20210323-collectd-mod-ubi

collectd: add bad blocks percent calculation for ubi plugin
lilik-openwrt-22.03
Hannu Nyman 4 years ago
committed by GitHub
parent
commit
90e45ce5be
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 123 additions and 1 deletions
  1. +1
    -1
      utils/collectd/Makefile
  2. +66
    -0
      utils/collectd/patches/934-ubi-prepare-read-for-percent.patch
  3. +56
    -0
      utils/collectd/patches/935-ubi-add-percent.patch

+ 1
- 1
utils/collectd/Makefile View File

@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
PKG_NAME:=collectd
PKG_VERSION:=5.12.0
PKG_RELEASE:=8
PKG_RELEASE:=9
PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.bz2
PKG_SOURCE_URL:=https://collectd.org/files/ \


+ 66
- 0
utils/collectd/patches/934-ubi-prepare-read-for-percent.patch View File

@ -0,0 +1,66 @@
--- a/src/ubi.c
+++ b/src/ubi.c
@@ -84,9 +84,8 @@ static void ubi_submit(const char *dev_n
plugin_dispatch_values(&vl);
} /* void ubi_submit */
-static int ubi_read_dev_attr(const char *dev_name, const char *attr) {
+static int ubi_read_dev_attr(const char *dev_name, const char *attr, int *value) {
FILE *f;
- int val;
char
str[sizeof(SYS_PATH) + strlen(dev_name) + sizeof("/") + strlen(attr) + 1];
int n;
@@ -98,7 +97,7 @@ static int ubi_read_dev_attr(const char
return -1;
}
- n = fscanf(f, "%d", &val);
+ n = fscanf(f, "%d", value);
fclose(f);
if (n != 1) {
@@ -106,17 +105,39 @@ static int ubi_read_dev_attr(const char
return -1;
}
- ubi_submit(dev_name, attr, (gauge_t)val);
-
return 0;
} /* int ubi_read_dev_attr */
static inline int ubi_read_dev_bad_count(const char *dev_name) {
- return ubi_read_dev_attr(dev_name, DEV_BAD_COUNT);
+ int ret;
+ int value;
+
+ ret = ubi_read_dev_attr(dev_name, DEV_BAD_COUNT, &value);
+
+ if (ret != 0) {
+ ERROR(PLUGIN_NAME " : Unable to read bat_peb_count");
+ return -1;
+ }
+
+ ubi_submit(dev_name, DEV_BAD_COUNT, (gauge_t)value);
+
+ return 0;
} /* int ubi_read_dev_bad_count */
static inline int ubi_read_max_ec(const char *dev_name) {
- return ubi_read_dev_attr(dev_name, MAXIMUM_ERASE);
+ int ret;
+ int value;
+
+ ret = ubi_read_dev_attr(dev_name, MAXIMUM_ERASE, &value);
+
+ if (ret != 0) {
+ ERROR(PLUGIN_NAME " : Unable to read max_ec");
+ return -1;
+ }
+
+ ubi_submit(dev_name, MAXIMUM_ERASE, (gauge_t)value);
+
+ return 0;
} /* int ubi_read_max_ec */
static int ubi_read(void) {

+ 56
- 0
utils/collectd/patches/935-ubi-add-percent.patch View File

@ -0,0 +1,56 @@
--- a/src/ubi.c
+++ b/src/ubi.c
@@ -35,6 +35,9 @@
#define DEV_BAD_COUNT \
"bad_peb_count" // Count of bad physical eraseblocks on the underlying MTD
// device.
+// Value reserved for bad block
+#define DEV_RESERVED_BAD_BLOCK "reserved_for_bad"
+
#define MAXIMUM_ERASE "max_ec" // Current maximum erase counter value
/*
@@ -140,6 +143,35 @@ static inline int ubi_read_max_ec(const
return 0;
} /* int ubi_read_max_ec */
+static inline int ubi_read_percent(const char *dev_name) {
+ int ret;
+ int bcount;
+ int bblock;
+
+ ret = ubi_read_dev_attr(dev_name, DEV_BAD_COUNT, &bcount);
+
+ if (ret != 0) {
+ ERROR(PLUGIN_NAME " : Unable to read bad_peb_count");
+ return -1;
+ }
+
+ ret = ubi_read_dev_attr(dev_name, DEV_RESERVED_BAD_BLOCK, &bblock);
+
+ if (ret != 0) {
+ ERROR(PLUGIN_NAME " : Unable to read reserved_for_bad");
+ return -1;
+ }
+
+ if (bblock == 0) {
+ ERROR(PLUGIN_NAME " : Percentage value cannot be determined (reserved_for_bad = 0)");
+ return -2;
+ }
+
+ ubi_submit(dev_name, "percent", (gauge_t)((float_t)bcount / (float_t)bblock * 100.0));
+
+ return 0;
+} /* int ubi_read_percent */
+
static int ubi_read(void) {
DIR *dir;
struct dirent *dirent;
@@ -155,6 +187,7 @@ static int ubi_read(void) {
ubi_read_dev_bad_count(dirent->d_name);
ubi_read_max_ec(dirent->d_name);
+ ubi_read_percent(dirent->d_name);
}
closedir(dir);

Loading…
Cancel
Save