Re: [PATCH 1/1] utils: Fix left debug info

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hey!

This patch is pretty messed up.

On 1/22/22 13:02, Petr Vorel wrote:
Patch for 497ffdf8 ('manpage: remove the no longer supported value
"vers2"') [1] didn't contain printf in exportfs.c (looks like debugging
info) and errno handling in stropts.c (maybe work on other patch) which
were applied. Thus removing it.
Someone pointed it to me that  with 2 v3 auto-negotiation
on the same mount the is error EPROTONOSUPPORT
instead of EBUSY so this test
     if (errno != EBUSY)
	errno = olderrno;

seemed to work but unfortunately I can
not find the patch/bz or thread we were
communicating in... So I am going to
remove the test until I get (or find) the
official patch

Committed!

thanks,

steved.


[1] https://lore.kernel.org/linux-nfs/20220117031356.13361-1-yoyang@xxxxxxxxxx/raw

Signed-off-by: Petr Vorel <pvorel@xxxxxxx>
---
  utils/exportfs/exportfs.c | 2 --
  utils/mount/stropts.c     | 3 +--
  2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/utils/exportfs/exportfs.c b/utils/exportfs/exportfs.c
index c247425b..6ba615d1 100644
--- a/utils/exportfs/exportfs.c
+++ b/utils/exportfs/exportfs.c
@@ -307,14 +307,12 @@ static int exportfs_generic(char *arg, char *options, int verbose)
  {
  	char *path;
-printf("exportfs_generic: arg '%s'\n", arg);
  	if ((path = strchr(arg, ':')) != NULL)
  		*path++ = '\0';
if (!path || *path != '/')
  		return 1;
-printf("exportfs_generic: path '%s'\n", path);
  	exportfs_parsed(arg, path, options, verbose);
  	return 0;
  }
diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c
index 3ca69862..3c4e218a 100644
--- a/utils/mount/stropts.c
+++ b/utils/mount/stropts.c
@@ -973,8 +973,7 @@ fall_back:
  	if ((result = nfs_try_mount_v3v2(mi, FALSE)))
  		return result;
- if (errno != EBUSY)
-		errno = olderrno;
+	errno = olderrno;
  	return result;
  }




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux