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] Check for valid CPU_CAP value when creating(cons

To: Michal Novotny <minovotn@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Check for valid CPU_CAP value when creating(constructing) new domain
From: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>
Date: Mon, 30 Mar 2009 16:52:13 +0900
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 30 Mar 2009 00:52:41 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <49D0764E.60300@xxxxxxxxxx>
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: <49D069FB.1020401@xxxxxxxxxx> <DDC9B1061F355Ekanno.masaki@xxxxxxxxxxxxxx> <49D0764E.60300@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Hi Michal,

I have added the following fixes to your patch.
 - Check scheduler type
 - Replace tab-indent with space-indent

How about the patch?

Signed-off-by: Michal Novotny <minovotn@xxxxxxxxxx>
Signed-off-by: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>

Best regards,
 Kan


Mon, 30 Mar 2009 09:35:42 +0200, Michal Novotny wrote:

>Hi Kan,
>this is new version of my patch implementing check for both CPU_CAP and 
>CPU_WEIGHT and also considering int type for both (asserting it).
>
>Best regards,
>Michal
>
>Masaki Kanno wrote:
>> Hi Michal,
>>
>> Why does the patch implement only CPU_CAP?
>> You should consider CPU_WEIGHT too.
>> And you should consider scheduler types too.
>>
>> Best regards,
>>  Kan
>>
>> Mon, 30 Mar 2009 08:43:07 +0200, Michal Novotny wrote:
>>
>>   
>>> Hi,
>>>
>>> This patch implements check for valid cpu_cap value when creating domain (
>>> in range 0 to VcpuCount * 100 as when changing scheduler parameters).
>>>
>>> Signed-off-by: Michal Novotny <minovotn@xxxxxxxxxx>
>>>
>>>
>>> -------------------------------text/plain-------------------------------
>>> _______________________________________________
>>> Xen-devel mailing list
>>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>>> http://lists.xensource.com/xen-devel
>>>     
>>
>>   
>
>
>-------------------------------text/plain-------------------------------
>_______________________________________________
>Xen-devel mailing list
>Xen-devel@xxxxxxxxxxxxxxxxxxx
>http://lists.xensource.com/xen-devel

Attachment: xen-valid-cap-value-when-creating-domain.patch
Description: Binary data

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