[Qemu-commits] [qemu/qemu] 82c147: virtiofsd: Help message fix for 'seconds'

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 82c1474e687f8d30cf58af611090cda5e4ec5e0e
      
https://github.com/qemu/qemu/commit/82c1474e687f8d30cf58af611090cda5e4ec5e0e
  Author: Dr. David Alan Gilbert <address@hidden>
  Date:   2020-02-21 (Fri, 21 Feb 2020)

  Changed paths:
    M tools/virtiofsd/helper.c

  Log Message:
  -----------
  virtiofsd: Help message fix for 'seconds'

second should be seconds.

Reported-by: Christophe de Dinechin <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>


  Commit: d4db6f545decdf67be8c4d6ca2bfb5dab99e462b
      
https://github.com/qemu/qemu/commit/d4db6f545decdf67be8c4d6ca2bfb5dab99e462b
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2020-02-21 (Fri, 21 Feb 2020)

  Changed paths:
    M tools/virtiofsd/passthrough_ll.c

  Log Message:
  -----------
  tools/virtiofsd/passthrough_ll: Remove unneeded variable assignment

Fix warning reported by Clang static code analyzer:

    CC      tools/virtiofsd/passthrough_ll.o
  tools/virtiofsd/passthrough_ll.c:1083:5: warning: Value stored to 'saverr' is 
never read
      saverr = ENOMEM;
      ^        ~~~~~~

Fixes: 7c6b66027
Reported-by: Clang Static Analyzer
Reviewed-by: Ján Tomko <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 4e1fb9e7bc4662470c644be7efec8d8ce4005863
      
https://github.com/qemu/qemu/commit/4e1fb9e7bc4662470c644be7efec8d8ce4005863
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2020-02-21 (Fri, 21 Feb 2020)

  Changed paths:
    M tools/virtiofsd/passthrough_ll.c

  Log Message:
  -----------
  tools/virtiofsd/passthrough_ll: Remove unneeded variable assignment

Fix warning reported by Clang static code analyzer:

    CC      tools/virtiofsd/passthrough_ll.o
  tools/virtiofsd/passthrough_ll.c:925:9: warning: Value stored to 'newfd' is 
never read
          newfd = -1;
          ^       ~~
  tools/virtiofsd/passthrough_ll.c:942:9: warning: Value stored to 'newfd' is 
never read
          newfd = -1;
          ^       ~~

Fixes: 7c6b66027
Reported-by: Clang Static Analyzer
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Ján Tomko <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 09c086b2a144324199f99a7d4de78c3276a486c1
      
https://github.com/qemu/qemu/commit/09c086b2a144324199f99a7d4de78c3276a486c1
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2020-02-21 (Fri, 21 Feb 2020)

  Changed paths:
    M tools/virtiofsd/fuse_lowlevel.c

  Log Message:
  -----------
  tools/virtiofsd/fuse_lowlevel: Fix fuse_out_header::error value

Fix warning reported by Clang static code analyzer:

    CC      tools/virtiofsd/fuse_lowlevel.o
  tools/virtiofsd/fuse_lowlevel.c:195:9: warning: Value stored to 'error' is 
never read
          error = -ERANGE;
          ^       ~~~~~~~

Fixes: 3db2876
Reported-by: Clang Static Analyzer
Reviewed-by: Ján Tomko <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 285eb7a704e6d26d687cdcd578dba74fbee77c61
      
https://github.com/qemu/qemu/commit/285eb7a704e6d26d687cdcd578dba74fbee77c61
  Author: Xiao Yang <address@hidden>
  Date:   2020-02-21 (Fri, 21 Feb 2020)

  Changed paths:
    R tools/virtiofsd/fuse.h
    M tools/virtiofsd/fuse_i.h

  Log Message:
  -----------
  virtiofsd: Remove fuse.h and struct fuse_module

All code in fuse.h and struct fuse_module are not used by virtiofsd
so removing them is safe.

Signed-off-by: Xiao Yang <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 5bb8e8beedb47fc0d0a44957a154918c4f4afc80
      
https://github.com/qemu/qemu/commit/5bb8e8beedb47fc0d0a44957a154918c4f4afc80
  Author: Miroslav Rezanina <address@hidden>
  Date:   2020-02-21 (Fri, 21 Feb 2020)

  Changed paths:
    M Makefile

  Log Message:
  -----------
  docs: Fix virtiofsd.1 location

Patch 6a7e2bbee5 docs: add virtiofsd(1) man page introduced new man
page virtiofsd.1. Unfortunately, wrong file location is used as
source for install command. This cause installation of docs fail.

Fixing wrong location so installation is successful.

Signed-off-by: Miroslav Rezanina <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>


  Commit: 88e2b97aa3e369a454c9d8360afddc348070c708
      
https://github.com/qemu/qemu/commit/88e2b97aa3e369a454c9d8360afddc348070c708
  Author: Peter Maydell <address@hidden>
  Date:   2020-02-21 (Fri, 21 Feb 2020)

  Changed paths:
    M Makefile
    R tools/virtiofsd/fuse.h
    M tools/virtiofsd/fuse_i.h
    M tools/virtiofsd/fuse_lowlevel.c
    M tools/virtiofsd/helper.c
    M tools/virtiofsd/passthrough_ll.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/dgilbert-gitlab/tags/pull-virtiofs-20200221' into staging

virtiofs pull 20200221

Mostly minor cleanups.
Miroslav's fixes a make install corner case.
Philippe's set includes an error corner case fix.

# gpg: Signature made Fri 21 Feb 2020 13:21:39 GMT
# gpg:                using RSA key 45F5C71B4A0CB7FB977A9FA90516331EBC5BFDE7
# gpg: Good signature from "Dr. David Alan Gilbert (RH2) <address@hidden>" 
[full]
# Primary key fingerprint: 45F5 C71B 4A0C B7FB 977A  9FA9 0516 331E BC5B FDE7

* remotes/dgilbert-gitlab/tags/pull-virtiofs-20200221:
  docs: Fix virtiofsd.1 location
  virtiofsd: Remove fuse.h and struct fuse_module
  tools/virtiofsd/fuse_lowlevel: Fix fuse_out_header::error value
  tools/virtiofsd/passthrough_ll: Remove unneeded variable assignment
  tools/virtiofsd/passthrough_ll: Remove unneeded variable assignment
  virtiofsd: Help message fix for 'seconds'

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/9ac5df20f51f...88e2b97aa3e3