Re: [PATCH 2/3] Btrfs: add ioctl to get and reset the device stats

2011-12-11 Thread Goffredo Baroncelli
th root privileges. BR G.Baroncelli [...] -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord..

Re: [PATCH 0/3] Btrfs: add IO error device stats

2011-12-11 Thread Goffredo Baroncelli
| 26 > fs/btrfs/ioctl.h | 27 > fs/btrfs/print-tree.c |3 + > fs/btrfs/scrub.c | 52 ++-- > fs/btrfs/transaction.c |4 + > fs/btrfs/volumes.c | 335 > +++- fs/btrfs/volumes.h | > 43 +++

Re: Cloning a Btrfs partition

2011-12-08 Thread Goffredo Baroncelli
d be worried more about the uuid of the device than the filesystem one... -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the bo

Re: Resize command syntax wrong?

2011-11-30 Thread Goffredo Baroncelli
st... I have to resubmit another time BR G.Baroncelli [1] http://btrfs.ipv5.de/index.php?title=Btrfs(command) -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 -- To unsubscribe from this list: send the line &quo

Re: Resize command syntax wrong?

2011-11-30 Thread Goffredo Baroncelli
; one? This is a more simpler case As general rule: # btrfs device add # btrfs device delete May be that the device removing is blocked in some RAID setup. -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 -

Re: Grow btrfs partition & filesystem backwards

2011-11-09 Thread Goffredo Baroncelli
; More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > -- > > 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://vg

[PATCH 3/7][V2] btrfs-progs: Update the makefile for generating the help messages.

2011-11-08 Thread Goffredo Baroncelli
Update the makefile for generating the help messages. Signed off by: Goffredo Baroncelli --- Makefile | 17 - 1 files changed, 16 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 413f5d5..e9555b3 100644 --- a/Makefile +++ b/Makefile @@ -38,8 +38,9 @@ all

[PATCH 4/7][V2] btrfs-progs: Add the header/footer/introduction of the man page.

2011-11-08 Thread Goffredo Baroncelli
Add the header/footer/introduction of the man page. There is also an introduction to the syntax recognized by the tool helpextract to format the information. Signed off by: Goffredo Baroncelli --- btrfs.c | 173 +++ 1 files changed

[PATCH 5/7][V2] btrfs-progs: The man/btrfs.8.in man page is auto-generated

2011-11-08 Thread Goffredo Baroncelli
Warn that the man/btrfs.8.in man page is auto-generated Signed off by: Goffredo Baroncelli --- btrfs.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/btrfs.c b/btrfs.c index 26bd4f6..d6c840e 100644 --- a/btrfs.c +++ b/btrfs.c @@ -88,6 +88,11

[PATCH 6/7][V2] btrfs-progs: Add info for the commands.

2011-11-08 Thread Goffredo Baroncelli
Add info for every btrfs sub-commands in the sources. Signed off by: Goffredo Baroncelli --- btrfs_cmds.c | 254 ++ scrub.c | 79 ++ 2 files changed, 333 insertions(+), 0 deletions(-) diff --git a/btrfs_cmds.c b

[PATCH 7/7][V2] btrfs-progs: get-default man page

2011-11-08 Thread Goffredo Baroncelli
Adde the man page info for the "btrfs subvolume get-default" command Signed off by: Goffredo Baroncelli --- btrfs_cmds.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/btrfs_cmds.c b/btrfs_cmds.c index 517b0de..b2fff2f 100644 --- a/btrfs_cm

[PATCH 2/7][V2] btrfs-progs: Update the makefile for generating the man page.

2011-11-08 Thread Goffredo Baroncelli
The makefile is update in order to generate the man/btrfs.8.in by the helpextract tool on the basis of the sources comments. The old man page is renamed as btrfs.8.in.old. Signed off by: Goffredo Baroncelli --- Makefile | 11 ++- man/btrfs.8.in | 322

[PATCH 1/7][V2] btrfs-progs: Show the help messages from the info in the comment.

2011-11-08 Thread Goffredo Baroncelli
lli - The function "print_help" prints the help message extracted from the source. Signed off by: Goffredo Baroncelli --- btrfs.c | 34 +- 1 files changed, 29 insertions(+), 5 deletions(-) diff --git a/btrfs.c b/btrfs.c index 1def354..cc2f7ca

[BTRFS-PROG][PATCH 7/7] Test for man page

2011-11-02 Thread Goffredo Baroncelli
This test checks that the man page cites all the btrfs commands Comments are welcome. BR G.Baroncelli Signed-off-by: Goffredo Baroncelli --- test/003s.sh | 40 1 files changed, 40 insertions(+), 0 deletions(-) create mode 100755 test/003s.sh diff

[PATCH 4/7] get-default man page

2011-11-02 Thread Goffredo Baroncelli
Add the man page info for the "btrfs subvolume get-default" command. This command was not mentioned in the man page before. Comments are welcome. BR G.Baroncelli Signed-off-by: Goffredo Baroncelli --- btrfs_cmds.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletion

[BTRFS-PROG][PATCH 3/7] The man/btrfs.8.in man page is auto-generated

2011-11-02 Thread Goffredo Baroncelli
Warn that the man/btrfs.8.in man page is auto-generated Comments are welcome. BR G.Baroncelli Signed-off-by: Goffredo Baroncelli --- btrfs.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/btrfs.c b/btrfs.c index 4235d54..d7ece5b 100644 --- a/btrfs.c +++ b

[PATCH 6/7] qgroup/quota info for man page

2011-11-02 Thread Goffredo Baroncelli
merging this patch you can see this text in the man page doing: make man/btrfs.8.in man man/btrfs.8.in Don't hesitate to contact me for further information. Comments are welcome. BR G.Baroncelli Signed-off-by: Goffredo Baroncelli --- btrfs.c |

[BTRFS-PROGS][PATCH 5/7] btrfs filesystem restriper man page added

2011-11-02 Thread Goffredo Baroncelli
e to contact me for further information. Comments are welcome. BR G.Baroncelli Signed-off-by: Goffredo Baroncelli --- btrfs_cmds.c | 79 -- 1 files changed, 76 insertions(+), 3 deletions(-) diff --git a/btrfs_cmds.c b/btrfs_cmds.

[BTRFS-PROG][PATCH 2/7] helpextract.c: removed some static buffer

2011-11-02 Thread Goffredo Baroncelli
Removed some static buffer, and replaced by a dynamic memory allocation in order to support bigger text. Comments are welcome. BR G.Baroncelli Signed-off-by: Goffredo Baroncelli --- helpextract.c | 11 +++ 1 files changed, 7 insertions(+), 4 deletions(-) diff --git a/helpextract.c

[BTRFS-PROG][PATCH 1/7] Correct xstrip() function

2011-11-02 Thread Goffredo Baroncelli
cumentation with the code. These patches are base on the latest "Hugo Mills" integration branch. This patch correct a bug in the xstrip() function, which prevented to erase the unneeded spaces. Comments are welcome. BR G.Baroncelli Signed-off-by: Goffredo Baroncelli --- helpextra

Re: [RFC] [PROGS] Regression tests

2011-11-02 Thread Goffredo Baroncelli
avoid test name collision. Then we can group the number in range like: 000-100-> tested the mkfs command 100-200 -> tested the basic commands (like subvol *) [...] The idea is that a test xxx could use a command already tested in a test yyy (where xxx > yyy), knowing that he cou

Re: [RFC] [PROGS] Regression tests

2011-11-02 Thread Goffredo Baroncelli
avoid test name collision. Then we can group the number in range like: 000-100-> tested the mkfs command 100-200 -> tested the basic commands (like subvol *) [...] The idea is that a test xxx could use a command already tested in a test yyy (where xxx > yyy), knowing that he could

Re: Snapshot rollback

2011-10-23 Thread Goffredo Baroncelli
5 path __active/tmp/c ID 261 top level 5 path __active/tmp/a/b ghigo@venice:/tmp$ mv a/b c ghigo@venice:/tmp$ sudo btrfs sub list /var/btrfs/ ID 258 top level 5 path __active ID 259 top level 5 path __active/tmp/a ID 260 top level 5 path __active/tmp/c ID 261 top level 5 path __active/tmp/c/b -- gp

Re: snapshots changed behavior

2011-10-21 Thread Goffredo Baroncelli
e through all directories (not subvolumes) below it. I am > assuming that this is expected behavior. Thanks again for taking the > time to help me here. > Jim -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512

Re: snapshots changed behavior

2011-10-21 Thread Goffredo Baroncelli
linux-btrfs" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 -- To unsubscribe from

Avoid error message during a scan of a static /dev directory

2011-10-21 Thread Goffredo Baroncelli
.git branch skip-not-existent-devices -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512commit 3635fcf3f7358f006a5025b9773034088bdd01fd Author: Goffredo Baroncelli Date: Fri Oct 21 19:00:28 2011 +0200

[RFC] aptitude & BTRFS slowness

2011-10-19 Thread Goffredo Baroncelli
bvolume snapshot "$ACTIVE" "$ROLLBACK" || exit sync ( export LD_PRELOAD=/usr/lib/libeatmydata/libeatmydata.so aptitude ) echo "Syncing..." sync sleep 5s sync echo "Remove $ROLLBACK" btrfs subvolume delete "$ROLLBACK"

Re: [PATCH 1/2][BTRFS-PROGS] add the "--force" switch to the mkfs.btrfs command

2011-10-18 Thread Goffredo Baroncelli
y, 18 October, 2011 18:45:13 Goffredo Baroncelli wrote: > As requested by Mitch, I resend the patch related to the "--force" option > for the mkfs.btrfs command. > > This patch allow to bypass the check which ensure that you are formatting a > not mounted partition/disk

[PATCH 2/2][BTRFS-PROGS] add the "--force" switch to the mkfs.btrfs command

2011-10-18 Thread Goffredo Baroncelli
This patch update the man page -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512>From 82c11672909e5cc784d08d9361c69f0efb122ef1 Mon Sep 17 00:00:00 2001 From: Goffredo Baroncelli Date: Mon, 3 Jan 2011 19:53:05 +0

[PATCH 1/2][BTRFS-PROGS] add the "--force" switch to the mkfs.btrfs command

2011-10-18 Thread Goffredo Baroncelli
ularly when all > devices are not fully described in /dev). > > The --force option skips device checking, and forces formating on the > specified target. > > Thanks. -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E

Re: btrfs still looks for not existing devices

2011-10-14 Thread Goffredo Baroncelli
ord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs

Re: [RFC] btrfs send and receive

2011-08-10 Thread Goffredo Baroncelli
On Tuesday, 02 August, 2011 11:43:39 you wrote: > On 01.08.2011 20:51, Goffredo Baroncelli wrote: [...] > > 1) If we are interested to transport only the file > > type/contents/timestamps/acls/owners/permissions, that could be obtained > > with a combination of "find-new

Re: [RFC] btrfs send and receive

2011-08-10 Thread Goffredo Baroncelli
On Wednesday, 03 August, 2011 17:04:40 Jan Schmidt wrote: > On 02.08.2011 19:42, Goffredo Baroncelli wrote: > >> Furthermore, receiving should not need kernel support at all (except for > >> an optional interface to create a file with a certain inode, we'll see). >

Re: [RFC] btrfs send and receive

2011-08-02 Thread Goffredo Baroncelli
every snapshot, which should be unique. > > I'll think that over, currently I tend to adding these relationship keys > around btrfs_ioctl_snap_create soon, so we have at least some file > systems in the wild that are ready for send and receive once it's done. > > Thanks,

Re: [RFC] btrfs send and receive

2011-08-01 Thread Goffredo Baroncelli
Hi Jan On 08/01/2011 02:22 PM, Jan Schmidt wrote: > I furthermore realized that the term "subvolume" is omitted in favor > of the term "snapshot". This is because I tend to think of snapshots > being read-only (though I very much appreciate they are not). Just > replace the term wherever you feel

Re: [PATCH] btrfs: do not allow mounting non-subvolumes via subvol option

2011-07-29 Thread Goffredo Baroncelli
Hi David, On 07/29/2011 12:14 PM, David Sterba wrote: There's a missing test whether the path passed to subvol=path option during mount is a real subvolume, allowing any directory located in default subovlume to be passed and accepted for mount. (current btrfs progs prevent this early) $ btrfs

[PATCH 6/6] Update the makefile for generating the help messages.

2011-07-19 Thread Goffredo Baroncelli
From: Goffredo Baroncelli Update the makefile for generating the help messages. --- Makefile | 17 - 1 files changed, 16 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index dc1342f..7b25b47 100644 --- a/Makefile +++ b/Makefile @@ -37,8 +37,9 @@ all: version

[PATCH 5/6] Show the help messages from the info in the comment.

2011-07-19 Thread Goffredo Baroncelli
From: Goffredo Baroncelli The makefile is update in order to use the tool "helpextract" to extract the info from the sources comments and to generate the file "helpmsg.c" which contains an array of string with all the information. Then the function "print_help&

[PATCH 4/6] Update the makefile for generating the man page.

2011-07-19 Thread Goffredo Baroncelli
From: Goffredo Baroncelli The makefile is update in order to generate the man/btrfs.8.in by the helpextract tool on the basis of the sources comments. The old man page is renamed as btrfs.8.in.old. --- Makefile | 11 +- man/btrfs.8.in | 359

[PATCH 3/6] helpextract: tool to extract the info for the help from the source.

2011-07-19 Thread Goffredo Baroncelli
From: Goffredo Baroncelli It is created the file helpextract.c, which is the source for the tool "helpextract". This program extract the info showed in the man page and the help command from the sources comments. --- helpextrac

[PATCH 2/6] Add the header/footer/introduction of the man page.

2011-07-19 Thread Goffredo Baroncelli
From: Goffredo Baroncelli Add the header/footer/introduction of the man page. There is also an introduction to the syntax recognized by the tool helpextract to format the information. --- btrfs.c | 176 +++ 1 files changed, 176

[PATCH 1/6] Add info for the commands.

2011-07-19 Thread Goffredo Baroncelli
From: Goffredo Baroncelli Add info for every btrfs sub-commands in the sources. --- btrfs_cmds.c | 276 ++ scrub.c | 79 + 2 files changed, 355 insertions(+), 0 deletions(-) diff --git a/btrfs_cmds.c b/btrfs_cmds.c

[PATCH 0/6] Move the infor for the help/man page in the source

2011-07-19 Thread Goffredo Baroncelli
http://cassiopea.homelinux.net/git/btrfs-progs-unstable.git branch extended-help and/or browse the code at http://cassiopea.homelinux.net/git/?p=btrfs-progs-unstable.git BR G.Baroncelli --- Goffredo Baroncelli (6): Add info for the commands. Add the hea

Re: [PATCH 0/6] Series short description

2011-07-18 Thread Goffredo Baroncelli
Sorry, discard this email On 07/18/2011 07:33 PM, Goffredo Baroncelli wrote: > The following series implements... > > --- > > Goffredo Baroncelli (6): > Add info for the commands. > Add the header/footer/introduction of the man page. > helpextract:

[PATCH 0/6] Series short description

2011-07-18 Thread Goffredo Baroncelli
The following series implements... --- Goffredo Baroncelli (6): Add info for the commands. Add the header/footer/introduction of the man page. helpextract: tool to extract the info for the help from the source. Update the makefile for generating the man page. Show

Re: Mis-Design of Btrfs?

2011-07-14 Thread Goffredo Baroncelli
On 07/14/2011 08:38 AM, NeilBrown wrote: > On Thu, 14 Jul 2011 07:02:22 +0100 Ric Wheeler wrote: > >>> I'm certainly open to suggestions and collaboration. Do you have in mind >>> any >>> particular way to make the interface richer?? >>> >>> NeilBrown >> >> Hi Neil, >> >> I know that Chris has

Re: [PATCH 2/2] add detailed help messages to btrfs command

2011-07-11 Thread Goffredo Baroncelli
On 07/11/2011 09:11 PM, Jan Schmidt wrote: > On 07/11/2011 08:38 PM, Goffredo Baroncelli wrote: >> what about generating the man page on the basis of the btrfs help >> detailed messages ? >> >> My idea is the following: >> before the function source assoc

Re: [PATCH 2/2] add detailed help messages to btrfs command

2011-07-11 Thread Goffredo Baroncelli
Hi, all. what about generating the man page on the basis of the btrfs help detailed messages ? My idea is the following: before the function source associated to the command we can put a comment with a detailed help. The comment may be: [...] /*** man:btrfs subvolume create * * btrfs subvolum

R: [PATCH V2] Btrfs-progs: add "btrfs subvolume get-default" subcommand

2011-07-11 Thread Goffredo Baroncelli
>Messaggio originale >Da: xin.zh...@intel.com >Data: 11/07/2011 10.56 >A: >Cc: >Ogg: [PATCH V2] Btrfs-progs: add "btrfs subvolume get-default" subcommand > >Add subcommand to get the default subvolume of btrfs filesystem > >Reported-by: Yang, Yi >Signed-off-by: Zhong, Xin >--- > btrfs-

Re: Kernel Modules

2011-07-10 Thread Goffredo Baroncelli
On 07/09/2011 07:28 PM, cac...@quantum-sci.com wrote: > On Saturday 9 July, 2011 10:12:43 you wrote: >> If your btrfs lives on two or more devices you will have to run >> 'btrfs device scan' prior to mount or give all devices as arguments >> to mount.btrfs. > > Ohhh, I'd added a disk drive without

Re: [PATCH v1 0/2] Btrfs-progs: commands "resolve inode" and "resolve logical"

2011-07-08 Thread Goffredo Baroncelli
On 07/08/2011 10:24 AM, Jan Schmidt wrote: > On 08.07.2011 01:19, Goffredo Baroncelli wrote: >> On 07/07/2011 06:01 PM, Jan Schmidt wrote: >>> The kernel patch series just sent (Subject: "Btrfs: scrub: print path to >>> corrupted files and trigger nodatasum fixu

Re: [PATCH v1 0/2] Btrfs-progs: commands "resolve inode" and "resolve logical"

2011-07-07 Thread Goffredo Baroncelli
On 07/07/2011 06:01 PM, Jan Schmidt wrote: > The kernel patch series just sent (Subject: "Btrfs: scrub: print path to > corrupted files and trigger nodatasum fixup") introduces two new ioctls to > do in-kernel filesystem path construction. This series provides the > corresponding userspace changes,

Re: [PATCH v2] print parent ID in btrfs suvolume list

2011-07-02 Thread Goffredo Baroncelli
On 07/01/2011 12:21 PM, Andreas Philipp wrote: > There was some discussion on "where" subvolumes live in. Why do we not > simply print the parent ID for each subvolume in btrfs subvolume list? > This patch adds this functionality when called with parameter "-p". > Can I ask you to update the man

Re: subvolumes missing from "btrfs subvolume list" output

2011-06-29 Thread Goffredo Baroncelli
Hi Hugo On 06/29/2011 06:47 PM, Hugo Mills wrote: > On Wed, Jun 29, 2011 at 12:16:06PM -0400, Josef Bacik wrote: >> On 06/29/2011 11:00 AM, Stephane Chazelas wrote: >>> 2011-06-29 15:37:47 +0100, Stephane Chazelas: >>> [...] I found http://thread.gmane.org/gmane.comp.file-systems.btrfs/

Re: [RFC][BTRFS-PROG] Avoid to scan cdrom and floppy

2011-06-16 Thread Goffredo Baroncelli
Hi Helmut On 06/16/2011 05:09 PM, Helmut Hullen wrote: Hallo, Josef, Du meintest am 16.06.11: Who the hell doesn't use udev? Me - p.e. "udev" may be interesting for desktop users, for multimedia computers. It's not necessary for a simple server, for a machine where the administrator wants

[RFC][BTRFS-PROG] Avoid to scan cdrom and floppy

2011-06-15 Thread Goffredo Baroncelli
Hi all, thanks to the last Hugo's email, I restart to work on the patch which avoid to scan cdrom and floppy during a "btrfs filesystem show" and a "btrfs device scan". Comparing to my previous patch I modified the strategy, and now the scan of the devices is based on the content of the file

Re: [PATCH 2/5][REPOST][BTRFS-PROGS] Be more verbose when a ioctl returns an error

2011-06-13 Thread Goffredo Baroncelli
g a S-o-B, and is damaged with line-wrapping. If you > fix it up and re-send, I can pull into my integration branch so it > doesn't get forgotten... > >Hugo. > > On Sun, Dec 05, 2010 at 06:47:38PM +0100, Goffredo Baroncelli wrote: >> Hi all, >> >> this p

Re: [Validation] How to properly use the subvolume and snapshot for a backup method

2011-05-25 Thread Goffredo Baroncelli
On 05/25/2011 05:49 AM, Sylvain Alain wrote: > Hi everyone, first of all, I want to say that I'm learning how to use > BTRFS and I really enjoy all the subvolume and the snapshot features. > > Also, I'm a backup freak :P > > Basically, I have 4 questions : > > 1. I would like to know what is the

Re: "btrfs filesystem label" command

2011-05-12 Thread Goffredo Baroncelli
[*] http://git.kernel.org/?p=linux/kernel/git/mason/btrfs-progs-unstable.git;a=commit;h=d1dc6a9cff7e2fe4f335ca783a4b033457b3e184 > > -BJ > > > From: "Goffredo Baroncelli" To: "BJ Quinn" > Cc: linux-btrfs@vger.kernel.org Sent: Tuesday, May 10, > 201

Re: "btrfs filesystem label" command

2011-05-10 Thread Goffredo Baroncelli
Hi, I suppose that you are familiar with 'git' and the standard unix command (like cd, make...). The quick way is the following: $ cd /tmp $ git clone -b label \ http://cassiopea.homelinux.net/git/btrfs-progs-unstable.git $ cd btrfs-progs-unstable.git $ make $ ./btrfs Hoping that this m

Re: Cannot resize btrfs volume

2011-05-04 Thread Goffredo Baroncelli
Hi, several time ago I posted a patch which addressed this lack of documentation [1]. Unfortunately when I revised this patch I missed this chunk, and now Chris merged the last (uncompleted) revision. So now I am publish a new patch which address this issue. Chris, if you want you can pull it fr

Re: [PATCH 1/5] Add support for read-only subvolumes.

2011-04-25 Thread Goffredo Baroncelli
Hi Andreas, On 04/25/2011 03:47 PM, Andreas Philipp wrote: > Use BTRFS_IOC_CREATE_SNAP_V2 instead of BTRFS_IOC_CREATE_SNAP and add > an option for the creation of a readonly snapshot. > > Signed-off-by: Andreas Philipp > --- > btrfs_cmds.c | 44 >

Re: [PATCH] Btrfs: fix subvolume fake mount problem when default subvolume is set

2011-04-06 Thread Goffredo Baroncelli
Hi On 04/06/2011 11:34 AM, Zhong, Xin wrote: > We create two subvolumes (meego_root and meego_home) in > btrfs root directory. And set meego_root as default mount > subvolume. After we remount btrfs, meego_root is mounted > to top directory by default. Then if we create a directory with > the same

Re: btrfs subvolume snapshot syntax too "smart"

2011-04-04 Thread Goffredo Baroncelli
On 04/04/2011 09:09 PM, krz...@gmail.com wrote: > I understand btrfs intent but same command run twice should not give > diffrent results. This really makes snapshot automation hard > > > root@sv12 [/ssd]# btrfs subvolume snapshot /ssd/sub1 /ssd/5 > Create a snapshot of '/ssd/sub1' in '/ssd/5' >

Re: minor ID for btrfs-control

2011-04-03 Thread Goffredo Baroncelli
Hello Helmut, On 04/03/2011 05:11 AM, Helmut Hullen wrote: > Hallo, Goffredo, [..] >> As wrote by Tomasz the minor number of the btrfs-control is allocated >> dynamically. > > At which place? I replied you too fast. Btrfs register a "misc" device, but its number is not dynamically allocated (see

Re: minor ID for btrfs-control

2011-04-02 Thread Goffredo Baroncelli
On 04/02/2011 06:45 PM, Helmut Hullen wrote: > Hallo, > > if I've understood some mails in this list correct, then btrfs needs a > character device "btrfs-control" with the ID 10:55 > > When I look for this device in > > /sys/class/misc/btrfs-control/dev > > I see 10:234 > > And whe

Re: drives with more than 2 TByte

2011-03-24 Thread Goffredo Baroncelli
On 03/24/2011 05:43 PM, Helmut Hullen wrote: > Hallo, linux-btrfs, > > what about disks with more than 2 TByte? Other filesystems (?) need GPT. The filesystems don't care about the partition system. The 2TB limits is related to the maximum partition size. Of course a filesystem cannot be greater

Re: how to know when all space from a snapshot delete is freed?

2011-03-02 Thread Goffredo Baroncelli
David worked for some time to a solution to this kind of problem. But I don't know if it was merged. See posts on this ML with subject "PATCH: IOCTL#21 progs" On 03/02/2011 02:12 PM, Brian J. Murrell wrote: > For some time after I issue a snapshot delete, the space in the volume > is freed. It s

Re: Comparing snapshots?

2011-02-25 Thread Goffredo Baroncelli
On 02/25/2011 08:32 PM, João Eduardo Luís wrote: > Hello, > > Please note that my experience with btrfs is both recent and, above > all, very small. However, I've been wondering about the same issue > for a different purpose and your question intrigues me. > > However, and I may be off-base here,

Re: Comparing snapshots?

2011-02-25 Thread Goffredo Baroncelli
On 02/25/2011 10:59 AM, Arvin Schnell wrote: > Hi, > > for a backup program I have to find all differing files > (including metadata) in two snapshots taken from the same > subvolume. > > Having looked at the find-new command I thought about this > process: > > 1. Get the two transids when the t

Re: [PATCH] Btrfs-progs use safe string manipulation functions

2011-02-10 Thread Goffredo Baroncelli
On 02/10/2011 02:29 PM, Eduardo Silva wrote: > On Thu, 2011-02-10 at 12:39 +0100, Olaf van der Spek wrote: >> On Thu, Feb 10, 2011 at 12:37 PM, Jeremy Sanders >> wrote: >>> Olaf van der Spek wrote: >>> On Thu, Feb 10, 2011 at 12:08 PM, Thomas Bellman wrote: > strncpy(args.name, sour

Re: mkfs.btrfs - error checking /dev/sda5 mount status

2011-02-10 Thread Goffredo Baroncelli
On 02/09/2011 09:12 PM, Lubos Kolouch wrote: > Goffredo Baroncelli, Wed, 09 Feb 2011 19:25:34 +0100: > >> On 02/08/2011 10:26 PM, Lubos Kolouch wrote: >>> Goffredo Baroncelli, Tue, 08 Feb 2011 21:00:25 +0100: >>> >>>> On 02/08/2011 07:57 AM, Lubos Kolouc

Re: mkfs.btrfs - error checking /dev/sda5 mount status

2011-02-09 Thread Goffredo Baroncelli
On 02/08/2011 10:26 PM, Lubos Kolouch wrote: > Goffredo Baroncelli, Tue, 08 Feb 2011 21:00:25 +0100: > >> On 02/08/2011 07:57 AM, Lubos Kolouch wrote: >>> Hi, >>> >>> I'm hitting this issue - sda5 is a normal device, nothing to do with >>>

Re: mkfs.btrfs - error checking /dev/sda5 mount status

2011-02-08 Thread Goffredo Baroncelli
On 02/08/2011 07:57 AM, Lubos Kolouch wrote: > Hi, > > I'm hitting this issue - sda5 is a normal device, nothing to do with > loop, encryption etc. > > # mkfs.btrfs /dev/sda5 > > WARNING! - Btrfs v0.19-35-g1b444cd-dirty IS EXPERIMENTAL > WARNING! - see http://btrfs.wiki.kernel.org before using

Re: [PATCH] Btrfs-progs use safe string manipulation functions

2011-02-07 Thread Goffredo Baroncelli
On 02/07/2011 01:22 PM, Eduardo Silva wrote: > Please find the attached patch which replace unsafe strcpy(3) by > strncpy(3) functions. > > regards, > > Eduardo Silva Hi Eduardo, even though some "strncpy" are unneeded because a check is performed before, I fully agree that "strncpy" is bette

Re: [RFC][PATCH] Btrfs: New inode number allocator

2011-01-26 Thread Goffredo Baroncelli
On 01/26/2011 02:53 AM, Li Zefan wrote: > Here comes the compatability issue. It's fine to mount old btrfs, because > we'll just use the original way to find free ino. But we can't mount new btrfs > in older kernels, because the OFFSET makes highest objectid overflow when it > is cast to unsigned l

Re: Bug in mkfs.btrfs?!

2011-01-23 Thread Goffredo Baroncelli
On 01/23/2011 07:18 PM, Hugo Mills wrote: >Hi, Felix, > > On Sat, Jan 22, 2011 at 04:56:12PM +0100, Felix Blanke wrote: >> It was a simple: >> >> mkfs.btrfs -L backup -d single /dev/loop2 >> >> But it also happens without the options, like: >> >> mkfs.btrfs /dev/loop2 >> >> >> /dev/loop2 is a

Re: snapshot strange behaviour

2011-01-23 Thread Goffredo Baroncelli
On 01/23/2011 04:05 PM, Lubos Kolouch wrote: > Goffredo Baroncelli, Sun, 23 Jan 2011 13:17:13 +0100: > >> Hi Lubos, >> >> On 01/23/2011 08:17 AM, Lubos Kolouch wrote: >>> Hello, >>> Is this a bug or intended behaviour and I am missing something

Re: [PATCH 2/2] add detailed help messages to btrfs command

2011-01-23 Thread Goffredo Baroncelli
/2011 03:28 PM, Hubert Kario wrote: > On Sunday 23 of January 2011 15:07:09 Goffredo Baroncelli wrote: >> Hello Hubert, >> >> please update the man page too. > > I started to do it but then I noticed that you have made a few changes to the > man page yourself. >

Re: [PATCH 2/2] add detailed help messages to btrfs command

2011-01-23 Thread Goffredo Baroncelli
Hello Hubert, please update the man page too. Regards G.Baroncelli On 01/23/2011 01:42 PM, Hubert Kario wrote: > extend the > > btrfs --help > > command to print detailed help message if available but fallback to > basic help message if detailed is unavailable > > add detailed help m

Re: snapshot strange behaviour

2011-01-23 Thread Goffredo Baroncelli
Hi Lubos, On 01/23/2011 08:17 AM, Lubos Kolouch wrote: > Hello, > > During doing backups I found strange behaviour... 2.6.37, latest btrfs- > progs from git > > nbgentoo ~ # btrfs subv crea a > Create subvolume './a' > nbgentoo ~ # cd a > nbgentoo a # btrfs subv crea b > Create subvolume './b' >

Re: Backup Command

2011-01-21 Thread Goffredo Baroncelli
On 01/21/2011 08:07 PM, cac...@quantum-sci.com wrote: > > Well thanks to some help from you guys I seem to have my backup server almost > fully running and functional with rsync. Amazing functions, this > snapshotting and rsync. > > I still don't know why I cannot remove snapshots though. (Deb

Re: Adding a disk fails

2011-01-21 Thread Goffredo Baroncelli
On 01/21/2011 12:10 AM, Carl Cook wrote: > On Thu 20 January 2011 14:13:22 Goffredo Baroncelli wrote: >> To add another disk you don't have to run mkfs.btrfs. For example: >> >> # add the first disk >> mkfs.btrfs /dev/sdb >> # mount the disk >> mount

Re: Adding a disk fails

2011-01-20 Thread Goffredo Baroncelli
On 01/20/2011 10:18 PM, Carl Cook wrote: > > Well I've just tried to add a disk to another, but it fails. I > created the first (starting with no traditional partitions) with: # > mkfs.btrfs /dev/sdb > > Then I mounted it to /media/backups and put lots of files on it. I > shut down the system,

Re: [GIT PULL] Btrfs updates

2011-01-18 Thread Goffredo Baroncelli
Hi Chris, there are a lot of patches related to the btrfs-progs[*]. Do you think that it is reasonable that you find a bit of time to care about that ? Regards G.Baroncelli [*] For example see my email "[REPOST][BTRFS-PROGS] Repost of some patches" dated 2010-12-05. On 01/18/2011 11:14 AM, Feli

Re: Synching a Backup Server

2011-01-06 Thread Goffredo Baroncelli
On 01/06/2011 06:35 PM, Carl Cook wrote: > I want to keep a duplicate copy of the HTPC data, on the backup > server, and I think a regular full file copy is not optimal and may > take days to do. So I'm looking for a way to sync the arrays at some > interval. Ideally the sync would scan the HTPC

[PATCH] add a --force option to mkfs.btrf [was Re: Odd mkbtrfs behavior inside of chroot]

2011-01-03 Thread Goffredo Baroncelli
me Regards G.Baroncelli - -- gpg key: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 On 01/03/2011 12:14 AM, Goffredo Baroncelli wrote: > On 01/02/2011 08:52 PM, J G wrote: >> I just encountered some odd behavior from mkbtrfs. &g

Re: Odd mkbtrfs behavior inside of chroot

2011-01-02 Thread Goffredo Baroncelli
On 01/02/2011 08:52 PM, J G wrote: > I just encountered some odd behavior from mkbtrfs. > The end goal is to restore a backup to newly created BTRFS partitions while > using the latest btrfs-tools. > Here's the steps to what I did: > * Booted SystemRescueCD > * Partitioned the drives (two 750GB d

Re: [TRIVIAL][PATCH] Improve error handling in the btrfs command

2010-12-20 Thread Goffredo Baroncelli
On Monday, 20 December, 2010, you (Chris Samuel) wrote: > On 21/12/10 07:06, Goffredo Baroncelli wrote: > > > below is enclosed a trivial patch, which has the aim to > > improve the error reporting of the "btrfs" command. > > Any reason to not just use perror()

Re: [PATCH] Improve error handling in filesystem df

2010-12-20 Thread Goffredo Baroncelli
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 > -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 051

[TRIVIAL][PATCH] Improve error handling in the btrfs command

2010-12-20 Thread Goffredo Baroncelli
list.c | 40 ++ btrfs_cmds.c | 77 - utils.c |6 3 files changed, 89 insertions(+), 34 deletions(-) -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054 BF04 F161 3DC5 0512 diff

[BUG] Calling BTRFS_IOC_TREE_SEARCH for a non existant key

2010-12-19 Thread Goffredo Baroncelli
if this is a workaround or the root cause is inside the function btrfs_search_forward() that doesn't check the key against max_key if cached_only is 0. Regards G.Baroncelli -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E

Re: What do the arguments of btrfs filesystem defragment do?

2010-12-15 Thread Goffredo Baroncelli
gth of defrag range (if len is <= 0, the rest of file is considered) -c -> force compress -t -> threshold for compression (if the size of file is bigger, compress the file ?) -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054

Re: Bug in the design of the tree search ioctl API ? [was Re: [PATCH 1/3] Btrfs: Really return keys within specified range]

2010-12-15 Thread Goffredo Baroncelli
application which depends on it. So we may be obligated to maintain it .. Goffredo > > -chris > -- > 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.

Re: Bug in the design of the tree search ioctl API ? [was Re: [PATCH 1/3] Btrfs: Really return keys within specified range]

2010-12-15 Thread Goffredo Baroncelli
On Wednesday, 15 December, 2010, Li Zefan wrote: > Goffredo Baroncelli wrote: > > On Wednesday, 15 December, 2010, Li Zefan wrote: > >> h4) objectid asis, type asis, offset++ -> we should get the correct result. > > > > This fix the problem of the "miss

Re: Bug in the design of the tree search ioctl API ? [was Re: [PATCH 1/3] Btrfs: Really return keys within specified range]

2010-12-15 Thread Goffredo Baroncelli
API. Of course, that doesn't means that we can remove the old API ignoring the backward compatibility. But I think that there are sufficient pros to develop a new API Please be patient: my english is very bad; I am not trying to blame anybody; I want only a "perfect fs" (TM) :-) > &g

Re: Bug in the design of the tree search ioctl API ? [was Re: [PATCH 1/3] Btrfs: Really return keys within specified range]

2010-12-14 Thread Goffredo Baroncelli
jectid < (u64)-1) { > sk->min_objectid++; > sk->min_type = BTRFS_ROOT_BACKREF_KEY; > sk->min_offset = 0; > -- gpg key@ keyserver.linux.it: Goffredo Baroncelli (ghigo) Key fingerprint = 4769 7E51 5293 D36C 814E C054

[PATCH] BTRFS_IOC_TREE_SEARCH: store and use the last key found

2010-12-14 Thread Goffredo Baroncelli
BTRFS_IOCTL_MAGIC, 17, \ + +/* buggy, don't use */ + +#define BTRFS_IOC_TREE_SEARCH_V0 _IOWR(BTRFS_IOCTL_MAGIC, 17, \ + struct btrfs_ioctl_search_args) +#define BTRFS_IOC_TREE_SEARCH_V1 _IOWR(BTRFS_IOCTL_MAGIC, 25, \ struct btr

Re: Bug in the design of the tree search ioctl API ? [was Re: [PATCH 1/3] Btrfs: Really return keys within specified range]

2010-12-14 Thread Goffredo Baroncelli
On Tuesday, 14 December, 2010, Li Zefan wrote: > Goffredo Baroncelli wrote: > > Hi Li, > > > > On Monday, 13 December, 2010, Li Zefan wrote: > >> The keys returned by tree search ioctl should be restricted to: > >> > >>key.objectid = [m

Bug in the design of the tree search ioctl API ? [was Re: [PATCH 1/3] Btrfs: Really return keys within specified range]

2010-12-13 Thread Goffredo Baroncelli
s_comp_cpu_keys(key, &test); > - if (ret > 0) > + if (key->offset < sk->min_offset || key->offset > sk->max_offset) > + return 0; > + if (key->objectid < sk->min_objectid || key->objectid > sk- >max_objectid) > retu

<    5   6   7   8   9   10   11   12   >