[PATCH 14/17] xen: pvblock: Read XenStore configuration and initialize

[Simon Glass]

Hi Anastasiia,

On Wed, 1 Jul 2020 at 10:30, Anastasiia Lukianenko <vicooodin at gmail.com> wrote:
>
> From: Anastasiia Lukianenko <anastasiia_lukianenko at epam.com>
>
> Read essential virtual block device configuration data from XenStore,
> initialize front ring and event channel.
> Update block device description with actual block size.
>
> Use code for XenStore from mini-os.
>
> Signed-off-by: Oleksandr Andrushchenko <oleksandr_andrushchenko at epam.com>
> Signed-off-by: Anastasiia Lukianenko <anastasiia_lukianenko at epam.com>
> ---
>  drivers/xen/pvblock.c | 272 +++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 271 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/xen/pvblock.c b/drivers/xen/pvblock.c
> index 6ce0ae97c3..9ed18be633 100644
> --- a/drivers/xen/pvblock.c
> +++ b/drivers/xen/pvblock.c
> @@ -1,6 +1,7 @@
>  /*
>   * SPDX-License-Identifier:    GPL-2.0+
>   *
> + * (C) 2007-2008 Samuel Thibault.
>   * (C) Copyright 2020 EPAM Systems Inc.
>   */
>  #include <blk.h>
> @@ -10,26 +11,289 @@
>  #include <malloc.h>
>  #include <part.h>
>
> +#include <asm/armv8/mmu.h>
> +#include <asm/io.h>
> +#include <asm/xen/system.h>
> +
> +#include <linux/compat.h>
> +
> +#include <xen/events.h>
> +#include <xen/gnttab.h>
> +#include <xen/hvm.h>
>  #include <xen/xenbus.h>
>
> +#include <xen/interface/io/ring.h>
> +#include <xen/interface/io/blkif.h>
> +#include <xen/interface/io/protocols.h>
> +
>  #define DRV_NAME       "pvblock"
>  #define DRV_NAME_BLK   "pvblock_blk"
>
> +#define O_RDONLY       00
> +#define O_RDWR         02
> +
> +struct blkfront_info {
> +       u64 sectors;
> +       unsigned int sector_size;
> +       int mode;
> +       int info;
> +       int barrier;
> +       int flush;
> +};
> +
>  struct blkfront_dev {
> -       char dummy;
> +       domid_t dom;
> +
> +       struct blkif_front_ring ring;
> +       grant_ref_t ring_ref;
> +       evtchn_port_t evtchn;
> +       blkif_vdev_t handle;
> +
> +       char *nodename;
> +       char *backend;
> +       struct blkfront_info info;
> +       unsigned int devid;

How about some comments?

Regards,
Simon