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] Add a new p2m type for broken memory

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, Tim Deegan <Tim.Deegan@xxxxxxxxxxxxx>
Subject: [Xen-devel] RE: [PATCH] Add a new p2m type for broken memory
From: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Date: Wed, 14 Jul 2010 16:10:06 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 14 Jul 2010 01:11:05 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C8632BC4.1AA26%keir.fraser@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: <789F9655DD1B8F43B48D77C5D30659731F57135D@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C8632BC4.1AA26%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcsjJ74Ru4oahV3SRouTFs6IpB7dSwAAgrBZAABdKpA=
Thread-topic: [PATCH] Add a new p2m type for broken memory

>-----Original Message-----
>From: Keir Fraser [mailto:keir.fraser@xxxxxxxxxxxxx]
>Sent: Wednesday, July 14, 2010 3:54 PM
>To: Jiang, Yunhong; Tim Deegan
>Cc: xen-devel
>Subject: Re: [PATCH] Add a new p2m type for broken memory
>
>On 14/07/2010 08:39, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx> wrote:
>
>
>> diff -r 29f0479830cd xen/include/asm-x86/page.h
>> --- a/xen/include/asm-x86/page.h Mon Jul 12 13:12:31 2010 +0800
>> +++ b/xen/include/asm-x86/page.h Mon Jul 12 13:56:21 2010 +0800
>> @@ -323,6 +323,7 @@ void setup_idle_pagetable(void);
>>  #define _PAGE_PSE_PAT 0x1000U
>>  #define _PAGE_PAGED   0x2000U
>>  #define _PAGE_SHARED  0x4000U
>> +#define _PAGE_BROKEN  0x8000U
>
>I don't see this used anywhere. Also, not that you started it, but it

Yes, this is not used now, although it will be used in future when we add the 
PV support. I will remove it and resend the patch after get Tim's feedback for 
the patch.

>doesn't seem nice to add extra _PAGE_* definitions in page.h for flags which
>never (afaict) actually appear in any pte. At least the extra

Yes, this confused me a bit at first glance.

>return-code-only flags should be separated off and commented. Having the
>macros not start with _PAGE_ might be a good idea too. Of course all this is
>ultimately to be acked/nacked by Tim.
>
> -- Keir
>
>>  /*
>>   * Debug option: Ensure that granted mappings are not implicitly unmapped.
>>
>>
>


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