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

[xen master] tools/xenstore: replace watch->relative_path with a prefix length



commit 4db46b04956272f11a588e183653a13c2dd2e422
Author:     Juergen Gross <jgross@xxxxxxxx>
AuthorDate: Wed Jan 18 10:50:05 2023 +0100
Commit:     Julien Grall <jgrall@xxxxxxxxxx>
CommitDate: Fri Jan 20 09:23:51 2023 +0000

    tools/xenstore: replace watch->relative_path with a prefix length
    
    Instead of storing a pointer to the path which is prepended to
    relative paths in struct watch, just use the length of the prepended
    path.
    
    It should be noted that the now removed special case of the
    relative path being "" in get_watch_path() can't happen at all.
    
    Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
    Reviewed-by: Julien Grall <jgrall@xxxxxxxxxx>
---
 tools/xenstore/xenstored_watch.c | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/tools/xenstore/xenstored_watch.c b/tools/xenstore/xenstored_watch.c
index 75748ac109..8ad0229df6 100644
--- a/tools/xenstore/xenstored_watch.c
+++ b/tools/xenstore/xenstored_watch.c
@@ -39,8 +39,8 @@ struct watch
        /* Current outstanding events applying to this watch. */
        struct list_head events;
 
-       /* Is this relative to connnection's implicit path? */
-       const char *relative_path;
+       /* Offset into path for skipping prefix (used for relative paths). */
+       unsigned int prefix_len;
 
        char *token;
        char *node;
@@ -66,15 +66,7 @@ static bool is_child(const char *child, const char *parent)
 
 static const char *get_watch_path(const struct watch *watch, const char *name)
 {
-       const char *path = name;
-
-       if (watch->relative_path) {
-               path += strlen(watch->relative_path);
-               if (*path == '/') /* Could be "" */
-                       path++;
-       }
-
-       return path;
+       return name + watch->prefix_len;
 }
 
 /*
@@ -211,10 +203,7 @@ static struct watch *add_watch(struct connection *conn, 
char *path, char *token,
                              no_quota_check))
                goto nomem;
 
-       if (relative)
-               watch->relative_path = get_implicit_path(conn);
-       else
-               watch->relative_path = NULL;
+       watch->prefix_len = relative ? strlen(get_implicit_path(conn)) + 1 : 0;
 
        INIT_LIST_HEAD(&watch->events);
 
--
generated by git-patchbot for /home/xen/git/xen.git#master



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.