From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg0-f68.google.com ([74.125.83.68]:46588 "EHLO mail-pg0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751615AbeFGFfD (ORCPT ); Thu, 7 Jun 2018 01:35:03 -0400 Date: Thu, 7 Jun 2018 13:34:56 +0800 From: Eryu Guan Subject: Re: [PATCH v2 2/2] overlay/016: split into two tests Message-ID: <20180607053456.GB2861@desktop> References: <1527965539-30607-1-git-send-email-amir73il@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: fstests-owner@vger.kernel.org To: Amir Goldstein Cc: Miklos Szeredi , Vivek Goyal , overlayfs , fstests List-ID: On Tue, Jun 05, 2018 at 03:09:59PM +0300, Amir Goldstein wrote: > On Sat, Jun 2, 2018 at 9:52 PM, Amir Goldstein wrote: > > overlay/016 tests two cases of ro/rw fd data inconsistecies - > > one using pread and one using mmap read (i.e. mread). > > > > The first case will pass as soon as stacked overlay operations > > patch set is merged. The second case requires enabling a new opt-in > > feature "copy_up_shared" in order to pass. > > > > Instead of changing the entire test from fail to notrun, split > > the two cases into two tests and require the new feature only > > for the new test. > > > > This way, the new test will change from notrun to pass with > > stacked overlay operations and the old test will change from > > fail to pass. > > > > Signed-off-by: Amir Goldstein > > --- > > Eryu, > > Please hold back on this patch - there seems to be a push back > on the "copy_up_shared" feature. > It may end up making sense to do the split anyway because > case #1 may be resolved in this merge cycle and case #2 only > later on, but let's wait to see what gets merged first and then decide. Thanks for the heads-up! I removed the patch from my to-review queue, please re-submit when the decision is made. > > No problem with merging the cleanup patch > "overlay: nicer report when features are not supported" OK. Thanks, Eryu