Date: Tue, 4 Apr 2017 10:07:23 -0500 (CDT)
From: Christoph Lameter <cl@linux.com>
To: Michal Hocko <mhocko@kernel.org>
cc: Kees Cook <keescook@chromium.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        Pekka Enberg <penberg@kernel.org>,
        David Rientjes <rientjes@google.com>,
        Joonsoo Kim <iamjoonsoo.kim@lge.com>, linux-mm@kvack.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mm: Add additional consistency check
In-Reply-To: <20170404113022.GC15490@dhcp22.suse.cz>
Message-ID: <alpine.DEB.2.20.1704041005570.23420@east.gentwo.org>
References: <20170331164028.GA118828@beast> <20170404113022.GC15490@dhcp22.suse.cz>
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 471
Lines: 12

On Tue, 4 Apr 2017, Michal Hocko wrote:

> NAK without a proper changelog. Seriously, we do not blindly apply
> changes from other projects without a deep understanding of all
> consequences.

Functionalitywise this is trivial. A page must be a slab page in order to
be able to determine the slab cache of an object. Its definitely not ok if
the page is not a slab page.

The main issue that may exist here is the adding of overhead to a critical
code path like kfree().