nbft: add global context as first argument to nbft API

Add the global context to the nbft related API. This allows to issue
warnings/errors via the global context.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

Author: igaw

libnvme/libnvme/nvme.i

@@ -1238,26 +1238,26 @@ struct nvme_ns {
 		return output;
 	}
 
-	PyObject *nbft_get(const char * filename)
+	PyObject *nbft_get(struct nvme_global_ctx *ctx, const char * filename)
 	{
 		struct nbft_info *nbft;
 		PyObject *output;
 		int ret;
 
-		ret = nvme_nbft_read(&nbft, filename);
+		ret = nvme_nbft_read(ctx, &nbft, filename);
 		if (ret) {
 			Py_RETURN_NONE;
 		}
 
 		output = nbft_to_pydict(nbft);
-		nvme_nbft_free(nbft);
+		nvme_nbft_free(ctx, nbft);
 		return output;
 	}
 %};
 
 %feature("autodoc", "@return an NBFT table as a dict on success, None otherwise.\n"
 		    "@param filename: file to read") nbft_get;
-PyObject *nbft_get(const char * filename);
+PyObject *nbft_get(struct nvme_global_ctx *ctx, const char * filename);
 
 // We want to swig all the #define and enum from types.h, but none of the structs.
 #pragma SWIG nowarn=503             // Supress warnings about unnamed struct

libnvme/libnvme/tests/test-nbft.py

@@ -77,7 +77,9 @@ def setUp(self):
 
     def test_read_nbft_file(self):
         """Make sure we get expected data when reading from binary NBFT file"""
-        actual_nbft = nvme.nbft_get(args.filename)
+        ctx = nvme.global_ctx()
+        ctx.log_level('debug')
+        actual_nbft = nvme.nbft_get(ctx, args.filename)
         self.assertEqual(actual_nbft, self.expected_nbft)
 
 

libnvme/src/nvme/fabrics.c

@@ -2708,7 +2708,8 @@ static int nbft_filter(const struct dirent *dent)
 	return !fnmatch(NBFT_SYSFS_FILENAME, dent->d_name, FNM_PATHNAME);
 }
 
-int nvmf_nbft_read_files(char *path, struct nbft_file_entry **head)
+int nvmf_nbft_read_files(struct nvme_global_ctx *ctx, char *path,
+		struct nbft_file_entry **head)
 {
 	struct nbft_file_entry *entry = NULL;
 	struct nbft_info *nbft;
@@ -2724,7 +2725,7 @@ int nvmf_nbft_read_files(char *path, struct nbft_file_entry **head)
 		snprintf(filename, sizeof(filename), "%s/%s", path,
 			dent[i]->d_name);
 
-		ret = nvme_nbft_read(&nbft, filename);
+		ret = nvme_nbft_read(ctx, &nbft, filename);
 		if (!ret) {
 			struct nbft_file_entry *new;
 
@@ -2746,12 +2747,12 @@ int nvmf_nbft_read_files(char *path, struct nbft_file_entry **head)
 	return 0;
 }
 
-void nvmf_nbft_free(struct nbft_file_entry *head)
+void nvmf_nbft_free(struct nvme_global_ctx *ctx, struct nbft_file_entry *head)
 {
 	while (head) {
 		struct nbft_file_entry *next = head->next;
 
-		nvme_nbft_free(head->nbft);
+		nvme_nbft_free(ctx, head->nbft);
 		free(head);
 
 		head = next;
@@ -2976,7 +2977,7 @@ int nvmf_discovery_nbft(struct nvme_global_ctx *ctx,
 		/* TODO: print discovery-type info from NBFT tables */
 		return 0;
 
-	ret = nvmf_nbft_read_files(nbft_path, &entry);
+	ret = nvmf_nbft_read_files(ctx, nbft_path, &entry);
 	if (ret) {
 		if (ret != -ENOENT)
 			nvme_msg(ctx, LOG_ERR,
@@ -3166,7 +3167,7 @@ int nvmf_discovery_nbft(struct nvme_global_ctx *ctx,
 		}
 	}
 out_free:
-	nvmf_nbft_free(entry);
+	nvmf_nbft_free(ctx, entry);
 	return ret;
 }
 

libnvme/src/nvme/fabrics.h

@@ -657,21 +657,24 @@ struct nbft_file_entry {
 
 /**
  * nvmf_nbft_read_files() - Read NBFT files from path
+ * @ctx: struct nvme_global_ctx object
  * @path: Path to NBFT files
  * @head: Pointer to store linked list of NBFT file entries
  *
  * Reads NBFT files from the specified path and populates a linked list.
  *
  * Return: 0 on success, or a negative error code on failure.
  */
-int nvmf_nbft_read_files(char *path, struct nbft_file_entry **head);
+int nvmf_nbft_read_files(struct nvme_global_ctx *ctx, char *path,
+		struct nbft_file_entry **head);
 
 /**
  * nvmf_nbft_free() - Free NBFT file entry list
+ * @ctx: struct nvme_global_ctx object
  * @head: Head of the NBFT file entry list
  *
  * Frees all memory associated with the NBFT file entry list.
  */
-void nvmf_nbft_free(struct nbft_file_entry *head);
+void nvmf_nbft_free(struct nvme_global_ctx *ctx, struct nbft_file_entry *head);
 
 #endif /* _LIBNVME_FABRICS_H */