This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-devel] Re: [RFC PATCH v01] Xen PVSCSI drivers for pvops xen/stable-

To: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [RFC PATCH v01] Xen PVSCSI drivers for pvops xen/stable-2.6.32.x kernel
From: Olaf Hering <olaf@xxxxxxxxx>
Date: Mon, 10 Jan 2011 11:15:38 +0100
Cc: Patrick Colp <pjcolp@xxxxxxxxx>, Jeremy Fitzhardinge <jeremy@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Nathanael Rensen <nathanael@xxxxxxxxxxxxxxxx>
Delivery-date: Mon, 10 Jan 2011 02:16:33 -0800
Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; t=1294654543; l=1797; s=domk; d=aepfle.de; h=In-Reply-To:Content-Transfer-Encoding:Content-Type:MIME-Version: References:Subject:Cc:To:From:Date:X-RZG-CLASS-ID:X-RZG-AUTH; bh=KqYvB39V4C2ZyG0O6Rdzgwkz+no=; b=J1yWFLYrMDR8VQMwQW1PIXFXD9/v7OcroAzTDrEnfjYPJWaPRFyyAqdgQyHxL6nzUDD MvS3grD/m2/9q909uNxilXbTK3ZQFWzVGWyrGxpMTf0EruT9aUpCJtc6M80NGiAr6P+vI 9Z/YgngjOvARklTJwbmAYyIA1yKl/ljxbrA=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1294396016.3831.4036.camel@xxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <AANLkTimbt0==v4fUo563X=D9wgvU2G-Qt5tEOmEEqBgJ@xxxxxxxxxxxxxx> <1294396016.3831.4036.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.20 (2009-06-14)
On Fri, Jan 07, Ian Campbell wrote:

> On Fri, 2011-01-07 at 10:03 +0000, Nathanael Rensen wrote:
> > On 7 January 2011 06:35,  Pasi Kärkkäinen <pasik@xxxxxx> wrote:
> > > Hello,
> > >
> > > http://pasik.reaktio.net/xen/patches/xen-pvscsi-drivers-linux-
> > >
> > > This is the first version of Xen PVSCSI drivers, both the scsiback 
> > > backend and
> > > scsifront frontend, ported from Novell SLES11SP1 2.6.32 Xenlinux kernel to
> > > pvops xen/stable-2.6.32.x branch.
> > >
> > > -- Pasi
> > 
> > That's great. Some comments based on my experience with pvusb:
> > 
> > 1) You've added GNTST_eagain into include/xen/interface/grant_table.h.
> > When porting the pvusb drivers I removed the "check and retry" loops
> > based on GNTST_eagain because I noticed that netback and blkback don't
> > do that. Looking at the grant_table code in xen-4.0-testing I don't
> > think GNTST_eagain is used now. Perhaps someone with knowledge of the
> > history of GNTST_eagain could help to clarify.
> IIRC GNTST_eagain was added to the classic Xen trees but never forward
> ported to pvops.
> It's part of the page sharing paging work (so dom0 can deal with a
> paged/shared guest rather than for paging of dom0 itself). I guess those
> don't work with pvops kernels? CC'ing the guys who are working on that
> stuff.

I have not tried the pvops kernel with xenpaging.

> (The presence of the use of msleep in the eagain case is a bit
> terrifying. Stripping it out for now and redoing it might not be a bad
> plan)

Right now the retry is required for xenpaging, otherwise the guests
filesystem will be corrupted.
In the long run the newly added waitqueue feature may be used to make
the page access transparent to the guest and dom0.


Xen-devel mailing list