WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] [PATCH] Add fallback code to blktap for missing poll-on-aio

To: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] [PATCH] Add fallback code to blktap for missing poll-on-aio support
From: Mark McLoughlin <markmc@xxxxxxxxxx>
Date: Thu, 26 Apr 2007 17:50:19 +0100
Delivery-date: Thu, 26 Apr 2007 09:49:12 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Red Hat Ltd. Registered Address: Red Hat Ltd, Brian O' Donnell and Partners, 62 Merrion Square, Dublin 2, Ireland. Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, at No. 304873 Directors: Charlie Peters (USA), Michael Cunningham (USA), Brendan Lane
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Hey,
        As Stephen explains in this mail:

  http://lists.xensource.com/archives/html/xen-devel/2006-10/msg00331.html

        blktap requires a xen specific kernel AIO ABI which has been vetoed by
upstream in favour of another approach. Rather than include this ABI,
Fedora has been carrying a patch which makes tap:aio use a thread to
poll for aio events and notify the main thread via a pipe.

        The upstream approach of allowing io_getevents() poll normal file
descriptors via epoll is still progressing:

  http://lkml.org/lkml/2007/1/3/16

        but when that does make it upstream, blktap will require significant
re-working to use that approach.

        In the meantime, here's a patch which uses the poll-in-a-thread
approach only if AIO poll support isn't available. It also hides the
details behind a simple abstraction and makes both tap:aio and tap:qcow
use it.

        Built and tested on 3.0.5 ... unless I'm jinxed :-)

  Signed-off-by: Mark McLoughlin <markmc@xxxxxxxxxx>

Cheers,
Mark.

Attachment: xen-blktap-aio-poll-in-thread.patch
Description: Text Data

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-devel] [PATCH] Add fallback code to blktap for missing poll-on-aio support, Mark McLoughlin <=