linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
@ 2013-01-17 17:47 David Sterba
  2013-01-17 17:51 ` Chris Mason
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: David Sterba @ 2013-01-17 17:47 UTC (permalink / raw)
  To: chris.mason; +Cc: linux-btrfs

Hi Chris,

please pull a few more fixes to the 0.20-rc series. I have reviewed and
verified them where possible. There are the long awaited ARM fixes, help
updates and unsorted code fixes.

  git://repo.or.cz/btrfs-progs-unstable/devel.git for-chris

Based on top of master commit
91d9eec1ff044394f2b98ee7fcb76713dd33b994

Thanks.
david

---
 btrfs-list.c      |   69 +++++++++++++++++++++++++-----------------------------
 btrfsck.c         |    3 +-
 cmds-filesystem.c |   26 --------------------
 cmds-inspect.c    |    5 ++-
 cmds-receive.c    |    4 ++-
 cmds-scrub.c      |    4 ++-
 cmds-send.c       |    9 ++++++-
 convert.c         |    5 +++
 ctree.h           |    8 ++++--
 debug-tree.c      |   10 +++++++
 disk-io.c         |   13 ++++++++--
 kerncompat.h      |   12 +++++++++
 man/btrfs.8.in    |    4 ---
 mkfs.c            |   35 ++-------------------------
 restore.c         |    7 +++--
 utils.c           |   48 +++++++++++++++++++++++++++++++++++++
 utils.h           |    2 +
 version.sh        |    2 -
 volumes.c         |    8 +++---
 19 files changed, 159 insertions(+), 115 deletions(-)
---
Alex Lyakas (1):
      btrfs-progs: Receive: preserve ownership for symlinks, by using 'lchown'

Arne Jansen (1):
      Btrfs-progs: fix unaligned accesses

David Sterba (2):
      btrfs-progs: do not send stream into a terminal
      btrfs-progs: ignore -a option in mkfs

Goffredo Baroncelli (3):
      Move parse_size() to utils.[hc]
      parse_size(): replace atoll() with strtoull()
      parse_size(): add new suffixes

Kenji Okimoto (3):
      btrfs-progs: plug a memory leak reported by cppcheck
      btrfs-progs: check malloc() result
      btrfs-progs: plug memory leaks in btrfs_scan_one_dir() reported by cppcheck.

Lluis Batlle i Rossell (1):
      btrfs-progs: Fix getopt on arm platforms

Sergei Trofimovich (1):
      version.sh: fix version when built from tarball

Simon Xu (2):
      btrfs-progs: remove "device show" from btrfs man page
      btrfs-progs: remove '-h' from btrfs man page

Wade Cline (1):
      Btrfs-progs: Fix compiler warnings on PPC64

Wang Sheng-Hui (3):
      btrfs-progs: add -V description in print_usage
      btrfs-progs: update usage description for debug-tree.c
      btrfs-progs: print the right mount checking info in restore.c

Zach Brown (1):
      btrfs-progs: fix 32bit int/pointer cast warnings



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-17 17:47 [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1 David Sterba
@ 2013-01-17 17:51 ` Chris Mason
  2013-01-18 17:32   ` Goffredo Baroncelli
  2013-01-18 14:16 ` Gene Czarcinski
  2013-01-18 15:33 ` Gene Czarcinski
  2 siblings, 1 reply; 12+ messages in thread
From: Chris Mason @ 2013-01-17 17:51 UTC (permalink / raw)
  To: David Sterba; +Cc: Chris Mason, linux-btrfs@vger.kernel.org

On Thu, Jan 17, 2013 at 10:47:21AM -0700, David Sterba wrote:
> Hi Chris,
> 
> please pull a few more fixes to the 0.20-rc series. I have reviewed and
> verified them where possible. There are the long awaited ARM fixes, help
> updates and unsorted code fixes.
> 
>   git://repo.or.cz/btrfs-progs-unstable/devel.git for-chris
> 
> Based on top of master commit
> 91d9eec1ff044394f2b98ee7fcb76713dd33b994

Thanks Dave, pulling these in.

-chris

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-17 17:47 [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1 David Sterba
  2013-01-17 17:51 ` Chris Mason
@ 2013-01-18 14:16 ` Gene Czarcinski
  2013-01-18 14:44   ` David Sterba
  2013-01-18 15:33 ` Gene Czarcinski
  2 siblings, 1 reply; 12+ messages in thread
From: Gene Czarcinski @ 2013-01-18 14:16 UTC (permalink / raw)
  To: dsterba; +Cc: chris.mason, linux-btrfs

On 01/17/2013 12:47 PM, David Sterba wrote:
> Hi Chris,
>
> please pull a few more fixes to the 0.20-rc series. I have reviewed and
> verified them where possible. There are the long awaited ARM fixes, help
> updates and unsorted code fixes.
>
>    git://repo.or.cz/btrfs-progs-unstable/devel.git for-chris
>
> Based on top of master commit
> 91d9eec1ff044394f2b98ee7fcb76713dd33b994
>
> Thanks.
> david
>
> ---
>   btrfs-list.c      |   69 +++++++++++++++++++++++++-----------------------------
>   btrfsck.c         |    3 +-
>   cmds-filesystem.c |   26 --------------------
>   cmds-inspect.c    |    5 ++-
>   cmds-receive.c    |    4 ++-
>   cmds-scrub.c      |    4 ++-
>   cmds-send.c       |    9 ++++++-
>   convert.c         |    5 +++
>   ctree.h           |    8 ++++--
>   debug-tree.c      |   10 +++++++
>   disk-io.c         |   13 ++++++++--
>   kerncompat.h      |   12 +++++++++
>   man/btrfs.8.in    |    4 ---
>   mkfs.c            |   35 ++-------------------------
>   restore.c         |    7 +++--
>   utils.c           |   48 +++++++++++++++++++++++++++++++++++++
>   utils.h           |    2 +
>   version.sh        |    2 -
>   volumes.c         |    8 +++---
>   19 files changed, 159 insertions(+), 115 deletions(-)
> ---
> Alex Lyakas (1):
>        btrfs-progs: Receive: preserve ownership for symlinks, by using 'lchown'
>
> Arne Jansen (1):
>        Btrfs-progs: fix unaligned accesses
>
> David Sterba (2):
>        btrfs-progs: do not send stream into a terminal
>        btrfs-progs: ignore -a option in mkfs
>
> Goffredo Baroncelli (3):
>        Move parse_size() to utils.[hc]
>        parse_size(): replace atoll() with strtoull()
>        parse_size(): add new suffixes
>
> Kenji Okimoto (3):
>        btrfs-progs: plug a memory leak reported by cppcheck
>        btrfs-progs: check malloc() result
>        btrfs-progs: plug memory leaks in btrfs_scan_one_dir() reported by cppcheck.
>
> Lluis Batlle i Rossell (1):
>        btrfs-progs: Fix getopt on arm platforms
>
> Sergei Trofimovich (1):
>        version.sh: fix version when built from tarball
While you are about it, how about adding in this fix from Dieter Ries to 
fix version a little more:
http://article.gmane.org/gmane.comp.file-systems.btrfs/20069
>
> Simon Xu (2):
>        btrfs-progs: remove "device show" from btrfs man page
>        btrfs-progs: remove '-h' from btrfs man page
>
> Wade Cline (1):
>        Btrfs-progs: Fix compiler warnings on PPC64
>
> Wang Sheng-Hui (3):
>        btrfs-progs: add -V description in print_usage
>        btrfs-progs: update usage description for debug-tree.c
>        btrfs-progs: print the right mount checking info in restore.c
>
> Zach Brown (1):
>        btrfs-progs: fix 32bit int/pointer cast warnings
>
>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-18 14:16 ` Gene Czarcinski
@ 2013-01-18 14:44   ` David Sterba
  2013-01-18 15:49     ` Gene Czarcinski
  0 siblings, 1 reply; 12+ messages in thread
From: David Sterba @ 2013-01-18 14:44 UTC (permalink / raw)
  To: Gene Czarcinski; +Cc: dsterba, chris.mason, linux-btrfs

On Fri, Jan 18, 2013 at 09:16:00AM -0500, Gene Czarcinski wrote:
> >Sergei Trofimovich (1):
> >       version.sh: fix version when built from tarball
> While you are about it, how about adding in this fix from Dieter Ries to fix
> version a little more:
> http://article.gmane.org/gmane.comp.file-systems.btrfs/20069

I did not notice it before, but Dieter's patch does not work when progs
are built from .git repository

-echo "#define BTRFS_BUILD_VERSION \"Btrfs $v\"" >> .build-version.h
+echo "#define BTRFS_BUILD_VERSION \"$v\"" >> .build-version.h

$v is set with the git tag, so the "Btrfs" string does not appear in the
output (eg. mkfs.btrfs).

Technically, $v should always contain some sort of a tag, Sergei's patch
hardcodes it to the latest one for the non-git case. This is IMHO the
right approach.

david

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-17 17:47 [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1 David Sterba
  2013-01-17 17:51 ` Chris Mason
  2013-01-18 14:16 ` Gene Czarcinski
@ 2013-01-18 15:33 ` Gene Czarcinski
  2013-01-18 17:22   ` David Sterba
  2 siblings, 1 reply; 12+ messages in thread
From: Gene Czarcinski @ 2013-01-18 15:33 UTC (permalink / raw)
  To: dsterba; +Cc: chris.mason, linux-btrfs

On 01/17/2013 12:47 PM, David Sterba wrote:
> Hi Chris,
>
> please pull a few more fixes to the 0.20-rc series. I have reviewed and
> verified them where possible. There are the long awaited ARM fixes, help
> updates and unsorted code fixes.
>
>    git://repo.or.cz/btrfs-progs-unstable/devel.git for-chris
>
> Based on top of master commit
> 91d9eec1ff044394f2b98ee7fcb76713dd33b994
>
> Thanks.
> david
>
> ---
>   btrfs-list.c      |   69 +++++++++++++++++++++++++-----------------------------
>   btrfsck.c         |    3 +-
>   cmds-filesystem.c |   26 --------------------
>   cmds-inspect.c    |    5 ++-
>   cmds-receive.c    |    4 ++-
>   cmds-scrub.c      |    4 ++-
>   cmds-send.c       |    9 ++++++-
>   convert.c         |    5 +++
>   ctree.h           |    8 ++++--
>   debug-tree.c      |   10 +++++++
>   disk-io.c         |   13 ++++++++--
>   kerncompat.h      |   12 +++++++++
>   man/btrfs.8.in    |    4 ---
>   mkfs.c            |   35 ++-------------------------
>   restore.c         |    7 +++--
>   utils.c           |   48 +++++++++++++++++++++++++++++++++++++
>   utils.h           |    2 +
>   version.sh        |    2 -
>   volumes.c         |    8 +++---
>   19 files changed, 159 insertions(+), 115 deletions(-)
> ---
> Alex Lyakas (1):
>        btrfs-progs: Receive: preserve ownership for symlinks, by using 'lchown'
>
> Arne Jansen (1):
>        Btrfs-progs: fix unaligned accesses
>
> David Sterba (2):
>        btrfs-progs: do not send stream into a terminal
>        btrfs-progs: ignore -a option in mkfs
>
> Goffredo Baroncelli (3):
>        Move parse_size() to utils.[hc]
>        parse_size(): replace atoll() with strtoull()
>        parse_size(): add new suffixes
>
> Kenji Okimoto (3):
>        btrfs-progs: plug a memory leak reported by cppcheck
>        btrfs-progs: check malloc() result
>        btrfs-progs: plug memory leaks in btrfs_scan_one_dir() reported by cppcheck.
>
> Lluis Batlle i Rossell (1):
>        btrfs-progs: Fix getopt on arm platforms
>
> Sergei Trofimovich (1):
>        version.sh: fix version when built from tarball
>
> Simon Xu (2):
>        btrfs-progs: remove "device show" from btrfs man page
>        btrfs-progs: remove '-h' from btrfs man page
>
> Wade Cline (1):
>        Btrfs-progs: Fix compiler warnings on PPC64
>
> Wang Sheng-Hui (3):
>        btrfs-progs: add -V description in print_usage
>        btrfs-progs: update usage description for debug-tree.c
>        btrfs-progs: print the right mount checking info in restore.c
>
> Zach Brown (1):
>        btrfs-progs: fix 32bit int/pointer cast warnings
>

David, first of all let me thank you for pulling these together.  I do 
not believe I am alone in wondering what was happening to the 
btrfs-progs patches on the mailing list.

I would like to recommend some additional patches for inclusion since 
they are currently part of the Fedora 18 btrfs-tools package.

Josef Bacik:  detect if the disk we are formatting is a ssd V2
http://article.gmane.org/gmane.comp.file-systems.btrfs/18749

This one may be a little more of a problem since the suggested man 
update is missing and there is a second part for the kernel (status 
unknown).
Josef Bacik:  add btrfs device ready command
http://article.gmane.org/gmane.comp.file-systems.btrfs/18129
Since Fedora uses a "stock" a kernel as possible, I suspect the 
corresponding patch is applied.

There are two or three other patches that appear to make sense but I 
have no idea if they have been submitted.

Gene

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-18 14:44   ` David Sterba
@ 2013-01-18 15:49     ` Gene Czarcinski
  2013-01-18 15:55       ` Hugo Mills
  0 siblings, 1 reply; 12+ messages in thread
From: Gene Czarcinski @ 2013-01-18 15:49 UTC (permalink / raw)
  To: dsterba; +Cc: chris.mason, linux-btrfs

On 01/18/2013 09:44 AM, David Sterba wrote:
> On Fri, Jan 18, 2013 at 09:16:00AM -0500, Gene Czarcinski wrote:
>>> Sergei Trofimovich (1):
>>>        version.sh: fix version when built from tarball
>> While you are about it, how about adding in this fix from Dieter Ries to fix
>> version a little more:
>> http://article.gmane.org/gmane.comp.file-systems.btrfs/20069
> I did not notice it before, but Dieter's patch does not work when progs
> are built from .git repository
>
> -echo "#define BTRFS_BUILD_VERSION \"Btrfs $v\"" >> .build-version.h
> +echo "#define BTRFS_BUILD_VERSION \"$v\"" >> .build-version.h
>
> $v is set with the git tag, so the "Btrfs" string does not appear in the
> output (eg. mkfs.btrfs).
>
> Technically, $v should always contain some sort of a tag, Sergei's patch
> hardcodes it to the latest one for the non-git case. This is IMHO the
> right approach.
>
OK, I see what you mean.  However, the current situation when btrfs is 
built from a tarball results in:

 >btrfs  --version
Btrfs Btrfs v0.19

and

 >mkfs.btrfs  -V
mkfs.btrfs, part of Btrfs Btrfs v0.19

There must be a way to reliably just have one "Btrfs" and I will look 
into it.

Gene

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-18 15:49     ` Gene Czarcinski
@ 2013-01-18 15:55       ` Hugo Mills
  2013-01-18 16:48         ` Gene Czarcinski
  0 siblings, 1 reply; 12+ messages in thread
From: Hugo Mills @ 2013-01-18 15:55 UTC (permalink / raw)
  To: Gene Czarcinski; +Cc: dsterba, chris.mason, linux-btrfs

[-- Attachment #1: Type: text/plain, Size: 2060 bytes --]

On Fri, Jan 18, 2013 at 10:49:20AM -0500, Gene Czarcinski wrote:
> On 01/18/2013 09:44 AM, David Sterba wrote:
> >On Fri, Jan 18, 2013 at 09:16:00AM -0500, Gene Czarcinski wrote:
> >>>Sergei Trofimovich (1):
> >>>       version.sh: fix version when built from tarball
> >>While you are about it, how about adding in this fix from Dieter Ries to fix
> >>version a little more:
> >>http://article.gmane.org/gmane.comp.file-systems.btrfs/20069
> >I did not notice it before, but Dieter's patch does not work when progs
> >are built from .git repository
> >
> >-echo "#define BTRFS_BUILD_VERSION \"Btrfs $v\"" >> .build-version.h
> >+echo "#define BTRFS_BUILD_VERSION \"$v\"" >> .build-version.h
> >
> >$v is set with the git tag, so the "Btrfs" string does not appear in the
> >output (eg. mkfs.btrfs).
> >
> >Technically, $v should always contain some sort of a tag, Sergei's patch
> >hardcodes it to the latest one for the non-git case. This is IMHO the
> >right approach.
> >
> OK, I see what you mean.  However, the current situation when btrfs
> is built from a tarball results in:
> 
> >btrfs  --version
> Btrfs Btrfs v0.19

   I guess one problem here is that there's no canonical way of
generating a tarball (without the git files) from a git checkout, with
detailed version information. So what's happening, I suspect, is that
the distributions/people producing source tarballs of btrfs-progs are
checking out the git tree, deleting .git, and tarring up that, which
loses any git versioning.

   I think a solution may be to have a "dist" or "tar" make target
which does the above automatically and includes a suitably-generated
version.h in the tarball.

   Hugo.

> and
> 
> >mkfs.btrfs  -V
> mkfs.btrfs, part of Btrfs Btrfs v0.19
> 
> There must be a way to reliably just have one "Btrfs" and I will
> look into it.
> 
> Gene

-- 
=== Hugo Mills: hugo@... carfax.org.uk | darksatanic.net | lug.org.uk ===
  PGP key: 515C238D from wwwkeys.eu.pgp.net or http://www.carfax.org.uk
    --- Klytus, I'm bored. What plaything can you offer me today? ---    

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-18 15:55       ` Hugo Mills
@ 2013-01-18 16:48         ` Gene Czarcinski
  0 siblings, 0 replies; 12+ messages in thread
From: Gene Czarcinski @ 2013-01-18 16:48 UTC (permalink / raw)
  To: Hugo Mills, dsterba, chris.mason, linux-btrfs

On 01/18/2013 10:55 AM, Hugo Mills wrote:
> On Fri, Jan 18, 2013 at 10:49:20AM -0500, Gene Czarcinski wrote:
>> On 01/18/2013 09:44 AM, David Sterba wrote:
>>> On Fri, Jan 18, 2013 at 09:16:00AM -0500, Gene Czarcinski wrote:
>>>>> Sergei Trofimovich (1):
>>>>>        version.sh: fix version when built from tarball
>>>> While you are about it, how about adding in this fix from Dieter Ries to fix
>>>> version a little more:
>>>> http://article.gmane.org/gmane.comp.file-systems.btrfs/20069
>>> I did not notice it before, but Dieter's patch does not work when progs
>>> are built from .git repository
>>>
>>> -echo "#define BTRFS_BUILD_VERSION \"Btrfs $v\"" >> .build-version.h
>>> +echo "#define BTRFS_BUILD_VERSION \"$v\"" >> .build-version.h
>>>
>>> $v is set with the git tag, so the "Btrfs" string does not appear in the
>>> output (eg. mkfs.btrfs).
>>>
>>> Technically, $v should always contain some sort of a tag, Sergei's patch
>>> hardcodes it to the latest one for the non-git case. This is IMHO the
>>> right approach.
>>>
>> OK, I see what you mean.  However, the current situation when btrfs
>> is built from a tarball results in:
>>
>>> btrfs  --version
>> Btrfs Btrfs v0.19
>     I guess one problem here is that there's no canonical way of
> generating a tarball (without the git files) from a git checkout, with
> detailed version information. So what's happening, I suspect, is that
> the distributions/people producing source tarballs of btrfs-progs are
> checking out the git tree, deleting .git, and tarring up that, which
> loses any git versioning.
>
>     I think a solution may be to have a "dist" or "tar" make target
> which does the above automatically and includes a suitably-generated
> version.h in the tarball.
>
>     Hugo.
If you use gitweb to addess btrfs-progs.git, there is a very nice label 
"snapshot" beside each commit entry and clicking it will get you a 
tarball of the tree from that point.

I am a newbie at the git stuff but it seems to me that a few more 
branches are needed with stable branches marked as such and one or more 
unstable development branches ... at least that is what I have seen done 
on other projects such as NetworkManager and libvirt.  The branch will 
then include a hardcoded value for the version id which will/can be 
different in each branch.  And they also usually have a "make dist" 
which packages things up into a tarball.

The snapshot-tarball from gitweb includes the first part of the git 
commit id.  It is just that it is not included in a file.  The tarball 
does not includes only ".gitignore".  Rpms (and I assume the debian 
packages also) are based on tarball source with additional patches as 
necessary.  I have seen only one fedora package actually use git in the 
rpm and that was grub2.

While running stuff directly from a local git repository may be nice for 
developer, actual inclusion in a distribution package will (I believe) 
be via tarball.
>
>> and
>>
>>> mkfs.btrfs  -V
>> mkfs.btrfs, part of Btrfs Btrfs v0.19
>>
>> There must be a way to reliably just have one "Btrfs" and I will
>> look into it.
>>
As I was sitting here typing this message, the "solution" occurred to me:

Do not use Dieter's patch because it is unnecessary ... Sergei's patch 
does things correctly already.

Sorry for the noise on the list.  However, I do suggest that more git 
branches be used rather than having everything under "master".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-18 15:33 ` Gene Czarcinski
@ 2013-01-18 17:22   ` David Sterba
  2013-01-18 21:02     ` Gene Czarcinski
  0 siblings, 1 reply; 12+ messages in thread
From: David Sterba @ 2013-01-18 17:22 UTC (permalink / raw)
  To: Gene Czarcinski; +Cc: dsterba, chris.mason, linux-btrfs

On Fri, Jan 18, 2013 at 10:33:18AM -0500, Gene Czarcinski wrote:
> I would like to recommend some additional patches for inclusion since they
> are currently part of the Fedora 18 btrfs-tools package.

Thanks. For now I'd like to gather all the small bugfixes and updates
that are likely not to create any significant conflicts with whatever
Chris has in his tree and skip new features, UI changes or non-obvious
fixes.

> Josef Bacik:  detect if the disk we are formatting is a ssd V2
> http://article.gmane.org/gmane.comp.file-systems.btrfs/18749

This is IMO a safe feature, the comments were addressed in the final
patch so we can try to push it.

> This one may be a little more of a problem since the suggested man update is
> missing and there is a second part for the kernel (status unknown).
> Josef Bacik:  add btrfs device ready command
> http://article.gmane.org/gmane.comp.file-systems.btrfs/18129
> Since Fedora uses a "stock" a kernel as possible, I suspect the
> corresponding patch is applied.

The kernel part of 'device ready' was merged to 3.6.

> There are two or three other patches that appear to make sense but I have no
> idea if they have been submitted.

Please send them to the list.

david

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-17 17:51 ` Chris Mason
@ 2013-01-18 17:32   ` Goffredo Baroncelli
  0 siblings, 0 replies; 12+ messages in thread
From: Goffredo Baroncelli @ 2013-01-18 17:32 UTC (permalink / raw)
  To: Chris Mason, David Sterba, Chris Mason,
	linux-btrfs@vger.kernel.org

Hi Chris,
On 01/17/2013 06:51 PM, Chris Mason wrote:
> On Thu, Jan 17, 2013 at 10:47:21AM -0700, David Sterba wrote:
>> Hi Chris,
>>
[...]
>
> Thanks Dave, pulling these in.
what about my patches "Enhance the command btrfs filesystem df" (see my 
emails dated on 2012/11/02) [1] ?
You stated an interesting in these patches; however then I never heard 
anything...

Do you want  I resubmit ? I reached a general consensus ( == nobody was 
too unsatisfied :-) ) about the wording and the format.

And I have another question: it is possible to access to the RAID5/6 
develop branch. I am interesting to check how my patches work with 
RAID5/6...



>
> -chris

BR
G.Baroncelli


[1] http://permalink.gmane.org/gmane.comp.file-systems.btrfs/21071
-- 
gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it>
Key fingerprint BBF5 1610 0B64 DAC6 5F7D  17B2 0EDA 9B37 8B82 E0B5

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-18 17:22   ` David Sterba
@ 2013-01-18 21:02     ` Gene Czarcinski
  2013-01-21 10:50       ` David Sterba
  0 siblings, 1 reply; 12+ messages in thread
From: Gene Czarcinski @ 2013-01-18 21:02 UTC (permalink / raw)
  To: dsterba; +Cc: chris.mason, linux-btrfs, josef

On 01/18/2013 12:22 PM, David Sterba wrote:
> On Fri, Jan 18, 2013 at 10:33:18AM -0500, Gene Czarcinski wrote:
>> I would like to recommend some additional patches for inclusion since they
>> are currently part of the Fedora 18 btrfs-tools package.
> Thanks. For now I'd like to gather all the small bugfixes and updates
> that are likely not to create any significant conflicts with whatever
> Chris has in his tree and skip new features, UI changes or non-obvious
> fixes.
>
>> Josef Bacik:  detect if the disk we are formatting is a ssd V2
>> http://article.gmane.org/gmane.comp.file-systems.btrfs/18749
> This is IMO a safe feature, the comments were addressed in the final
> patch so we can try to push it.
>
>> This one may be a little more of a problem since the suggested man update is
>> missing and there is a second part for the kernel (status unknown).
>> Josef Bacik:  add btrfs device ready command
>> http://article.gmane.org/gmane.comp.file-systems.btrfs/18129
>> Since Fedora uses a "stock" a kernel as possible, I suspect the
>> corresponding patch is applied.
> The kernel part of 'device ready' was merged to 3.6.
>
>> There are two or three other patches that appear to make sense but I have no
>> idea if they have been submitted.
> Please send them to the list.
>
Josef Bacik <josef@toxicpanda.com> is the current Fedora maintainer as 
indicated in bugzilla.
<mailto:josef@toxicpanda.com>
Currently in Fedora 18, btrfs-progs is the "v0.20-rc1 with git commit id 
<mailto:josef@toxicpanda.com>91d9eec" plus a total of seven patches.  
Two of those patches from the list which I have already indicated.  The 
rest of those patches have been around for a while (at least since mid 
2011) and I cannot find any doc.

While I could send them in (and will if needed), I feel a little uneasy 
since Josef is responsible in Fedora.  While this may be a different 
individual since the email address is different, I suspect this Josef is 
the same Josef who wrote the other two patches and he may be able to 
speak to them better.

I have been scraping mailing list for all of the btrfs-progs patches and 
plan to gradually test them.  Since it appears you will be doing 
something similar, if you want to point me to some for testing, I will 
give it a shot with some virtual and some real-iron testing.  And 
sometimes it is also a matter of the time needed to refit/rebase patches.

Gene

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1
  2013-01-18 21:02     ` Gene Czarcinski
@ 2013-01-21 10:50       ` David Sterba
  0 siblings, 0 replies; 12+ messages in thread
From: David Sterba @ 2013-01-21 10:50 UTC (permalink / raw)
  To: Gene Czarcinski; +Cc: dsterba, chris.mason, linux-btrfs, josef

On Fri, Jan 18, 2013 at 04:02:09PM -0500, Gene Czarcinski wrote:
> >Please send them to the list.
> >
> Josef Bacik <josef@toxicpanda.com> is the current Fedora maintainer as
> indicated in bugzilla.
> <mailto:josef@toxicpanda.com>
> Currently in Fedora 18, btrfs-progs is the "v0.20-rc1 with git commit id
> <mailto:josef@toxicpanda.com>91d9eec" plus a total of seven patches.  Two of
> those patches from the list which I have already indicated.  The rest of
> those patches have been around for a while (at least since mid 2011) and I
> cannot find any doc.
> 
> While I could send them in (and will if needed), I feel a little uneasy
> since Josef is responsible in Fedora.  While this may be a different
> individual since the email address is different, I suspect this Josef is the
> same Josef who wrote the other two patches and he may be able to speak to
> them better.

Sorry, I somehow assumed you were speaking on behalf of Fedora.

> I have been scraping mailing list for all of the btrfs-progs patches and
> plan to gradually test them.  Since it appears you will be doing something
> similar, if you want to point me to some for testing, I will give it a shot
> with some virtual and some real-iron testing.  And sometimes it is also a
> matter of the time needed to refit/rebase patches.

Great, thanks! Testing of the userspace utilities is relatively safe
(compared to the kernel module) and I'm using using the tool compiled
from the master+patches on my regular testboxes. So far no disasterous
problems encountered and I can immediatelly see if something goes wrong
during normal use of the tools.

Hugo once started a regression testsuite that should eg. verify sanity
of the option parsing, so extending these tests would be welcome.

http://git.darksatanic.net/cgi/gitweb.cgi?p=btrfs-progs-unstable.git;a=commit;h=c18d54a5bab5f5e0c6dc3c8cf4a1cc073e82f876


david

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2013-01-21 10:50 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-17 17:47 [GIT PULL] btrfs-progs: more bugfixes for 0.20-rc1 David Sterba
2013-01-17 17:51 ` Chris Mason
2013-01-18 17:32   ` Goffredo Baroncelli
2013-01-18 14:16 ` Gene Czarcinski
2013-01-18 14:44   ` David Sterba
2013-01-18 15:49     ` Gene Czarcinski
2013-01-18 15:55       ` Hugo Mills
2013-01-18 16:48         ` Gene Czarcinski
2013-01-18 15:33 ` Gene Czarcinski
2013-01-18 17:22   ` David Sterba
2013-01-18 21:02     ` Gene Czarcinski
2013-01-21 10:50       ` David Sterba

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).