xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCHv3] 1/3] libxc: prefer using privcmd character device
@ 2015-12-01 19:27 Doug Goldstein
  2015-12-01 19:27 ` [PATCHv3] 2/3] update outdated header comment on privcmd.h Doug Goldstein
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Doug Goldstein @ 2015-12-01 19:27 UTC (permalink / raw)
  To: xen-devel
  Cc: Ian Jackson, Wei Liu, Doug Goldstein, Ian Campbell,
	Stefano Stabellini

Prefer using the character device over the proc file if the character
device exists. This follows similar conversions of xenbus to avoid
issues with FMODE_ATOMIC_POS added in Linux 3.14 and newer.

CC: Ian Jackson <ian.jackson@eu.citrix.com>
CC: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
CC: Ian Campbell <ian.campbell@citrix.com>
CC: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
---
 tools/libxc/xc_linux_osdep.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tools/libxc/xc_linux_osdep.c b/tools/libxc/xc_linux_osdep.c
index 76c55ff..c3a3a14 100644
--- a/tools/libxc/xc_linux_osdep.c
+++ b/tools/libxc/xc_linux_osdep.c
@@ -46,7 +46,13 @@
 static xc_osdep_handle linux_privcmd_open(xc_interface *xch)
 {
     int flags, saved_errno;
-    int fd = open("/proc/xen/privcmd", O_RDWR);
+    int fd = open("/dev/xen/privcmd", O_RDWR); /* prefer this newer interface */
+
+    if ( fd == -1 && ( errno == ENOENT || errno == ENXIO || errno == ENODEV ))
+    {
+        /* Fallback to /proc/xen/privcmd */
+        fd = open("/proc/xen/privcmd", O_RDWR);
+    }
 
     if ( fd == -1 )
     {
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-01-27 14:45 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-01 19:27 [PATCHv3] 1/3] libxc: prefer using privcmd character device Doug Goldstein
2015-12-01 19:27 ` [PATCHv3] 2/3] update outdated header comment on privcmd.h Doug Goldstein
2015-12-01 19:27 ` [PATCHv3] 3/3] xendomains initscript: test for privcmd char device Doug Goldstein
2015-12-02 11:42 ` [PATCHv3] 1/3] libxc: prefer using privcmd character device Wei Liu
2015-12-02 15:52   ` Ian Campbell
2016-01-27 14:40 ` Doug Goldstein
2016-01-27 14:45   ` Processed: " xen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).