From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: blkback feature announcement Date: Mon, 7 Dec 2015 09:54:07 -0500 Message-ID: <20151207145406.GF22814@char.us.oracle.com> References: <566582EC02000078000BC975@prv-mh.provo.novell.com> <56657EA1.5050008@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail6.bemta4.messagelabs.com ([85.158.143.247]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1a5xBA-0002Um-84 for xen-devel@lists.xenproject.org; Mon, 07 Dec 2015 14:54:16 +0000 Content-Disposition: inline In-Reply-To: <56657EA1.5050008@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Roger Pau =?iso-8859-1?Q?Monn=E9?= , bob.liu@oracle.com Cc: xen-devel , Jan Beulich List-Id: xen-devel@lists.xenproject.org On Mon, Dec 07, 2015 at 01:42:09PM +0100, Roger Pau Monn=E9 wrote: > El 07/12/15 a les 13.00, Jan Beulich ha escrit: > > Hello, Ccing Bob. > > = > > is there a particular reason why "max-ring-page-order" gets written in > > xen_blkbk_probe(), but e.g. "feature-max-indirect-segments" and > > "feature-persistent" get written only in connect(), despite both having > > constant values (and hence the node value effectively being known as > > soon as the device exists)? > = > No, AFAIK there's no specific reason. > = > > Or in more general terms: Shouldn't it be well defined at what time > > a frontend can rely on certain nodes to be available for inspection? > > And in doing so, I'd expect the determination to be done such that > > widest flexibility is provided towards the actual implementation, i.e. > > nodes should be written as early as possible. (Of course this applies > > to other frontend/backend pairs too.) > = > I agree. Regarding blkback the nodes about persistent grants, indirect > descriptors and the ring page order should be written in > xen_blkbk_probe, while the specific information about this virtual disk > (sectors, sector size...) should be written before switching to the > connected state (ie: after hotplug scripts have run). > = > Roger. > =