From owner-freebsd-xen@FreeBSD.ORG Wed Jul 6 22:55:20 2011 Return-Path: Delivered-To: freebsd-xen@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 58BA3106566B; Wed, 6 Jul 2011 22:55:20 +0000 (UTC) (envelope-from gibbs@FreeBSD.org) Received: from aslan.scsiguy.com (www.scsiguy.com [70.89.174.89]) by mx1.freebsd.org (Postfix) with ESMTP id 3060B8FC18; Wed, 6 Jul 2011 22:55:20 +0000 (UTC) Received: from Justins-MacBook-Pro.local (207-225-98-3.dia.static.qwest.net [207.225.98.3]) (authenticated bits=0) by aslan.scsiguy.com (8.14.4/8.14.4) with ESMTP id p66Muhlp079759 (version=TLSv1/SSLv3 cipher=DHE-RSA-CAMELLIA256-SHA bits=256 verify=NO); Wed, 6 Jul 2011 16:56:43 -0600 (MDT) (envelope-from gibbs@FreeBSD.org) Message-ID: <4E14E7D2.9040708@FreeBSD.org> Date: Wed, 06 Jul 2011 16:55:14 -0600 From: "Justin T. Gibbs" Organization: The FreeBSD Project User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.18) Gecko/20110616 Thunderbird/3.1.11 MIME-Version: 1.0 To: Colin Percival References: <4DF18EE5.5090704@freebsd.org> <4DF27EFF.5060005@FreeBSD.org> <4E13B6E9.9070202@freebsd.org> <4E13C2A4.5040202@FreeBSD.org> <4E13CB8F.8000009@freebsd.org> <4E13E111.9070005@freebsd.org> In-Reply-To: <4E13E111.9070005@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.6 (aslan.scsiguy.com [70.89.174.89]); Wed, 06 Jul 2011 16:56:43 -0600 (MDT) Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Content-Filtered-By: Mailman/MimeDel 2.1.5 Cc: "freebsd-xen@freebsd.org" Subject: Re: breakage in blkfront with ring_pages > 1 X-BeenThere: freebsd-xen@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: gibbs@FreeBSD.org List-Id: Discussion of the freebsd port to xen - implementation and usage List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Jul 2011 22:55:20 -0000 On 7/5/11 10:14 PM, Colin Percival wrote: > On 07/05/11 19:42, Colin Percival wrote: > > On 07/05/11 19:04, Justin T. Gibbs wrote: > >> On 7/5/11 7:14 PM, Colin Percival wrote: > >>> Maybe the right option is to have a loader tunable dev.xn.linuxback to > >>> control which version of the protocol is used? > >> > >> What a mess. > > > > Yep. Mess or not, shall I go ahead with having a loader tunable control this, > > or can you think of a better solution? > > Does anyone object to the attached patch? It keeps the differing behaviour to > a minimum -- we MUST set ring-ref with a FreeBSD blkback, and we MUST NOT set > it with a linux blkback -- but otherwise errs in the direction of setting more > variables than are needed, to maximize the possibility of a future blkback > being compatible with both blkback_is_linux=0 and blkback_is_linux=1. It would be better to just change the FreeBSD blkback driver to be compatible with the RedHat convention. I'm still unclear on why the current FreeBSD blkfront driver believes that it can use more than one page in your configuration given that the RedHat blkfront doesn't advertise this capability in a way that the FreeBSD blkfront understands (max-ring-pages isn'te set by blkback). Did you do something to force blkfront to use more than one page? -- Justin