Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-17 Thread Satoru Takeuchi
Hi Gui, (2014/06/18 7:21), David Rientjes wrote: On Fri, 13 Jun 2014, Gui Hecheng wrote: For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng Acked-by: David Rientjes It looks good to me.

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-17 Thread David Rientjes
On Fri, 13 Jun 2014, Gui Hecheng wrote: > For modern filesystems such as btrfs, t/p/e size level operations > are common. > add size unit t/p/e parsing to memparse > > Signed-off-by: Gui Hecheng Acked-by: David Rientjes Sorry, didn't do this before. Good luck with the simple_strto*() work!

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-17 Thread David Rientjes
On Fri, 13 Jun 2014, Gui Hecheng wrote: For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng guihc.f...@cn.fujitsu.com Acked-by: David Rientjes rient...@google.com Sorry, didn't do this before.

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-17 Thread Satoru Takeuchi
Hi Gui, (2014/06/18 7:21), David Rientjes wrote: On Fri, 13 Jun 2014, Gui Hecheng wrote: For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng guihc.f...@cn.fujitsu.com Acked-by: David Rientjes

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-16 Thread Gui Hecheng
On Mon, 2014-06-16 at 18:29 -0700, David Rientjes wrote: > On Tue, 17 Jun 2014, Gui Hecheng wrote: > > > > > diff --git a/lib/cmdline.c b/lib/cmdline.c > > > > index d4932f7..76a712e 100644 > > > > --- a/lib/cmdline.c > > > > +++ b/lib/cmdline.c > > > > @@ -121,11 +121,7 @@

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-16 Thread David Rientjes
On Tue, 17 Jun 2014, Gui Hecheng wrote: > > > diff --git a/lib/cmdline.c b/lib/cmdline.c > > > index d4932f7..76a712e 100644 > > > --- a/lib/cmdline.c > > > +++ b/lib/cmdline.c > > > @@ -121,11 +121,7 @@ EXPORT_SYMBOL(get_options); > > > * @retptr: (output) Optional pointer to next char

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-16 Thread Gui Hecheng
On Fri, 2014-06-13 at 14:58 -0700, David Rientjes wrote: > On Fri, 13 Jun 2014, Gui Hecheng wrote: > > > diff --git a/lib/cmdline.c b/lib/cmdline.c > > index d4932f7..76a712e 100644 > > --- a/lib/cmdline.c > > +++ b/lib/cmdline.c > > @@ -121,11 +121,7 @@ EXPORT_SYMBOL(get_options); > > *

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-16 Thread Gui Hecheng
On Fri, 2014-06-13 at 14:58 -0700, David Rientjes wrote: On Fri, 13 Jun 2014, Gui Hecheng wrote: diff --git a/lib/cmdline.c b/lib/cmdline.c index d4932f7..76a712e 100644 --- a/lib/cmdline.c +++ b/lib/cmdline.c @@ -121,11 +121,7 @@ EXPORT_SYMBOL(get_options); * @retptr: (output)

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-16 Thread David Rientjes
On Tue, 17 Jun 2014, Gui Hecheng wrote: diff --git a/lib/cmdline.c b/lib/cmdline.c index d4932f7..76a712e 100644 --- a/lib/cmdline.c +++ b/lib/cmdline.c @@ -121,11 +121,7 @@ EXPORT_SYMBOL(get_options); * @retptr: (output) Optional pointer to next char after parse

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-16 Thread Gui Hecheng
On Mon, 2014-06-16 at 18:29 -0700, David Rientjes wrote: On Tue, 17 Jun 2014, Gui Hecheng wrote: diff --git a/lib/cmdline.c b/lib/cmdline.c index d4932f7..76a712e 100644 --- a/lib/cmdline.c +++ b/lib/cmdline.c @@ -121,11 +121,7 @@ EXPORT_SYMBOL(get_options); *

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-13 Thread David Rientjes
On Fri, 13 Jun 2014, Gui Hecheng wrote: > diff --git a/lib/cmdline.c b/lib/cmdline.c > index d4932f7..76a712e 100644 > --- a/lib/cmdline.c > +++ b/lib/cmdline.c > @@ -121,11 +121,7 @@ EXPORT_SYMBOL(get_options); > * @retptr: (output) Optional pointer to next char after parse completes > * >

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-13 Thread Gui Hecheng
On Fri, 2014-06-13 at 07:55 +0200, Brendan Hide wrote: > On 13/06/14 03:42, Gui Hecheng wrote: > > For modern filesystems such as btrfs, t/p/e size level operations > > are common. > > add size unit t/p/e parsing to memparse > > > > Signed-off-by: Gui Hecheng > > --- > > changelog > > v1->v2:

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-13 Thread Brendan Hide
On 13/06/14 03:42, Gui Hecheng wrote: For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng --- changelog v1->v2: replace kilobyte with kibibyte, and others v2->v3: add missing unit

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-13 Thread Brendan Hide
On 13/06/14 03:42, Gui Hecheng wrote: For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng guihc.f...@cn.fujitsu.com --- changelog v1-v2: replace kilobyte with kibibyte, and others v2-v3:

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-13 Thread Gui Hecheng
On Fri, 2014-06-13 at 07:55 +0200, Brendan Hide wrote: On 13/06/14 03:42, Gui Hecheng wrote: For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng guihc.f...@cn.fujitsu.com --- changelog

Re: [PATCH v4] lib: add size unit t/p/e to memparse

2014-06-13 Thread David Rientjes
On Fri, 13 Jun 2014, Gui Hecheng wrote: diff --git a/lib/cmdline.c b/lib/cmdline.c index d4932f7..76a712e 100644 --- a/lib/cmdline.c +++ b/lib/cmdline.c @@ -121,11 +121,7 @@ EXPORT_SYMBOL(get_options); * @retptr: (output) Optional pointer to next char after parse completes * *

[PATCH v4] lib: add size unit t/p/e to memparse

2014-06-12 Thread Gui Hecheng
For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng --- changelog v1->v2: replace kilobyte with kibibyte, and others v2->v3: add missing unit "bytes" in comment v3->v4: remove

[PATCH v4] lib: add size unit t/p/e to memparse

2014-06-12 Thread Gui Hecheng
For modern filesystems such as btrfs, t/p/e size level operations are common. add size unit t/p/e parsing to memparse Signed-off-by: Gui Hecheng guihc.f...@cn.fujitsu.com --- changelog v1-v2: replace kilobyte with kibibyte, and others v2-v3: add missing unit bytes in comment