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

Re: [Xen-devel] [PATCH] xl: (Linux) extend SCSI device name to major map

To: "Jan Beulich" <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xl: (Linux) extend SCSI device name to major mapping
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Tue, 21 Sep 2010 16:31:25 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 21 Sep 2010 08:32:16 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4C98B083020000780001816E@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/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>
Newsgroups: chiark.mail.xen.devel
References: <4C98B083020000780001816E@xxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Jan Beulich writes ("[Xen-devel] [PATCH] xl: (Linux) extend SCSI device name to 
major mapping"):
> Rather than only permitting SCSI_DISK0_MAJOR, permit all that blkfront
> supports. However, as it is unclear to me what the meaning of major
> is on non-Linux (it seems questionable whether non-Linux, if using a
> similar major/minor concept, would happen to use major 8 as the first
> SCSI disk one), do this extension on Linux only.

I think this is wrong, I'm afraid.

Firstly, according to the document I'm trying to get agreed (see my
mail just sent, with another copy of it), sd* devices beyond disk 15
are deprecated and not supported.  You should be using xvd* if you
can.

For backwards compatibility we do support specifying the plain
xenstore number, in decimal, hex or octal.  So you can use that if you
have old guests which for some reason you can't change.

> +#ifdef __linux__

Secondly, that is entirely wrong, because that switches on the
operating system in dom0.  That may be entirely different to the
operating system in the guest.  That shows why this whole idea of
trying to specify the major/minor numbers in the guest is wrong.  Some
guests may not even _have_ major/minor numbers.

The environment provided to a Xen guest should not reference
Linux-specific numbering schemes.  For historical reasons (ie, people
having made this mistake in the past), we have to import and redefine
some existing usages.  That's what my document does.

However:

> Also, is it intentional that the /dev/ prefix (accepted by xm) is not
> accepted on the third argument by xl?

/dev doesn't make much sense once you look at it in the way I do,
above, but this should be supported for compatibility.

Ian.

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

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