From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf1-f68.google.com ([209.85.167.68]:46087 "EHLO mail-lf1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726708AbeJPBHx (ORCPT ); Mon, 15 Oct 2018 21:07:53 -0400 Date: Mon, 15 Oct 2018 20:21:40 +0300 From: Cyrill Gorcunov To: Rafael David Tinoco Cc: Alexey Dobriyan , linux-kselftest@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org, shuah@kernel.org Subject: Re: [PATCH] proc: fix proc-self-map-files selftest for arm Message-ID: <20181015172140.GJ2401@uranus.lan> References: <20181011184359.15627-1-rafael.tinoco@linaro.org> <20181011205601.GA32757@avx2> <20181011210256.GE2401@uranus.lan> <20181011213006.GA13485@avx2> <20181011220009.GF2401@uranus.lan> <52504bd5-b4ed-518b-f748-c634172c6c68@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <52504bd5-b4ed-518b-f748-c634172c6c68@linaro.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Mon, Oct 15, 2018 at 01:55:14PM -0300, Rafael David Tinoco wrote: > That is what I also had in mind, thus the patch. I just realized we had > another issue on LKFT (our functional tests tool) for > proc-self-map-files-001.c. Test 001 does pretty much the same as 002, but > without the MAP_FIXED mmap flag. > > Is it okay to consolidate both tests into just 1, and focus in checking > procfs numbers conversion only, rather than if mapping 0 is allowed or not ? > Can I send a v2 with that in mind ? As to me -- yes, I would move zero page testing to a separate memory testcase. But since Alexey is the former author of the tests better wait for his opinion.