On Tue, Dec 10, 2013 at 1:29 PM, H. Peter Anvin <hpa@xxxxxxxxx> wrote: > On 12/10/2013 01:06 PM, Yinghai Lu wrote: >> On Tue, Dec 10, 2013 at 1:06 AM, Xishi Qiu <qiuxishi@xxxxxxxxxx> wrote: >>> In the following case, e820_all_mapped() will return 1. >>> A < start < B-1 and B < end < C, it means <start, end> spans two regions. >>> <start, end>: [start - end] >>> e820 addr: ...[A - B-1][B - C]... >> >> should be [start, end) right? >> and >> [A, B),[B, C) >> > > What happens if it spans more than two regions? [A, B), [B+1, C), [C+1, D) ? start in [A, B), and end in [C+1, D). old code: first with [A, B), start set to B. then with [B+1, C), start still keep as B. then with [C+1, D), start still keep as B. at last still return 0...aka not_all_mapped. old code is still right. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>