Re: [PATCH] btrfs: add btrfs resize unit t/p/e support

2014-03-27 Thread Brendan Hide

On 2014/03/27 04:51 AM, Gui Hecheng wrote:

[snip]

We add t/p/e support by replacing lib/cmdline.c:memparse
with btrfs_memparse. The btrfs_memparse copies memparse's code
and add unit t/p/e parsing.

Is there a conflict preventing adding this to memparse directly?

--
__
Brendan Hide
http://swiftspirit.co.za/
http://www.webafrica.co.za/?AFF1E97

--
To unsubscribe from this list: send the line unsubscribe linux-btrfs in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] btrfs: add btrfs resize unit t/p/e support

2014-03-27 Thread David Sterba
On Thu, Mar 27, 2014 at 09:35:41AM +0200, Brendan Hide wrote:
 On 2014/03/27 04:51 AM, Gui Hecheng wrote:
 [snip]
 
 We add t/p/e support by replacing lib/cmdline.c:memparse
 with btrfs_memparse. The btrfs_memparse copies memparse's code
 and add unit t/p/e parsing.
 Is there a conflict preventing adding this to memparse directly?

Agreed, there's no reason do duplicate this function.
--
To unsubscribe from this list: send the line unsubscribe linux-btrfs in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] btrfs: add btrfs resize unit t/p/e support

2014-03-27 Thread Gui Hecheng
On Thu, 2014-03-27 at 16:27 +0100, David Sterba wrote:
 On Thu, Mar 27, 2014 at 09:35:41AM +0200, Brendan Hide wrote:
  On 2014/03/27 04:51 AM, Gui Hecheng wrote:
  [snip]
  
  We add t/p/e support by replacing lib/cmdline.c:memparse
  with btrfs_memparse. The btrfs_memparse copies memparse's code
  and add unit t/p/e parsing.
  Is there a conflict preventing adding this to memparse directly?
 
 Agreed, there's no reason do duplicate this function.
Yes, I will try to modify the original memparse soon.

Thanks all!

-Gui

--
To unsubscribe from this list: send the line unsubscribe linux-btrfs in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html