On Thu, 12 May 2011, Daniel Kiper wrote: > > > Memory hotplug code strictly depends on CONFIG_SPARSEMEM. > > > It means that code depending on CONFIG_FLATMEM in online_page() > > > is never compiled. Remove it because it is not needed anymore. > > > > > > Signed-off-by: Daniel Kiper <dkiper@xxxxxxxxxxxx> > > > > The code you're patching depends on CONFIG_MEMORY_HOTPLUG_SPARSE, so this > > is valid. The changelog should be updated to reflect that, however. > > > > Acked-by: David Rientjes <rientjes@xxxxxxxxxx> > > No problem, however, this bundle of patches was added to the -mm tree. > In this situation should I repost whole bundle with relevant changes > or post only those two patches requested by you ??? For which tree > should I prepare new version of patches ??? > No, I would just reply to the email notification you received when the patch went into -mm saying that the changelog should be adjusted to read something like online_pages() is only compiled for CONFIG_MEMORY_HOTPLUG_SPARSE, so there is no need to support CONFIG_FLATMEM code within it. This patch removes code that is never used. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>