Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760265Ab2FGJGx (ORCPT ); Thu, 7 Jun 2012 05:06:53 -0400 Received: from mga02.intel.com ([134.134.136.20]:30449 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760189Ab2FGJGv (ORCPT ); Thu, 7 Jun 2012 05:06:51 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.67,351,1309762800"; d="asc'?scan'208";a="154117551" Message-ID: <1339060240.6875.90.camel@sauron.fi.intel.com> Subject: Re: [PATCH v4] UBIFS: compute KSA size and store in superblock From: Artem Bityutskiy Reply-To: dedekind1@gmail.com To: Joel Reardon Cc: linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Date: Thu, 07 Jun 2012 12:10:40 +0300 In-Reply-To: References: <1337952271.30969.37.camel@sauron.fi.intel.com> <1338391121.2536.94.camel@sauron.fi.intel.com> <1338459575.2536.120.camel@sauron.fi.intel.com> <1338460565.2536.126.camel@sauron.fi.intel.com> Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-HBj7PDUNvFQ440jAsqb8" X-Mailer: Evolution 3.2.3 (3.2.3-3.fc16) Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2779 Lines: 70 --=-HBj7PDUNvFQ440jAsqb8 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, 2012-06-06 at 20:52 +0200, Joel Reardon wrote: > This patch makes the super block reserve space for the KSA. It computes t= he > number of KSA LEBs and adds the ksa_lebs to on-medium superblock. The val= ue is > read off the media. The decision to use a KSA is controlled by use_ubifse= c > switch, added as a flag to the superblock, ubifs_info, and is controlled = as a mount > option. Validate superblock also checks these values. >=20 > This is tested by creating a drive and checking the mount debug output. T= he > number of LEBS assigned during create_default_filesystem was changed to e= nsure > that remounting does infact read the old value. When the use_ubifsec opti= on > was not enabled, no LEBs were assigned. Integck was run and the drive was > filled to capacity. The LEBs in the KSA were not used when filled. Invali= d > value for ksa_lebs were used and then rejected by validate superblock. >=20 > Signed-off-by: Joel Reardon Thank Joel, but this patch has several major issues with the KSA size calculation, I believe. I've pushed it anyway to the 'joel' branch, then on top of it I've added several clean-ups and little improvements, and then on top of it I've added a patch which adds several TODO comments for you. Please, take a look. I've also sent the patches to you separately. --=20 Best Regards, Artem Bityutskiy --=-HBj7PDUNvFQ440jAsqb8 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) iQIcBAABAgAGBQJP0HAQAAoJECmIfjd9wqK06MoQAMavzvs6AkPYvyuPk/5zgpAr KjDBcB1/tsTfqnxmnvDcLm9DIgnWMlvZJLRtQudDM9mH+1KpN6Tgkqs9iVpuHJxA sGbdTux3pAeTKtLezz+9ybL3s2V3gnydO+NRER7y0fxJdAYi9+y1rKRyMmTO/nJJ aMb1UMA45wl7YYrkuFujrIqpbYtJVpuTE3XajDDkYbiXQGtq3EOt2J2HXtGGKxYh UN1bGlUgedRFz0vOotAWfub6cEK4a/eeD1tmKuze+xZcwqYSTOLJFT2vkwXM2x5W pSuicSZAYxrRFa4HNzgAKWOMT/q147fH2YLgrZJ0G7IddeUApGfEYx4FAOX6Dke1 wTsQ++cBOUb3k4ilhS0fz8ZM5rrBICqK9kYyOVh2VeXYqL4YDYJ7uZGtF7dEN+Ov dxucOutRdPeOFvfM6toJ4I4roRTD7WhE7VZCPy4WsXTIXlYf5HBqR86UT4yf8KpY 4s5rS297hpFk96dfC71yaEozU5sP4FXpp45hO7Xc7CODMpSOgUP0xp3+XPAaZl9T ltyw0HaM01Mq6Y369mhtxGyoT5JM++2WBZ4lugYQmC079mE+z1s6EZ9X8KyQeMsi ITSoP5HS5ZHXnRXdl/ErZhDnPG+8+Vujox0aFU2XU/2EmedRegOoPXgmCEuwQ83r 7wCrc6r4IhzNWGy1fkaH =iFa0 -----END PGP SIGNATURE----- --=-HBj7PDUNvFQ440jAsqb8-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/