[PATCH 0/7] Clean up virFileIsSharedFSType

Peter Krempa posted 7 patches 1 year, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1662384799.git.pkrempa@redhat.com
src/util/virfile.c | 139 +++++++++++++++++++++------------------------
src/util/virfile.h |  19 ++++---
2 files changed, 76 insertions(+), 82 deletions(-)
[PATCH 0/7] Clean up virFileIsSharedFSType
Posted by Peter Krempa 1 year, 8 months ago
Few cleanups for problems I've noticed when I had to understand how the
shared FS detection is supposed to work.

Peter Krempa (7):
  util: virFileIsSharedFSType: Pass bitmap of checked fs types as
    unsigned
  util: virFileIsSharedFSType: Annotate (some) shared filesystem names
  util: virFileIsSharedFixFUSE: Refactor cleanup
  util: virfile: Rewrite matching of FUSE-based shared filesystems
  util: virfile: Don't use VIR_FILE_SHFS_GFS2 for glusterfs
  util: virfile: Drop QB_MAGIC constant
  util: virFileIsSharedFSType: Simplify shared fs type declarations

 src/util/virfile.c | 139 +++++++++++++++++++++------------------------
 src/util/virfile.h |  19 ++++---
 2 files changed, 76 insertions(+), 82 deletions(-)

-- 
2.37.1
Re: [PATCH 0/7] Clean up virFileIsSharedFSType
Posted by Ján Tomko 1 year, 8 months ago
On a Monday in 2022, Peter Krempa wrote:
>Few cleanups for problems I've noticed when I had to understand how the
>shared FS detection is supposed to work.
>
>Peter Krempa (7):
>  util: virFileIsSharedFSType: Pass bitmap of checked fs types as
>    unsigned
>  util: virFileIsSharedFSType: Annotate (some) shared filesystem names
>  util: virFileIsSharedFixFUSE: Refactor cleanup
>  util: virfile: Rewrite matching of FUSE-based shared filesystems
>  util: virfile: Don't use VIR_FILE_SHFS_GFS2 for glusterfs
>  util: virfile: Drop QB_MAGIC constant
>  util: virFileIsSharedFSType: Simplify shared fs type declarations
>
> src/util/virfile.c | 139 +++++++++++++++++++++------------------------
> src/util/virfile.h |  19 ++++---
> 2 files changed, 76 insertions(+), 82 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano