+ get-rid-of-zone_table-fix-2.patch added to -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The patch titled

     get-rid-of-zone_table-fix 2

has been added to the -mm tree.  Its filename is

     get-rid-of-zone_table-fix-2.patch

See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this

------------------------------------------------------
Subject: get-rid-of-zone_table-fix 2
From: Christoph Lameter <clameter@xxxxxxx>

There is still a problem after Andy's patch in connection with Optional
ZONE DMA that I pointed out earlier.  If we only have a single zone then
ZONEID_PGSHIFT == 0.  This is fine for the non NUMA case in which we have
only a single zone and ZONEID_MASK == 0 too.  page_zone_id() will always be
0.

In the NUMA case we still have one zone per node.  Thus we need to have a
correct ZONEID_PGSHIFT in order to isolate the node number from page flags.
 Right now we take NODES_SHIFT bits starting from 0!

I am not exactly sure how to fix that the right way given the complex
nested macros.  Andy may know.

The following patch checks for that condition.  We can only allow
ZONEID_PGSHIFT to be zero if the ZONEID_MASK is also zero.  (We cannot
check that with an #if because ZONEID_SHIFT contains a "sizeof(...)"
element)

Signed-off-by: Christoph Lameter <clameter@xxxxxxx>
Cc: Andy Whitcroft <apw@xxxxxxxxxxxx>
Cc: Dave Hansen <haveblue@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---

 include/linux/mm.h |    1 +
 1 file changed, 1 insertion(+)

diff -puN include/linux/mm.h~get-rid-of-zone_table-fix-2 include/linux/mm.h
--- a/include/linux/mm.h~get-rid-of-zone_table-fix-2
+++ a/include/linux/mm.h
@@ -445,6 +445,7 @@ static inline enum zone_type page_zonenu
  */
 static inline int page_zone_id(struct page *page)
 {
+	BUG_ON(ZONEID_PGSHIFT == 0 && ZONEID_MASK);
 	return (page->flags >> ZONEID_PGSHIFT) & ZONEID_MASK;
 }
 
_

Patches currently in -mm which might be from clameter@xxxxxxx are

origin.patch
get-rid-of-zone_table.patch
get-rid-of-zone_table-fix.patch
get-rid-of-zone_table-fix-2.patch
deal-with-cases-of-zone_dma-meaning-the-first-zone.patch
deal-with-cases-of-zone_dma-meaning-the-first-zone-fix.patch
get-rid-of-zone_table-fix-3.patch
introduce-config_zone_dma.patch
optional-zone_dma-in-the-vm.patch
optional-zone_dma-for-ia64.patch
remove-zone_dma-remains-from-parisc.patch
remove-zone_dma-remains-from-sh-sh64.patch
set-config_zone_dma-for-arches-with-generic_isa_dma.patch
radix-tree-rcu-lockless-readside.patch
scheduler-numa-aware-placement-of-sched_group_allnodes.patch
zvc-support-nr_slab_reclaimable--nr_slab_unreclaimable-swap_prefetch.patch
reduce-max_nr_zones-swap_prefetch-remove-incorrect-use-of-zone_highmem.patch
numa-add-zone_to_nid-function-swap_prefetch.patch
readahead-state-based-method-aging-accounting-apply-type-enum-zone_type-readahead.patch

-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Newbies FAQ]     [Kernel Archive]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Photo]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux