[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] add CephFS support in VirtFS
From: |
Aneesh Kumar K.V |
Subject: |
Re: [Qemu-devel] [PATCH] add CephFS support in VirtFS |
Date: |
Sun, 14 Feb 2016 11:31:23 +0530 |
User-agent: |
Notmuch/0.20.2 (http://notmuchmail.org) Emacs/24.5.1 (x86_64-pc-linux-gnu) |
> diff --git a/hw/9pfs/virtio-9p.c b/hw/9pfs/virtio-9p.c
> index f972731..385c01d 100644
> --- a/hw/9pfs/virtio-9p.c
> +++ b/hw/9pfs/virtio-9p.c
> @@ -1326,7 +1326,7 @@ out_nofid:
> static int32_t get_iounit(V9fsPDU *pdu, V9fsPath *path)
> {
> struct statfs stbuf;
> - int32_t iounit = 0;
> + int32_t iounit = 0, unit = 0;
> V9fsState *s = pdu->s;
>
> /*
> @@ -1334,8 +1334,21 @@ static int32_t get_iounit(V9fsPDU *pdu, V9fsPath
> *path)
> * and as well as less than (client msize - P9_IOHDRSZ))
> */
> if (!v9fs_co_statfs(pdu, path, &stbuf)) {
> - iounit = stbuf.f_bsize;
> - iounit *= (s->msize - P9_IOHDRSZ)/stbuf.f_bsize;
> + /*
> + * If host filesystem block size is larger than client msize,
> + * we will use AGESIZE as the unit. The reason why we choose
> + * AGESIZE is because the data will be splitted in terms of
> + * AGESIZE in the virtio layer. In this case, the final
> + * iounit is equal to the value of ((msize/unit) - 1) * unit.
> + */
> + if (stbuf.f_bsize > s->msize) {
> + iounit = 4096;
> + unit = 4096;
> + } else {
> + iounit = stbuf.f_bsize;
> + unit = stbuf.f_bsize;
> + }
> + iounit *= (s->msize - P9_IOHDRSZ)/unit;
> }
> if (!iounit) {
> iounit = s->msize - P9_IOHDRSZ;
> --
Can you split this out as a separate patch ?. Also explain what is
AGESIZE.
-aneesh