diff mbox series

[ndctl,trivial] test: Remove a redundant ndctl_namespace_foreach

Message ID 20200806124702.305084-1-santosh@fossix.org (mailing list archive)
State Accepted
Commit b2d9280a2ebbe79a62225b643420fb1549d69dc7
Headers show
Series [ndctl,trivial] test: Remove a redundant ndctl_namespace_foreach | expand

Commit Message

Santosh Sivaraj Aug. 6, 2020, 12:47 p.m. UTC
I don't think this was intended to be in the code.

Signed-off-by: Santosh Sivaraj <santosh@fossix.org>
---
 test/multi-pmem.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Ira Weiny Aug. 7, 2020, 6:33 p.m. UTC | #1
On Thu, Aug 06, 2020 at 06:17:02PM +0530, Santosh Sivaraj wrote:
> I don't think this was intended to be in the code.
> 
> Signed-off-by: Santosh Sivaraj <santosh@fossix.org>

Reviewed-by: Ira Weiny <ira.weiny@intel.com>

> ---
>  test/multi-pmem.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/test/multi-pmem.c b/test/multi-pmem.c
> index 668662c..cb7cd40 100644
> --- a/test/multi-pmem.c
> +++ b/test/multi-pmem.c
> @@ -162,7 +162,6 @@ static int do_multi_pmem(struct ndctl_ctx *ctx, struct ndctl_test *test)
>  		char uuid_str1[40], uuid_str2[40];
>  		uuid_t uuid_check;
>  
> -		ndctl_namespace_foreach(region, ndns)
>  		sprintf(devname, "namespace%d.%d",
>  				ndctl_region_get_id(region), i);
>  		ndctl_namespace_foreach(region, ndns)
> -- 
> 2.26.2
> _______________________________________________
> Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org
> To unsubscribe send an email to linux-nvdimm-leave@lists.01.org
diff mbox series

Patch

diff --git a/test/multi-pmem.c b/test/multi-pmem.c
index 668662c..cb7cd40 100644
--- a/test/multi-pmem.c
+++ b/test/multi-pmem.c
@@ -162,7 +162,6 @@  static int do_multi_pmem(struct ndctl_ctx *ctx, struct ndctl_test *test)
 		char uuid_str1[40], uuid_str2[40];
 		uuid_t uuid_check;
 
-		ndctl_namespace_foreach(region, ndns)
 		sprintf(devname, "namespace%d.%d",
 				ndctl_region_get_id(region), i);
 		ndctl_namespace_foreach(region, ndns)