Re: [PATCH v2 1/6] utils: Implement virFileIsNamedPipe

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

 



On Tue, Oct 08, 2024 at 01:39:34PM -0500, Praveen K Paladugu wrote:
> 
> 
> On 10/1/2024 7:20 AM, Purna Pavan Chandra Aekkaladevi wrote:
> >virFileIsNamedPipe checks whether passed path is a FIFO file or not.
> >
> >Signed-off-by: Purna Pavan Chandra Aekkaladevi <paekkaladevi@xxxxxxxxxxxxxxxxxxx>
> >Co-authored-by: Vineeth Pillai <viremana@xxxxxxxxxxxxxxxxxxx>
> >---
> >  src/libvirt_private.syms | 1 +
> >  src/util/virfile.c       | 8 ++++++++
> >  src/util/virfile.h       | 1 +
> >  3 files changed, 10 insertions(+)
> >
> >diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
> >index 02dacea857..6bf6cfd20b 100644
> >--- a/src/libvirt_private.syms
> >+++ b/src/libvirt_private.syms
> >@@ -2357,6 +2357,7 @@ virFileIsDir;
> >  virFileIsExecutable;
> >  virFileIsLink;
> >  virFileIsMountPoint;
> >+virFileIsNamedPipe;
> >  virFileIsRegular;
> >  virFileIsSharedFS;
> >  virFileIsSharedFSType;
> >diff --git a/src/util/virfile.c b/src/util/virfile.c
> >index d820172405..19b074ed8a 100644
> >--- a/src/util/virfile.c
> >+++ b/src/util/virfile.c
> >@@ -2023,6 +2023,14 @@ virFileIsDir(const char *path)
> >  }
> >+bool
> >+virFileIsNamedPipe(const char *path)
> With v2, you are not using this method anywhere.

Sure, will remove it. Also, will take care of comments from other
patches as well in V3.

> >+{
> >+    struct stat s;
> >+    return (stat(path, &s) == 0) && S_ISFIFO(s.st_mode);
> >+}
> >+
> >+
> >  bool
> >  virFileIsRegular(const char *path)
> >  {
> >diff --git a/src/util/virfile.h b/src/util/virfile.h
> >index 7df3fcb840..4f7ff72483 100644
> >--- a/src/util/virfile.h
> >+++ b/src/util/virfile.h
> >@@ -215,6 +215,7 @@ void virFileActivateDirOverrideForLib(void);
> >  off_t virFileLength(const char *path, int fd) ATTRIBUTE_NONNULL(1);
> >  bool virFileIsDir (const char *file) ATTRIBUTE_NONNULL(1);
> >+bool virFileIsNamedPipe (const char *file) ATTRIBUTE_NONNULL(1);
> >  bool virFileExists(const char *file) ATTRIBUTE_NONNULL(1) G_NO_INLINE;
> >  bool virFileIsExecutable(const char *file) ATTRIBUTE_NONNULL(1);
> >  bool virFileIsRegular(const char *file) ATTRIBUTE_NONNULL(1);
> 
> -- 
> Regards,
> Praveen K Paladugu


Regards,
Pavan



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux