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] Re: [PATCH 3/3] xen-blkfront: Fix minor offset calculation f

To: Stefan Bader <stefan.bader@xxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH 3/3] xen-blkfront: Fix minor offset calculation for emulated IDE disks
From: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Date: Fri, 15 Jul 2011 11:20:26 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
Delivery-date: Fri, 15 Jul 2011 03:15:48 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4E1FDCD8.6070002@xxxxxxxxxxxxx>
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: <4E1EEEA1.9050102@xxxxxxxxxxxxx> <1310650247-7366-1-git-send-email-stefan.bader@xxxxxxxxxxxxx> <alpine.DEB.2.00.1107141820070.12963@kaball-desktop> <4E1FDCD8.6070002@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Alpine 2.00 (DEB 1167 2008-08-23)
On Fri, 15 Jul 2011, Stefan Bader wrote:
> On 14.07.2011 19:34, Stefano Stabellini wrote:
> > On Thu, 14 Jul 2011, Stefan Bader wrote:
> >> Before this a block device defined as hda1 in the configuration files
> >> would be mapped to hda, as well as hda2.
> >>
> > 
> > Don't you mean xvda and xvda2?
> > 
> Bah, yes, you are right. I had a "hda1" in a PVM guests definition and within
> the guest it had become "xvda". Basically it seems that the current code would
> map any minor number between 0-63 to 0.

That would be correct because hda1 is not allowed, the closest thing to
a valid configuration would be hda and therefore xvda.


> > Besides hda1 doesn't mean anything for an HVM guest, it is not an
> > allowed disk configuration.
> > It is also a bug in the toolstack that propagates such nonsense to
> > xenstore so I would rather fix xend and/or libxenlight.
> >
> Which seems actually to be something that could be intentional for HVM guests.
> As you say, "hdaX" does make no sense there... Unfortunately this is shared 
> for
> PVM and HVM. So my change may make sense with the first but not the latter... 
> :(

I see. The source of the problem is ancient disk configuration lines in
PV config files: like Ian wrote before, people should be using xvd* only
in PV guests config file, rather than hd* or sd*...
I think the best thing to do here would be to catch the error at the
toolstack level.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>