From: Huang Shijie <shijie8@gmail.com> To: dedekind1@gmail.com Cc: dwmw2@infradead.org, linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, shmulik.ladkani@gmail.com Subject: Re: [PATCH 3/3] mtd: cmdlinepart: fix the wrong partitions number when truncating occurs Date: Thu, 30 Aug 2012 14:39:55 +0800 [thread overview] Message-ID: <CAMiH66HaRG--3bC9HCfNu7S_WWKWx_COEuJdtgmBTNbdYdVt8Q@mail.gmail.com> (raw) In-Reply-To: <1346309023.2848.531.camel@sauron.fi.intel.com> On Thu, Aug 30, 2012 at 2:43 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote: > On Sun, 2012-08-26 at 13:21 -0400, Huang Shijie wrote: >> This patch is based on the assumption that all the partitions are >> in the right offset order. >> >> Assume we have a 1GB(8Gb) nand chip, and we set the partitions >> in the command line like this: >> #gpmi-nand:100m(boot),100m(kernel),1g(rootfs) >> >> In this case, the partition truncating occurs. The current code will >> get the following result: >> >> ---------------------------------- >> root@freescale ~$ cat /proc/mtd >> dev: size erasesize name >> mtd0: 06400000 00040000 "boot" >> mtd1: 06400000 00040000 "kernel" >> ---------------------------------- >> >> It is obvious that we lost the truncated partition `rootfs` which should >> be 824M in this case. >> >> Why? The old code sets the wrong partitions number when the truncating >> occurs. This patch fixes it. Alao add a `break` to shortcut the code in this >> case. >> >> After apply this patch, the result becomes: >> ---------------------------------- >> root@freescale ~$ cat /proc/mtd >> dev: size erasesize name >> mtd0: 06400000 00040000 "boot" >> mtd1: 06400000 00040000 "kernel" >> mtd2: 33800000 00040000 "rootfs" >> ---------------------------------- >> >> We get the right result. >> >> Signed-off-by: Huang Shijie <shijie8@gmail.com> > > Should this have CC to -stable? It's ok to CC to stable. thanks Huang Shijie > > -- > Best Regards, > Artem Bityutskiy
WARNING: multiple messages have this Message-ID (diff)
From: Huang Shijie <shijie8@gmail.com> To: dedekind1@gmail.com Cc: linux-mtd@lists.infradead.org, dwmw2@infradead.org, linux-kernel@vger.kernel.org, shmulik.ladkani@gmail.com Subject: Re: [PATCH 3/3] mtd: cmdlinepart: fix the wrong partitions number when truncating occurs Date: Thu, 30 Aug 2012 14:39:55 +0800 [thread overview] Message-ID: <CAMiH66HaRG--3bC9HCfNu7S_WWKWx_COEuJdtgmBTNbdYdVt8Q@mail.gmail.com> (raw) In-Reply-To: <1346309023.2848.531.camel@sauron.fi.intel.com> On Thu, Aug 30, 2012 at 2:43 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote: > On Sun, 2012-08-26 at 13:21 -0400, Huang Shijie wrote: >> This patch is based on the assumption that all the partitions are >> in the right offset order. >> >> Assume we have a 1GB(8Gb) nand chip, and we set the partitions >> in the command line like this: >> #gpmi-nand:100m(boot),100m(kernel),1g(rootfs) >> >> In this case, the partition truncating occurs. The current code will >> get the following result: >> >> ---------------------------------- >> root@freescale ~$ cat /proc/mtd >> dev: size erasesize name >> mtd0: 06400000 00040000 "boot" >> mtd1: 06400000 00040000 "kernel" >> ---------------------------------- >> >> It is obvious that we lost the truncated partition `rootfs` which should >> be 824M in this case. >> >> Why? The old code sets the wrong partitions number when the truncating >> occurs. This patch fixes it. Alao add a `break` to shortcut the code in this >> case. >> >> After apply this patch, the result becomes: >> ---------------------------------- >> root@freescale ~$ cat /proc/mtd >> dev: size erasesize name >> mtd0: 06400000 00040000 "boot" >> mtd1: 06400000 00040000 "kernel" >> mtd2: 33800000 00040000 "rootfs" >> ---------------------------------- >> >> We get the right result. >> >> Signed-off-by: Huang Shijie <shijie8@gmail.com> > > Should this have CC to -stable? It's ok to CC to stable. thanks Huang Shijie > > -- > Best Regards, > Artem Bityutskiy
next prev parent reply other threads:[~2012-08-30 6:39 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-08-26 17:21 [PATCH 1/3] mtd: cmdlinepart: make the partitions rule more strict Huang Shijie 2012-08-26 17:21 ` Huang Shijie 2012-08-26 17:21 ` [PATCH 2/3] mtd: cmdlinepart: sort the unsorted partitions Huang Shijie 2012-08-26 17:21 ` Huang Shijie 2012-08-31 13:59 ` Artem Bityutskiy 2012-08-31 13:59 ` Artem Bityutskiy 2012-08-31 14:29 ` Huang Shijie 2012-08-31 14:29 ` Huang Shijie 2012-09-03 7:21 ` Artem Bityutskiy 2012-09-03 7:21 ` Artem Bityutskiy 2012-08-26 17:21 ` [PATCH 3/3] mtd: cmdlinepart: fix the wrong partitions number when truncating occurs Huang Shijie 2012-08-26 17:21 ` Huang Shijie 2012-08-30 6:43 ` Artem Bityutskiy 2012-08-30 6:43 ` Artem Bityutskiy 2012-08-30 6:39 ` Huang Shijie [this message] 2012-08-30 6:39 ` Huang Shijie 2012-08-31 11:45 ` [PATCH 1/3] mtd: cmdlinepart: make the partitions rule more strict Artem Bityutskiy 2012-08-31 11:45 ` Artem Bityutskiy 2012-08-31 13:36 ` Huang Shijie 2012-08-31 13:36 ` Huang Shijie 2012-08-31 14:30 ` Huang Shijie 2012-08-31 14:30 ` Huang Shijie 2012-09-03 7:18 ` Artem Bityutskiy 2012-09-03 7:18 ` Artem Bityutskiy 2012-09-03 15:09 ` Huang Shijie 2012-09-03 15:09 ` Huang Shijie 2012-09-03 15:35 ` Artem Bityutskiy 2012-09-04 3:23 ` Huang Shijie 2012-09-04 11:48 ` Shmulik Ladkani 2012-09-05 2:12 ` Huang Shijie 2012-09-05 5:30 ` Shmulik Ladkani 2012-09-22 16:01 ` Artem Bityutskiy 2012-12-17 1:11 ` Christopher Cordahi 2012-12-18 5:27 ` Brian Norris 2013-01-15 11:49 ` Artem Bityutskiy
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAMiH66HaRG--3bC9HCfNu7S_WWKWx_COEuJdtgmBTNbdYdVt8Q@mail.gmail.com \ --to=shijie8@gmail.com \ --cc=dedekind1@gmail.com \ --cc=dwmw2@infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mtd@lists.infradead.org \ --cc=shmulik.ladkani@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.