linux-kernel - Re: [PATCH v1] mm/memory_hotplug: don't check the nid in find_(smallest|biggest)_section

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <c1857505-4565-99c8-d86d-efa6c076312a@redhat.com>
Date:   Thu, 28 Nov 2019 15:52:10 +0100
From:   David Hildenbrand <david@...hat.com>
To:     Michal Hocko <mhocko@...nel.org>, Qian Cai <cai@....pw>
Cc:     linux-kernel@...r.kernel.org, linux-mm@...ck.org,
        Andrew Morton <akpm@...ux-foundation.org>,
        Oscar Salvador <osalvador@...e.de>
Subject: Re: [PATCH v1] mm/memory_hotplug: don't check the nid in
 find_(smallest|biggest)_section_pfn

On 28.11.19 15:42, Michal Hocko wrote:
> On Thu 28-11-19 09:30:29, Qian Cai wrote:
>>
>>
>>> On Nov 28, 2019, at 9:03 AM, David Hildenbrand <david@...hat.com> wrote:
>>>
>>> That's why we have linux-next and plenty of people playing with it
>>> (including you and me for example).
>>
>> As mentioned, it is an expensive development practice. Once a patch
>> was merged into linux-next, it becomes someone else’s problems
>> because if nobody flags it as problematic, all it needs is a good eye
>> review and some time before it gets merged into mainline eventually.
> 
> I would tend to agree. linux-next shouldn't be considered a low bar
> target. Things should be reviewed before showing up there. There are
> obviously some exceptions, as always, but it shouldn't be over used.
> 
> I wish MM patches would be applied to mmotm (and linux-next) more
> conservatively.

I also agree that it should not be used for basic functional/compile
tests (I said "It is a way of giving patches *more* testing."). It
should not be the only place to test stuff (especially to let somebody
else do it).

However, sometimes we really have to get additional test coverage via
linux-next, especially for weird archs/configurations/setups.

... and if we don't have enough reviewers, it's really hard to get stuff
upstream.

I wish MM patches would get reviewed more thoroughly.

(If we all make a wish, maybe Santa Clause will listen ;) )

-- 
Thanks,

David / dhildenb