From: Caizhiyong <caizhiyong@huawei.com> To: Brian Norris <computersforpeace@gmail.com>, Andrew Morton <akpm@linux-foundation.org> Cc: Karel Zak <kzak@redhat.com>, "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "Wanglin (Albert)" <albert.wanglin@huawei.com>, Artem Bityutskiy <dedekind1@gmail.com>, Shmulik Ladkani <shmulik.ladkani@gmail.com>, Huang Shijie <b32955@freescale.com> Subject: RE: [PATCH] block: add command line partition parser Date: Thu, 15 Aug 2013 03:38:47 +0000 [thread overview] Message-ID: <C3050A4DBA34F345975765E43127F10F1C063B36@szxeml512-mbx.china.huawei.com> (raw) In-Reply-To: <CAN8TOE-aGcKQNayzEVQaS1wJ_Uqp3RAZ5pTuVEhaaY9f487-xA@mail.gmail.com> > -----Original Message----- > From: Brian Norris [mailto:computersforpeace@gmail.com] > Sent: Thursday, August 15, 2013 8:12 AM > To: Andrew Morton > Cc: Caizhiyong; Karel Zak; linux-mtd@lists.infradead.org; > linux-kernel@vger.kernel.org; Wanglin (Albert); Artem Bityutskiy; Shmulik Ladkani; > Huang Shijie > Subject: Re: [PATCH] block: add command line partition parser > > On Wed, Aug 14, 2013 at 3:57 PM, Andrew Morton > <akpm@linux-foundation.org> wrote: > > On Tue, 13 Aug 2013 06:02:17 +0000 Caizhiyong <caizhiyong@huawei.com> wrote: > > > >> move the command line parser to a separate module, and change it into > >> library-style code. > >> > >> reference: https://lkml.org/lkml/2013/8/6/550 > > The most recent patch is an addendum to this linked patch then? > > > Well OK. But to prove the library's usefulness and to generally clean > > up the kernel, someone needs to sign up to the task of converting > > drivers/mtd/cmdlinepart.c to use this code. > > > > I've been hopefully cc'ing various MTD people but am not being > > overwhelmed with waves of enthusiasm ;) > > "I've been" implies that you have done so prior to this email. And > "people" implies more than one person. I see that you CC'd David > Woodhouse over a week ago, but he's fairly silent these days on MTD > things. It's Artem or me who handle most of the day-to-day of MTD. And > this is the first time I've seen this! (BTW, please include > linux-mtd@lists.infradead.org for anything involving MTD.) > > This seems reasonable, and I'd be willing to work with this proposal. > > Caizhiyong, can you submit a clear single patch (or series of > patches), CC'd to linux-mtd at least? Then we can see about supporting > it in MTD. It doesn't look too difficult, but I need to check that it > faithfully mimics the capability we currently rely on. There have been > previous discussions on changing it, but this was rejected in favor of > allowing more flexibility. Here's part of one such conversation: > > http://lists.infradead.org/pipermail/linux-mtd/2012-August/043599.html > http://lists.infradead.org/pipermail/linux-mtd/2012-September/043825.html > http://lists.infradead.org/pipermail/linux-mtd/2012-December/045322.html > > So I would recommend: > (1) consider carefully the implications of your command-line format > now, rather than later > (2) if you want MTD to use it, it needs to support the features we use now It is fully functional reference MTD, :-). > > Some particular cases to consider: overlapping partitions (how do > block devices handle overlapping partitions?), out-of-order > specification, zero sized partitions, mixed syntax (some specified > with an offset, some not), multiple '-' partitions. I think the 'offset' just is used to hide some MTD space. There are two way: 1) redefine the 'offset' as a gap between forward partition and next partition. 2) add code forbid command line partitions overlapping and out-of-order. I recommend 1), it seems to solve those problem(overlapping and out-of-order), but it will affect habit. > > Anyway, if you resend, we can review. > > Thanks, > Brian
WARNING: multiple messages have this Message-ID (diff)
From: Caizhiyong <caizhiyong@huawei.com> To: Brian Norris <computersforpeace@gmail.com>, Andrew Morton <akpm@linux-foundation.org> Cc: "Wanglin \(Albert\)" <albert.wanglin@huawei.com>, Artem Bityutskiy <dedekind1@gmail.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Huang Shijie <b32955@freescale.com>, Karel Zak <kzak@redhat.com>, "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>, Shmulik Ladkani <shmulik.ladkani@gmail.com> Subject: RE: [PATCH] block: add command line partition parser Date: Thu, 15 Aug 2013 03:38:47 +0000 [thread overview] Message-ID: <C3050A4DBA34F345975765E43127F10F1C063B36@szxeml512-mbx.china.huawei.com> (raw) In-Reply-To: <CAN8TOE-aGcKQNayzEVQaS1wJ_Uqp3RAZ5pTuVEhaaY9f487-xA@mail.gmail.com> > -----Original Message----- > From: Brian Norris [mailto:computersforpeace@gmail.com] > Sent: Thursday, August 15, 2013 8:12 AM > To: Andrew Morton > Cc: Caizhiyong; Karel Zak; linux-mtd@lists.infradead.org; > linux-kernel@vger.kernel.org; Wanglin (Albert); Artem Bityutskiy; Shmulik Ladkani; > Huang Shijie > Subject: Re: [PATCH] block: add command line partition parser > > On Wed, Aug 14, 2013 at 3:57 PM, Andrew Morton > <akpm@linux-foundation.org> wrote: > > On Tue, 13 Aug 2013 06:02:17 +0000 Caizhiyong <caizhiyong@huawei.com> wrote: > > > >> move the command line parser to a separate module, and change it into > >> library-style code. > >> > >> reference: https://lkml.org/lkml/2013/8/6/550 > > The most recent patch is an addendum to this linked patch then? > > > Well OK. But to prove the library's usefulness and to generally clean > > up the kernel, someone needs to sign up to the task of converting > > drivers/mtd/cmdlinepart.c to use this code. > > > > I've been hopefully cc'ing various MTD people but am not being > > overwhelmed with waves of enthusiasm ;) > > "I've been" implies that you have done so prior to this email. And > "people" implies more than one person. I see that you CC'd David > Woodhouse over a week ago, but he's fairly silent these days on MTD > things. It's Artem or me who handle most of the day-to-day of MTD. And > this is the first time I've seen this! (BTW, please include > linux-mtd@lists.infradead.org for anything involving MTD.) > > This seems reasonable, and I'd be willing to work with this proposal. > > Caizhiyong, can you submit a clear single patch (or series of > patches), CC'd to linux-mtd at least? Then we can see about supporting > it in MTD. It doesn't look too difficult, but I need to check that it > faithfully mimics the capability we currently rely on. There have been > previous discussions on changing it, but this was rejected in favor of > allowing more flexibility. Here's part of one such conversation: > > http://lists.infradead.org/pipermail/linux-mtd/2012-August/043599.html > http://lists.infradead.org/pipermail/linux-mtd/2012-September/043825.html > http://lists.infradead.org/pipermail/linux-mtd/2012-December/045322.html > > So I would recommend: > (1) consider carefully the implications of your command-line format > now, rather than later > (2) if you want MTD to use it, it needs to support the features we use now It is fully functional reference MTD, :-). > > Some particular cases to consider: overlapping partitions (how do > block devices handle overlapping partitions?), out-of-order > specification, zero sized partitions, mixed syntax (some specified > with an offset, some not), multiple '-' partitions. I think the 'offset' just is used to hide some MTD space. There are two way: 1) redefine the 'offset' as a gap between forward partition and next partition. 2) add code forbid command line partitions overlapping and out-of-order. I recommend 1), it seems to solve those problem(overlapping and out-of-order), but it will affect habit. > > Anyway, if you resend, we can review. > > Thanks, > Brian
next prev parent reply other threads:[~2013-08-15 3:38 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-08-03 9:57 [PATCH] block: support embedded device command line partition Caizhiyong 2013-08-05 22:22 ` Andrew Morton 2013-08-06 10:53 ` Caizhiyong 2013-08-07 0:10 ` Andrew Morton 2013-08-13 6:02 ` [PATCH] block: add command line partition parser Caizhiyong 2013-08-14 22:57 ` Andrew Morton 2013-08-14 22:57 ` Andrew Morton 2013-08-15 0:11 ` Brian Norris 2013-08-15 0:11 ` Brian Norris 2013-08-15 0:30 ` Andrew Morton 2013-08-15 0:30 ` Andrew Morton 2013-08-15 3:38 ` Caizhiyong [this message] 2013-08-15 3:38 ` Caizhiyong 2013-08-15 5:00 ` Brian Norris 2013-08-15 5:00 ` Brian Norris 2013-08-15 6:16 ` Caizhiyong 2013-08-15 6:16 ` Caizhiyong 2013-08-15 7:09 ` Brian Norris 2013-08-15 7:09 ` Brian Norris 2013-08-15 7:45 ` Caizhiyong 2013-08-15 7:45 ` Caizhiyong 2013-08-15 8:32 ` Brian Norris 2013-08-15 8:32 ` Brian Norris 2013-08-15 15:52 ` [PATCH] block: support embedded device command line partition Stephen Warren 2013-08-16 2:54 ` Caizhiyong 2013-08-16 16:02 ` Stephen Warren 2013-08-19 8:36 ` Caizhiyong 2013-08-19 16:05 ` Stephen Warren 2013-09-17 11:15 ` Linus Walleij 2013-09-17 13:08 ` Caizhiyong
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=C3050A4DBA34F345975765E43127F10F1C063B36@szxeml512-mbx.china.huawei.com \ --to=caizhiyong@huawei.com \ --cc=akpm@linux-foundation.org \ --cc=albert.wanglin@huawei.com \ --cc=b32955@freescale.com \ --cc=computersforpeace@gmail.com \ --cc=dedekind1@gmail.com \ --cc=kzak@redhat.com \ --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.