All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
To: Greg Kurz <gkurz@linux.vnet.ibm.com>, Jevon Qiao <scaleqiao@gmail.com>
Cc: haomaiwang@gmail.com, mst@redhat.com, qemu-devel@nongnu.org,
	sage@newdream.net, ceph-devel@vger.kernel.org,
	gfarnum@redhat.com
Subject: Re: [PATCH] hw/9pfs: Add CephFS support in VirtFS
Date: Sun, 17 Apr 2016 00:23:09 +0530	[thread overview]
Message-ID: <87potp9yi2.fsf@skywalker.in.ibm.com> (raw)
In-Reply-To: <20160407175030.5b6f2edb@bahia.huguette.org>

Greg Kurz <gkurz@linux.vnet.ibm.com> writes:

> On Tue, 15 Mar 2016 00:02:48 +0800
> Jevon Qiao <scaleqiao@gmail.com> wrote:
>
>> Ceph as a promising unified distributed storage system is widely used in the
>> world of OpenStack. OpenStack users deploying Ceph for block (Cinder) and
>> object (S3/Swift) are unsurprisingly looking at Manila and CephFS to round out
>> a unified storage solution. Since the typical hypervisor people are using is
>> Qemu/KVM, it is necessary to provide a high performance, easy to use, file
>> system service in it. VirtFS aims to offers paravirtualized system services and
>> simple passthrough for directories from host to guest, which currently only
>> support local file system, this patch wants to add CephFS support in VirtFS.
>> 
>> Signed-off-by: Jevon Qiao <scaleqiao@gmail.com>
>> ---
>
> Jevon,
>
> There's still work to be done on this patch.
>
> One general remark is that there are far too many traces: it obfuscates the code
> and does not bring much benefit in my opinion. If you look at the other fsdev
> drivers, you see they don't do traces at all !
>
> Also, I've found several errors where the code simply cannot work... please run
> a file/io oriented testsuite in the guest to check all the fsdev operations are
> working as expected... maybe some tests from LTP ?

Also use this http://tuxera.com/sw/qa/pjd-fstest-20090130-RC.tgz 

-aneesh

WARNING: multiple messages have this Message-ID (diff)
From: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
To: Greg Kurz <gkurz@linux.vnet.ibm.com>, Jevon Qiao <scaleqiao@gmail.com>
Cc: haomaiwang@gmail.com, mst@redhat.com, qemu-devel@nongnu.org,
	sage@newdream.net, ceph-devel@vger.kernel.org,
	gfarnum@redhat.com
Subject: Re: [Qemu-devel] [PATCH] hw/9pfs: Add CephFS support in VirtFS
Date: Sun, 17 Apr 2016 00:23:09 +0530	[thread overview]
Message-ID: <87potp9yi2.fsf@skywalker.in.ibm.com> (raw)
In-Reply-To: <20160407175030.5b6f2edb@bahia.huguette.org>

Greg Kurz <gkurz@linux.vnet.ibm.com> writes:

> On Tue, 15 Mar 2016 00:02:48 +0800
> Jevon Qiao <scaleqiao@gmail.com> wrote:
>
>> Ceph as a promising unified distributed storage system is widely used in the
>> world of OpenStack. OpenStack users deploying Ceph for block (Cinder) and
>> object (S3/Swift) are unsurprisingly looking at Manila and CephFS to round out
>> a unified storage solution. Since the typical hypervisor people are using is
>> Qemu/KVM, it is necessary to provide a high performance, easy to use, file
>> system service in it. VirtFS aims to offers paravirtualized system services and
>> simple passthrough for directories from host to guest, which currently only
>> support local file system, this patch wants to add CephFS support in VirtFS.
>> 
>> Signed-off-by: Jevon Qiao <scaleqiao@gmail.com>
>> ---
>
> Jevon,
>
> There's still work to be done on this patch.
>
> One general remark is that there are far too many traces: it obfuscates the code
> and does not bring much benefit in my opinion. If you look at the other fsdev
> drivers, you see they don't do traces at all !
>
> Also, I've found several errors where the code simply cannot work... please run
> a file/io oriented testsuite in the guest to check all the fsdev operations are
> working as expected... maybe some tests from LTP ?

Also use this http://tuxera.com/sw/qa/pjd-fstest-20090130-RC.tgz 

-aneesh

  parent reply	other threads:[~2016-04-16 18:53 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-14 16:02 [PATCH] hw/9pfs: Add CephFS support in VirtFS Jevon Qiao
2016-03-15  9:30 ` Greg Kurz
2016-03-15  9:30   ` [Qemu-devel] " Greg Kurz
2016-03-15 13:39   ` Jevon Qiao
2016-03-15 13:39     ` [Qemu-devel] " Jevon Qiao
2016-03-15 13:46     ` Greg Kurz
2016-03-15 13:46       ` [Qemu-devel] " Greg Kurz
2016-03-15 14:16       ` Jevon Qiao
2016-03-15 14:16         ` [Qemu-devel] " Jevon Qiao
2016-04-05 15:27         ` Jevon Qiao
2016-04-05 15:27           ` [Qemu-devel] " Jevon Qiao
2016-04-05 15:31           ` Greg Kurz
2016-04-05 15:31             ` [Qemu-devel] " Greg Kurz
2016-04-06  4:28             ` Jevon Qiao
2016-04-06  4:28               ` [Qemu-devel] " Jevon Qiao
2016-03-15 13:52 ` Michael S. Tsirkin
2016-03-15 13:52   ` [Qemu-devel] " Michael S. Tsirkin
2016-03-15 14:33   ` Jevon Qiao
2016-03-15 14:33     ` [Qemu-devel] " Jevon Qiao
2016-04-07 15:50 ` Greg Kurz
2016-04-07 15:50   ` [Qemu-devel] " Greg Kurz
2016-04-10  6:55   ` Jevon Qiao
2016-04-10  6:55     ` [Qemu-devel] " Jevon Qiao
2016-04-13 14:20     ` Greg Kurz
2016-04-16 18:53   ` Aneesh Kumar K.V [this message]
2016-04-16 18:53     ` Aneesh Kumar K.V
2016-04-15 12:01 ` Greg Kurz
2016-04-15 12:01   ` [Qemu-devel] " Greg Kurz
2016-04-15 13:21 ` Greg Kurz
2016-04-15 13:21   ` [Qemu-devel] " Greg Kurz
  -- strict thread matches above, loose matches on Subject: below --
2016-03-02 15:41 Jevon Qiao
2016-03-08  0:51 ` Jevon Qiao
2016-03-09  9:59   ` Greg Kurz
2016-03-09 19:02 ` Greg Kurz
2016-03-14  2:02   ` Jevon Qiao

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87potp9yi2.fsf@skywalker.in.ibm.com \
    --to=aneesh.kumar@linux.vnet.ibm.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=gfarnum@redhat.com \
    --cc=gkurz@linux.vnet.ibm.com \
    --cc=haomaiwang@gmail.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=sage@newdream.net \
    --cc=scaleqiao@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.