Re: [PATCH 2/8] Create the ZONE_MOVABLE zone

On Fri, 26 Jan 2007, Christoph Lameter wrote:

On Thu, 25 Jan 2007, Mel Gorman wrote:

@@ -166,6 +168,8 @@ enum zone_type {
 #define ZONES_SHIFT 1
 #elif __ZONE_COUNT <= 4
 #define ZONES_SHIFT 2
+#elif __ZONE_COUNT <= 8
+#define ZONES_SHIFT 3
 #else

You do not need a shift of 3. Even with ZONE_MOVABLE the maximum
number of zones is still 4.

Yep, this is correct. If it's ever wrong, there is an additional check for __ZONE_COUNT that will print out the appropriate warning.

Thanks

--
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH 0/8] Create ZONE_MOVABLE to partition memory between movable and non-movable pages
    • From: Mel Gorman <[email protected]>
  • [PATCH 2/8] Create the ZONE_MOVABLE zone
    • From: Mel Gorman <[email protected]>
  • Re: [PATCH 2/8] Create the ZONE_MOVABLE zone
    • From: Christoph Lameter <[email protected]>

• Prev by Date: Re: slab: start_cpu_timer/cache_reap CONFIG_HOTPLUG_CPU problems
• Next by Date: Re: [-mm patch] BUG at net/sunrpc/svc.c:128 (was Re: 2.6.20-rc6-mm2)
• Previous by thread: Re: [PATCH 2/8] Create the ZONE_MOVABLE zone
• Next by thread: Re: [PATCH 2/8] Create the ZONE_MOVABLE zone
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]