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] XendOptionsSMF, missing get_config_value method

To: levon@xxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] [Patch] XendOptionsSMF, missing get_config_value method on Solaris
From: Juergen Keil <jk@xxxxxxxx>
Date: Fri, 6 Jun 2008 15:31:03 +0200 (CEST)
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 06 Jun 2008 06:31:27 -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>
Reply-to: Juergen Keil <jk@xxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
John Levon wrote:
> On Fri, Jun 06, 2008 at 02:57:13PM +0200, Juergen Keil wrote:
> 
> > On Solaris, xend XendOptions.py uses an additional interface XendOptionsSMF.
> > This allows xend configuration via Solaris SMF as well as the config file.
> > (Added by changeset 13407).
> > 
> > Problem: XendOptionsSMF doesn't define a get_config_value; but these
> > is get_config_string.  The code in XendOptions.py needs both; 
> > get_keymap() needs get_config_value.
> 
> I'm not sure about this: SMF values are always typed and there's really
> no such thing as just a value. Instead, I think get_keymap() needs to
> use get_config_string(). Note it's the only base class routine that does
> this.

Ok, changing the base class to use get_config_string instead of 
of get_config_value would be another way to fix this.

> Note it's the only base class routine that does this.

In xen-unstable, there also is get_resource_label_change_script().


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