Re: [Xen-devel] [PATCH v2 3/7] xen/9pfs: introduce Xen 9pfs transport driver

2017-03-13 Thread Stefano Stabellini 
On Thu, 9 Mar 2017, Boris Ostrovsky wrote:
> On 03/08/2017 06:58 PM, Stefano Stabellini wrote:
> > Introduce the Xen 9pfs transport driver: add struct xenbus_driver to
> > register as a xenbus driver and add struct p9_trans_module to register
> > as v9fs driver.
> >
> > All functions are empty stubs for now.
> >
> > Signed-off-by: Stefano Stabellini 
> > CC: boris.ostrov...@oracle.com
> > CC: jgr...@suse.com
> > CC: Eric Van Hensbergen 
> > CC: Ron Minnich 
> > CC: Latchesar Ionkov 
> > CC: v9fs-develo...@lists.sourceforge.net
> > ---
> >  net/9p/trans_xen.c | 101 
> > +
> >  1 file changed, 101 insertions(+)
> >  create mode 100644 net/9p/trans_xen.c
> >
> > diff --git a/net/9p/trans_xen.c b/net/9p/trans_xen.c
> > new file mode 100644
> > index 000..877dfd0
> > --- /dev/null
> > +++ b/net/9p/trans_xen.c
> > @@ -0,0 +1,101 @@
> > +/*
> > + * linux/fs/9p/trans_xen
> > + *
> > + * Xen transport layer.
> 
> Does this need full copyright header?

I'll import the same one as netfront.


> Either way
> 
> Reviewed-by: Boris Ostrovsky 

Thank you very much!

___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 3/7] xen/9pfs: introduce Xen 9pfs transport driver

2017-03-09 Thread Boris Ostrovsky 
On 03/08/2017 06:58 PM, Stefano Stabellini wrote:
> Introduce the Xen 9pfs transport driver: add struct xenbus_driver to
> register as a xenbus driver and add struct p9_trans_module to register
> as v9fs driver.
>
> All functions are empty stubs for now.
>
> Signed-off-by: Stefano Stabellini 
> CC: boris.ostrov...@oracle.com
> CC: jgr...@suse.com
> CC: Eric Van Hensbergen 
> CC: Ron Minnich 
> CC: Latchesar Ionkov 
> CC: v9fs-develo...@lists.sourceforge.net
> ---
>  net/9p/trans_xen.c | 101 
> +
>  1 file changed, 101 insertions(+)
>  create mode 100644 net/9p/trans_xen.c
>
> diff --git a/net/9p/trans_xen.c b/net/9p/trans_xen.c
> new file mode 100644
> index 000..877dfd0
> --- /dev/null
> +++ b/net/9p/trans_xen.c
> @@ -0,0 +1,101 @@
> +/*
> + * linux/fs/9p/trans_xen
> + *
> + * Xen transport layer.

Does this need full copyright header?

Either way

Reviewed-by: Boris Ostrovsky 




___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

[Xen-devel] [PATCH v2 3/7] xen/9pfs: introduce Xen 9pfs transport driver

2017-03-08 Thread Stefano Stabellini 
Introduce the Xen 9pfs transport driver: add struct xenbus_driver to
register as a xenbus driver and add struct p9_trans_module to register
as v9fs driver.

All functions are empty stubs for now.

Signed-off-by: Stefano Stabellini 
CC: boris.ostrov...@oracle.com
CC: jgr...@suse.com
CC: Eric Van Hensbergen 
CC: Ron Minnich 
CC: Latchesar Ionkov 
CC: v9fs-develo...@lists.sourceforge.net
---
 net/9p/trans_xen.c | 101 +
 1 file changed, 101 insertions(+)
 create mode 100644 net/9p/trans_xen.c

diff --git a/net/9p/trans_xen.c b/net/9p/trans_xen.c
new file mode 100644
index 000..877dfd0
--- /dev/null
+++ b/net/9p/trans_xen.c
@@ -0,0 +1,101 @@
+/*
+ * linux/fs/9p/trans_xen
+ *
+ * Xen transport layer.
+ *
+ * Copyright (C) 2017 by Stefano Stabellini 
+ */ 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#include 
+#include 
+#include 
+#include 
+
+static int p9_xen_cancel(struct p9_client *client, struct p9_req_t *req)
+{
+   return 0;
+}
+
+static int p9_xen_create(struct p9_client *client, const char *addr, char 
*args)
+{
+   return 0;
+}
+
+static void p9_xen_close(struct p9_client *client)
+{
+}
+
+static int p9_xen_request(struct p9_client *client, struct p9_req_t *p9_req)
+{
+   return 0;
+}
+
+static struct p9_trans_module p9_xen_trans = {
+   .name = "xen",
+   .maxsize = (1 << (XEN_9PFS_RING_ORDER + XEN_PAGE_SHIFT)),
+   .def = 1,
+   .create = p9_xen_create,
+   .close = p9_xen_close,
+   .request = p9_xen_request,
+   .cancel = p9_xen_cancel,
+   .owner = THIS_MODULE,
+};
+
+static const struct xenbus_device_id xen_9pfs_front_ids[] = {
+   { "9pfs" },
+   { "" }
+};
+
+static int xen_9pfs_front_remove(struct xenbus_device *dev)
+{
+   return 0;
+}
+
+static int xen_9pfs_front_probe(struct xenbus_device *dev,
+   const struct xenbus_device_id *id)
+{
+   return 0;
+}
+
+static int xen_9pfs_front_resume(struct xenbus_device *dev)
+{
+   return 0;
+}
+
+static void xen_9pfs_front_changed(struct xenbus_device *dev,
+   enum xenbus_state backend_state)
+{
+}
+
+static struct xenbus_driver xen_9pfs_front_driver = {
+   .ids = xen_9pfs_front_ids,
+   .probe = xen_9pfs_front_probe,
+   .remove = xen_9pfs_front_remove,
+   .resume = xen_9pfs_front_resume,
+   .otherend_changed = xen_9pfs_front_changed,
+};
+
+int p9_trans_xen_init(void)
+{
+   if (!xen_domain())
+   return -ENODEV;
+
+   pr_info("Initialising Xen transport for 9pfs\n");
+
+   v9fs_register_trans(_xen_trans);
+   return xenbus_register_frontend(_9pfs_front_driver);
+}
+module_init(p9_trans_xen_init);
+
+void p9_trans_xen_exit(void)
+{
+   v9fs_unregister_trans(_xen_trans);
+   return xenbus_unregister_driver(_9pfs_front_driver);
+}
+module_exit(p9_trans_xen_exit);
-- 
1.9.1


___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel