linux-arch.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Yinghai Lu <yinghai@kernel.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	David Miller <davem@davemloft.net>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Johannes Weiner <hannes@cmpxchg.org>,
	linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org
Subject: Re: [PATCH -v25 00/49] Use memblock with x86
Date: Mon, 19 Jul 2010 17:35:05 -0700	[thread overview]
Message-ID: <4C44EF39.9000500@kernel.org> (raw)
In-Reply-To: <AANLkTimC5Yio93MQ3QPjat_vpOp4gKkV_p1hmI-Nvl3B@mail.gmail.com>

On 07/19/2010 05:14 PM, Linus Torvalds wrote:
> On Mon, Jul 19, 2010 at 4:56 PM, Yinghai Lu <yinghai@kernel.org> wrote:
>> New memblock could be used to replace early_res in x86.
>>
>> Suggested by: David, Ben, and Thomas
> 
> So how is this related to Ben's git tree that was tested on sparc? Not
> clear from the changelog.

base on Ben's git tree that was tested on sparc.
fold one patch from me to patch 19. so will not break bisecting.

Maybe Ben can check patch from 3 to 27 to see if the folding is right or not.

> 
>> First two patches are needed for 2.6.35. need to be applied at first.
> 
> Please send those two as a separate series with a separate cover-sheet
> and explanation. If they are regressions or major fixes for existing
> code, they should not go into some 50-patch series for the future.
> That way they just get lost in the noise, and it's not at all as clear
> that they are important for current kernels.

those two are already in Andrew's -mm, but were not into tip/x86 yet.
and some of patches (patch 28-49) could have some merging conflicts if
those two are not applied at first.

> 
> In fact, in general, the fewer 50-series patches we see, the better.
> If a series of 50 patches could be split up into separate independent
> series ("independent" in the sense that they do different things -
> maybe one series depends on another series for infrastructure, but
> then actually concentrates on a different issue), that would be good.
> 
> Because, quite frankly, when I get email-bombed by tens of patches, I
> immediately lose about 99% of my eagerness to actually check the
> patches out. And I bet I'm not the only one. So it would likely be
> much more productive if these kinds of things can be sent out as
> multiple smaller series that can be looked at separately (and not sent
> out at the same time).

ok, 
1. let's wait Ingo or hpa to pick first two.
2. wait Ben to rebase patch 3-27, and send them out at first.
3. then I will resend from 28 to 49.

Thanks

Yinghai

  reply	other threads:[~2010-07-20  0:37 UTC|newest]

Thread overview: 103+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-19 23:56 [PATCH -v25 00/49] Use memblock with x86 Yinghai Lu
2010-07-19 23:56 ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 01/49] x86, numa: fix boot without RAM on node0 again Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 02/49] x86,mm: fix 32bit numa sparsemem Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 03/49] memblock: Rename memblock_region to memblock_type and memblock_property to memblock_region Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 04/49] memblock: No reason to include asm/memblock.h late Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 05/49] memblock: Introduce for_each_memblock() and new accessors, and use it Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 06/49] memblock: Remove nid_range argument, arch provides memblock_nid_range() instead Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 07/49] memblock: Factor the lowest level alloc function Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 08/49] memblock: Expose MEMBLOCK_ALLOC_ANYWHERE Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 09/49] memblock: Introduce default allocation limit and use it to replace explicit ones Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 10/49] memblock: Remove rmo_size, burry it in arch/powerpc where it belongs Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 11/49] memblock: Change u64 to phys_addr_t Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 12/49] memblock: Remove unused memblock.debug struct member Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 13/49] memblock: Remove memblock_type.size and add memblock.memory_size instead Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 14/49] memblock: Move memblock arrays to static storage in memblock.c and make their size a variable Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 15/49] memblock: Add debug markers at the end of the array Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 16/49] memblock: Make memblock_find_region() out of memblock_alloc_region() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 17/49] memblock: Define MEMBLOCK_ERROR internally instead of using ~(phys_addr_t)0 Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 18/49] memblock: Move memblock_init() to the bottom of the file Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 19/49] memblock: split memblock_find_base() out of __memblock_alloc_base() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 20/49] memblock: Move functions around into a more sensible order Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 21/49] memblock: Add array resizing support Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 22/49] memblock: Add arch function to control coalescing of memblock memory regions Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 23/49] memblock: Add "start" argument to memblock_find_base() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 24/49] memblock: NUMA allocate can now use early_pfn_map Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 25/49] memblock: Separate memblock_alloc_nid() and memblock_alloc_try_nid() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 26/49] memblock: Make memblock_alloc_try_nid() fallback to MEMBLOCK_ALLOC_ANYWHERE Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 27/49] memblock: Add debugfs files to dump the arrays content Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 28/49] memblock: Prepare x86 to use memblock to replace early_res Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 29/49] memblock: Print new doubled array location info Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 30/49] memblock: Export MEMBLOCK_ERROR again Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 31/49] memblock: Prepare to include linux/memblock.h in core file Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 32/49] memblock: Add ARCH_DISCARD_MEMBLOCK to put memblock code to .init Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 33/49] memblock: Add memblock_find_in_range() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 34/49] x86, memblock: Add memblock_x86_find_in_range_size() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 35/49] bootmem, x86: Add weak version of reserve_bootmem_generic Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 36/49] x86, memblock: Add memblock_x86_to_bootmem() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 37/49] x86,memblock: Add memblock_x86_reserve_range/memblock_x86_free_range Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 38/49] x86, memblock: Add get_free_all_memory_range() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 39/49] x86, memblock: Add memblock_x86_register_active_regions() and memblock_x86_hole_size() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 40/49] memblock: Add find_memory_core_early() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 41/49] x86, memblock: Add memblock_x86_find_in_range_node() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 42/49] x86, memblock: Add memblock_x86_free_memory_in_range() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 43/49] x86, memblock: Add memblock_x86_memory_in_range() Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 44/49] x86, memblock: Use memblock_debug to control debug message print out Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 45/49] x86: Use memblock to replace early_res Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 46/49] x86: Replace e820_/_early string with memblock_ Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 47/49] x86: Remove not used early_res code Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 48/49] x86, memblock: Use memblock_memory_size()/memblock_free_memory_size() to get correct dma_reserve Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-19 23:56 ` [PATCH 49/49] x86: remove old bootmem code Yinghai Lu
2010-07-19 23:56   ` Yinghai Lu
2010-07-20  0:14 ` [PATCH -v25 00/49] Use memblock with x86 Linus Torvalds
2010-07-20  0:35   ` Yinghai Lu [this message]
2010-07-20 18:29     ` H. Peter Anvin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C44EF39.9000500@kernel.org \
    --to=yinghai@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=benh@kernel.crashing.org \
    --cc=davem@davemloft.net \
    --cc=hannes@cmpxchg.org \
    --cc=hpa@zytor.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).