On Mon 29-05-17 12:11:28, Heiko Carstens wrote: > On Mon, May 29, 2017 at 10:52:31AM +0200, Michal Hocko wrote: > > > Why is it a problem to change the default for 'online'? As far as I can see > > > that doesn't have too much to do with the order of zones, no? > > > > `online' (aka MMOP_ONLINE_KEEP) should always inherit its current zone. > > The previous implementation made an exception to allow to shift to > > another zone if it is on the border of two zones. This is what I wanted > > to get rid of because it is just too ugly to live. > > > > But now I am not really sure what is the usecase here. I assume you know > > how to online the memoery. That's why you had to play tricks with the > > zones previously. All you need now is to use the proper MMOP_ONLINE* > > Yes, however that implies that existing user space has to be changed to > achieve the same semantics as before. That's the usecase I'm talking about. Yes that is really unfortunate. It is even more unfortunate how the original behavior got merged without a deeper consideration. > On the other hand this change would finally make s390 behave like all other > architectures, which is certainly not a bad thing. So, while thinking again > I think you convinced me to agree with this change. That is definitely good to hear. Btw. I plan to change the semantic even further. MMOP_ONLINE_KEEP currently ignores movable_node setting and I plan to change that. Hopefully this won't break more userspace... > > > 2) Another oddity is that after a memory block was brought online it's > > > association to ZONE_NORMAL or ZONE_MOVABLE seems to be fixed. Even if it > > > is brought offline afterwards: > > > > This is intended behavior because I got rid of the tricky&ugly zone > > shifting code. Ultimately I would like to allow for overlapping zones > > so the explicit online_{movable,kernel} will _always_ work. > > Ok, I see. This change (fixed memory block to zone mapping after first > online) is a bit surprising. On the other hand I can't think of a sane > usecase why one wants to change the zone a memory block belongs to. Longeterm I would really like to remove any constrains on where to online movable or kernel memory. So even if this will be problem it will be only temporary. -- Michal Hocko SUSE Labs -- 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>