[PATCH v5 1/2] lib/string.c: add __sysfs_match_string_with_gaps() helper

* [PATCH v5 1/2] lib/string.c: add __sysfs_match_string_with_gaps() helper
@ 2020-12-22  9:52 Alexandru Ardelean
  2020-12-22  9:52 ` [PATCH v5 2/2] iio: Handle enumerated properties with gaps Alexandru Ardelean
  2020-12-22 11:12 ` [PATCH v5 1/2] lib/string.c: add __sysfs_match_string_with_gaps() helper Greg KH
  0 siblings, 2 replies; 4+ messages in thread
From: Alexandru Ardelean @ 2020-12-22  9:52 UTC (permalink / raw)
  To: linux-iio, linux-kernel
  Cc: jic23, lars, akpm, gregkh, andy.shevchenko, Alexandru Ardelean

The original docstring of the __sysfs_match_string() and match_string()
helper, implied that -1 could be used to search through NULL terminated
arrays, and positive 'n' can be used to go through arrays that may have
NULL elements in the middle of the array.

This isn't true. Regardless of the value of 'n', the first NULL element in
the array.

To allow for a behavior where we can use the __sysfs_match_string() to
search over arrays with NULL elements in the middle, the
__sysfs_match_string_with_gaps() helper is added.
If n > 0, the search will continue until the element is found or n is
reached.
If n < 0, the search will continue until the element is found or a NULL
character is found.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
---

Changelog v4 -> v5:
* https://patchwork.kernel.org/project/linux-iio/patch/20201221151551.52511-1-alexandru.ardelean@analog.com/
* add 'lib/string.c: add __sysfs_match_string_with_gaps() helper' patch

The main intent here is to implement the
'iio: Handle enumerated properties with gaps' patch.

The original __sysfs_match_string() helper stops at the first NULL
element regardless of whether 'n' is -1 or not.
That behavior is documented in the docstring.
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next-history.git/commit/lib/string.c?id=c11d3fa0116a6bc832a9e387427caa16f8de5ef2

However, a __sysfs_match_string_with_gaps() helper can still be useful.
Trying to fix the behavior of __sysfs_match_string() is a bit
difficult/slow and can introduce issues.

Some older references on this:
https://lore.kernel.org/linux-iio/20190422140251.8960-1-alexandru.ardelean@analog.com/
https://lore.kernel.org/lkml/20200212144723.21884-1-alexandru.ardelean@analog.com/
https://lore.kernel.org/lkml/20190508111913.7276-1-alexandru.ardelean@analog.com/
https://lore.kernel.org/lkml/20190625130104.29904-1-alexandru.ardelean@analog.com/
https://lore.kernel.org/lkml/20190422083257.21805-1-alexandru.ardelean@analog.com/
  
 include/linux/string.h |  1 +
 lib/string.c           | 55 +++++++++++++++++++++++++++++++++---------
 2 files changed, 44 insertions(+), 12 deletions(-)

diff --git a/include/linux/string.h b/include/linux/string.h
index b1f3894a0a3e..d7999e50dae1 100644
--- a/include/linux/string.h
+++ b/include/linux/string.h
@@ -191,6 +191,7 @@ static inline int strtobool(const char *s, bool *res)
 
 int match_string(const char * const *array, size_t n, const char *string);
 int __sysfs_match_string(const char * const *array, size_t n, const char *s);
+int __sysfs_match_string_with_gaps(const char * const *array, ssize_t n, const char *s);
 
 /**
  * sysfs_match_string - matches given string in an array
diff --git a/lib/string.c b/lib/string.c
index 4288e0158d47..3dbd362f28a2 100644
--- a/lib/string.c
+++ b/lib/string.c
@@ -751,6 +751,26 @@ int match_string(const char * const *array, size_t n, const char *string)
 }
 EXPORT_SYMBOL(match_string);
 
+static int __sysfs_match_string_common(const char * const *array, ssize_t n,
+				       const char *str, bool gaps)
+{
+	const char *item;
+	int index;
+
+	for (index = 0; index < n; index++) {
+		item = array[index];
+		if (!item) {
+			if (gaps && n > 0)
+				continue;
+			break;
+		}
+		if (sysfs_streq(item, str))
+			return index;
+	}
+
+	return -EINVAL;
+}
+
 /**
  * __sysfs_match_string - matches given string in an array
  * @array: array of strings
@@ -770,21 +790,32 @@ EXPORT_SYMBOL(match_string);
  */
 int __sysfs_match_string(const char * const *array, size_t n, const char *str)
 {
-	const char *item;
-	int index;
-
-	for (index = 0; index < n; index++) {
-		item = array[index];
-		if (!item)
-			break;
-		if (sysfs_streq(item, str))
-			return index;
-	}
-
-	return -EINVAL;
+	return __sysfs_match_string_common(array, n, str, false);
 }
 EXPORT_SYMBOL(__sysfs_match_string);
 
+/**
+ * __sysfs_match_string_with_gaps - matches given string in an array with gaps
+ * @array: array of strings
+ * @n: number of strings in the array or negative for NULL terminated arrays
+ * @str: string to match with
+ *
+ * Returns index of @str in the @array or -EINVAL, similar to match_string().
+ * Uses sysfs_streq instead of strcmp for matching.
+ *
+ * This routine will look for a string in an array of strings.
+ * If n > 0, then the search will continue until the element is found or
+ * the n-th element is reached, regardless of any NULL elements in the array.
+ * If n < 0, then the search will continue until the element is found or
+ * util the first NULL element.
+ */
+int __sysfs_match_string_with_gaps(const char * const *array, ssize_t n,
+				   const char *str)
+{
+	return __sysfs_match_string_common(array, n, str, true);
+}
+EXPORT_SYMBOL(__sysfs_match_string_with_gaps);
+
 #ifndef __HAVE_ARCH_MEMSET
 /**
  * memset - Fill a region of memory with the given value
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread