xen-netback: add reference from xenvif to backend_info to facilitate coredump analysis
authorDongli Zhang <dongli.zhang@oracle.com>
Fri, 12 Apr 2019 06:53:24 +0000 (14:53 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 12 Apr 2019 17:10:28 +0000 (10:10 -0700)
During coredump analysis, it is not easy to obtain the address of
backend_info in xen-netback.

So far there are two ways to obtain backend_info:

1. Do what xenbus_device_find() does for vmcore to find the xenbus_device
and then derive it from dev_get_drvdata().

2. Extract backend_info from callstack of xenwatch (e.g., netback_remove()
or frontend_changed()).

This patch adds a reference from xenvif to backend_info so that it would be
much more easier to obtain backend_info during coredump analysis.

Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/xen-netback/common.h
drivers/net/xen-netback/xenbus.c

index 936c0b3e0ba28ec1f6586a5bab86d403a86d0dd6..05847eb91a1b4a24c29567384d0a1b7094f56235 100644 (file)
@@ -248,6 +248,22 @@ struct xenvif_hash {
        struct xenvif_hash_cache cache;
 };
 
+struct backend_info {
+       struct xenbus_device *dev;
+       struct xenvif *vif;
+
+       /* This is the state that will be reflected in xenstore when any
+        * active hotplug script completes.
+        */
+       enum xenbus_state state;
+
+       enum xenbus_state frontend_state;
+       struct xenbus_watch hotplug_status_watch;
+       u8 have_hotplug_status_watch:1;
+
+       const char *hotplug_script;
+};
+
 struct xenvif {
        /* Unique identifier for this interface. */
        domid_t          domid;
@@ -283,6 +299,8 @@ struct xenvif {
        struct xenbus_watch credit_watch;
        struct xenbus_watch mcast_ctrl_watch;
 
+       struct backend_info *be;
+
        spinlock_t lock;
 
 #ifdef CONFIG_DEBUG_FS
index 330ddb64930f98f694a40afd72acb8f78515aca9..41c9e8f2e52032d24309f6971e8c947b5b949a93 100644 (file)
 #include <linux/vmalloc.h>
 #include <linux/rtnetlink.h>
 
-struct backend_info {
-       struct xenbus_device *dev;
-       struct xenvif *vif;
-
-       /* This is the state that will be reflected in xenstore when any
-        * active hotplug script completes.
-        */
-       enum xenbus_state state;
-
-       enum xenbus_state frontend_state;
-       struct xenbus_watch hotplug_status_watch;
-       u8 have_hotplug_status_watch:1;
-
-       const char *hotplug_script;
-};
-
 static int connect_data_rings(struct backend_info *be,
                              struct xenvif_queue *queue);
 static void connect(struct backend_info *be);
@@ -472,6 +456,7 @@ static int backend_create_xenvif(struct backend_info *be)
                return err;
        }
        be->vif = vif;
+       vif->be = be;
 
        kobject_uevent(&dev->dev.kobj, KOBJ_ONLINE);
        return 0;