From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from a.ns.miles-group.at ([95.130.255.143] helo=radon.swed.at) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1YvWwm-00076I-Es for linux-mtd@lists.infradead.org; Thu, 21 May 2015 20:20:05 +0000 Message-ID: <555E3DDA.1090405@nod.at> Date: Thu, 21 May 2015 22:19:38 +0200 From: Richard Weinberger MIME-Version: 1.0 To: Sheng Yong Subject: Re: [RFC PATCH 4/4] UBI: Fastmap: Do not add vol if it already exists References: <1432202165-22351-1-git-send-email-shengyong1@huawei.com> <1432202165-22351-5-git-send-email-shengyong1@huawei.com> <555DB062.5070701@nod.at> In-Reply-To: <555DB062.5070701@nod.at> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 8bit Cc: linux-mtd@lists.infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Am 21.05.2015 um 12:16 schrieb Richard Weinberger: > Am 21.05.2015 um 11:56 schrieb Sheng Yong: >> When writing fastmap, there is no guarantee that two same vol ID are >> written to flash by mistake. Although this unlikely happens, we still >> should check it when attaching. > > Not sure if I understand the error scenario. > How can this happen? ...thought a bit more about that. I think your fix is correct but unless I'm very mistaken one can hit the issue only by modifying the fastmap by hand. Thanks, //richard