From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Novotny <minovotn@redhat.com>
Subject: Re: [PATCH] Check for valid CPU_CAP value when
	creating(constructing) new domain
Date: Mon, 30 Mar 2009 09:55:23 +0200
Message-ID: <49D07AEB.2050500@redhat.com>
References: <49D069FB.1020401@redhat.com>
	<DDC9B1061F355Ekanno.masaki@jp.fujitsu.com>
	<49D0764E.60300@redhat.com>
	<DEC9B10C7013DDkanno.masaki@jp.fujitsu.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <DEC9B10C7013DDkanno.masaki@jp.fujitsu.com>
List-Unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: Masaki Kanno <kanno.masaki@jp.fujitsu.com>, xen-devel@lists.xensource.com
List-Id: xen-devel@lists.xenproject.org

Hi Kan,
looks fine to me. Sorry about indents - I forgot I was writing it in 
gedit now and not Midnight Commander editor - and that I forgot about 
scheduler type.

Best regards,
   Michal

Masaki Kanno wrote:
> 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@redhat.com>
> Signed-off-by: Masaki Kanno <kanno.masaki@jp.fujitsu.com>
>
> 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@redhat.com>
>>>>
>>>>
>>>> -------------------------------text/plain-------------------------------
>>>> _______________________________________________
>>>> Xen-devel mailing list
>>>> Xen-devel@lists.xensource.com
>>>> http://lists.xensource.com/xen-devel
>>>>     
>>>>         
>>>   
>>>       
>> -------------------------------text/plain-------------------------------
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@lists.xensource.com
>> http://lists.xensource.com/xen-devel
>>