From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Date: Tue, 5 Jun 2018 17:38:40 +0800 From: Wang YanQing To: axboe@kernel.dk, gregkh@linuxfoundation.org, pombredanne@nexb.com, tglx@linutronix.de, caizhiyong@hisilicon.com, rdunlap@infradead.org, jiangheng@hisilicon.com, linux-block@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5] block: add verifier for cmdline partition Message-ID: <20180605093840.GA9488@udknight> References: <20180510030637.GA25734@udknight> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20180510030637.GA25734@udknight> List-ID: On Thu, May 10, 2018 at 11:06:37AM +0800, Wang YanQing wrote: > I meet strange filesystem corruption issue recently, the reason > is there are overlaps partitions in cmdline partition argument. > > This patch add verifier for cmdline partition, then if there are > overlaps partitions, cmdline_partition will log a warning. We don't > treat overlaps partition as a error: > " > Caizhiyong said: > Partition overlap was intentionally designed in this cmdline partition. > reference http://lists.infradead.org/pipermail/linux-mtd/2013-August/048092.html > " > > Signed-off-by: Wang YanQing > --- > Changes > v4-v5: > 1:Delete unnecessary blank line at begin and end of logs. > 2:Delete newline character in log, pr_* will add newline > automatically. > > v3-v4: > 1:Fix grammar typo, reported by Randy Dunlap. > 2:Delete unnecessary type casting, reported by Caizhiyong. > 3:Record the reason why we can't treat overlapping partitions > as errors into changelog. > > v2-v3: > 1:Fix log one pair of overlaps partitions twice in cmdline_parts_verifier. > 2:Fix out of bound access in cmdline_parts_verifier. > > v1-v2: > 1:Don't treat overlaps partition as a error, but log a warning. > > Hi! Jens Axboe > > What is your suggestion? Hi! All I just want to know what is the status about this patch? ACK or NACK is welcome. Thanks