Re: [PATCH v12 08/11] arm64/kasan: add and use kasan_map_populate()

Hi Andrey,

I asked Will, about it, and he preferred to have this patched added to 
the end of my series instead of replacing "arm64/kasan: add and use 
kasan_map_populate()".

In addition, Will's patch stops using large pages for kasan memory, and 
thus might add some regression in which case it is easier to revert just 
that patch instead of the whole series. It is unlikely that regression 
is going to be detectable, because kasan by itself makes system quiet 
slow already.

Pasha

On Wed, Oct 18, 2017 at 01:03:10PM -0400, Pavel Tatashin wrote:

I asked Will, about it, and he preferred to have this patched added to the
end of my series instead of replacing "arm64/kasan: add and use
kasan_map_populate()".

As I said, I'm fine either way, I just didn't want to cause extra work
or rebasing:

http://lists.infradead.org/pipermail/linux-arm-kernel/2017-October/535703.html

In addition, Will's patch stops using large pages for kasan memory, and thus
might add some regression in which case it is easier to revert just that
patch instead of the whole series. It is unlikely that regression is going
to be detectable, because kasan by itself makes system quiet slow already.

If it causes problems, I'll just fix them. No need to revert.

Will

As I said, I'm fine either way, I just didn't want to cause extra work
or rebasing:

http://lists.infradead.org/pipermail/linux-arm-kernel/2017-October/535703.html

Makes sense. I am also fine either way, I can submit a new patch merging 
together the two if needed.

Pavel