On mon, 16 Dec 2013 15:25:33 +0800, Liu Bo wrote:
Inode cache is similar to free space cache and in fact shares the same
code, however, we don't load inode cache unless we're about to allocate
inode id, then there is a case where we only commit the transaction during
other operations,
On Tue, Nov 19, 2013 at 10:29:35PM +, Filipe David Borba Manana wrote:
The inode eviction can be very slow, because during eviction we
tell the VFS to truncate all of the inode's pages. This results
in calls to btrfs_invalidatepage() which in turn does calls to
lock_extent_bits() and
Hans-Kristian Bakke posted on Mon, 16 Dec 2013 01:06:36 +0100 as
excerpted:
torrents are really only one thing my storage server get hammered with.
It also does a lot more IO intensive stuff. I actually run enterprise
storage drives in a Supermicro-server for a reason, even if it is my
home
On Mon, Dec 16, 2013 at 05:04:33PM +0800, Miao Xie wrote:
Onmon, 16 Dec 2013 15:25:33 +0800, Liu Bo wrote:
Inode cache is similar to free space cache and in fact shares the same
code, however, we don't load inode cache unless we're about to allocate
inode id, then there is a case where
Michael Welsh Duggan posted on Sun, 15 Dec 2013 22:40:55 -0500 as
excerpted:
I built the new btrfs-progs 3.12 recently. I note that the version
information doesn't seem to match this:
# ./btrfs --version Btrfs v0.20-rc1-358-g194aa4a
I see Wang Shilong dealt with the snapshot issue;
On Mon, 16 Dec 2013 18:26:02 +0800, Liu Bo wrote:
On Mon, Dec 16, 2013 at 05:04:33PM +0800, Miao Xie wrote:
On mon, 16 Dec 2013 15:25:33 +0800, Liu Bo wrote:
Inode cache is similar to free space cache and in fact shares the same
code, however, we don't load inode cache unless we're about to
Stupid me, I completely forgot that you can run multidisk arrays with
just block level partitions, just like with md raid! It will introduce
a rather significant management overhead in my case though, as
managing several individual partitions per drive is quite annoying
with so many drives.
What
On Mon, Dec 16, 2013 at 9:27 AM, Liu Bo bo.li@oracle.com wrote:
On Tue, Nov 19, 2013 at 10:29:35PM +, Filipe David Borba Manana wrote:
The inode eviction can be very slow, because during eviction we
tell the VFS to truncate all of the inode's pages. This results
in calls to
On Mon, Dec 16, 2013 at 11:05:31AM +, Filipe David Manana wrote:
On Mon, Dec 16, 2013 at 9:27 AM, Liu Bo bo.li@oracle.com wrote:
On Tue, Nov 19, 2013 at 10:29:35PM +, Filipe David Borba Manana wrote:
The inode eviction can be very slow, because during eviction we
tell the VFS to
On Mon, Dec 16, 2013 at 11:45 AM, Liu Bo bo.li@oracle.com wrote:
On Mon, Dec 16, 2013 at 11:05:31AM +, Filipe David Manana wrote:
On Mon, Dec 16, 2013 at 9:27 AM, Liu Bo bo.li@oracle.com wrote:
On Tue, Nov 19, 2013 at 10:29:35PM +, Filipe David Borba Manana wrote:
The inode
On Mon, Dec 16, 2013 at 11:48:08AM +, Filipe David Manana wrote:
On Mon, Dec 16, 2013 at 11:45 AM, Liu Bo bo.li@oracle.com wrote:
On Mon, Dec 16, 2013 at 11:05:31AM +, Filipe David Manana wrote:
On Mon, Dec 16, 2013 at 9:27 AM, Liu Bo bo.li@oracle.com wrote:
On Tue, Nov 19,
On Mon, Dec 16, 2013 at 11:57 AM, Liu Bo bo.li@oracle.com wrote:
On Mon, Dec 16, 2013 at 11:48:08AM +, Filipe David Manana wrote:
On Mon, Dec 16, 2013 at 11:45 AM, Liu Bo bo.li@oracle.com wrote:
On Mon, Dec 16, 2013 at 11:05:31AM +, Filipe David Manana wrote:
On Mon, Dec 16,
this patch will handle the strerror reporting of the error instead of
printing errno, and also replaced the BUG_ON with the error handling
Signed-off-by: Anand Jain anand.j...@oracle.com
---
cmds-device.c | 7 +++
cmds-replace.c | 10 --
mkfs.c | 9 -
utils.c
a clean up patch, the BTRFS_STRIPE_LEN is been duplicated across
btrfs_progs, the kernel defines it in volume.h so do the same
for progs.
Signed-off-by: Anand Jain anand.j...@oracle.com
---
btrfs-convert.c | 19 +--
chunk-recover.c | 1 -
cmds-chunk.c| 1 -
volumes.h
Signed-off-by: Anand Jain anand.j...@oracle.com
---
btrfs-convert.c | 2 +-
btrfs-image.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/btrfs-convert.c b/btrfs-convert.c
index 65fe707..df20c15 100644
--- a/btrfs-convert.c
+++ b/btrfs-convert.c
@@ -1715,7 +1715,7 @@
On Thu, Dec 12, 2013 at 1:38 AM, Chandra Seetharaman
sekha...@us.ibm.com wrote:
In order to handle a blocksize that is smaller than the
PAGE_SIZE, we need align all IOs to PAGE_SIZE.
This patch defines a new macro btrfs_align_size() that
calculates the alignment size based on the sectorsize
Hello Anand,
s/btrfs_progs/btrfs-progs?
more comments below…
this patch will handle the strerror reporting of the error instead of
printing errno, and also replaced the BUG_ON with the error handling
Signed-off-by: Anand Jain anand.j...@oracle.com
---
cmds-device.c | 7 +++
On Fri, Dec 13, 2013 at 3:07 AM, David Sterba dste...@suse.cz wrote:
On Wed, Dec 11, 2013 at 05:38:42PM -0600, Chandra Seetharaman wrote:
This is the final patch of the series that allows filesystems with
blocksize smaller than the PAGE_SIZE.
- if (sectorsize != PAGE_SIZE) {
You've
On Sun, Dec 15, 2013 at 10:03:35PM +0800, Shilong Wang wrote:
I compile btrfs in kernel 3.2 and 3.12.
# mkfs.btrfs -f /dev/sda2
# mount /dev/sda2 /mnt
# dd if=/dev/zero of=/mnt/data bs=1M count=1024
# umount /mnt
The above operations is in 3.2
and then i switch my system into 3.12,
Thanks, good cleanup, FYI I'll fold in the remaining cases from volumes.c
david
--
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
On Fri, Dec 13, 2013 at 07:39:34PM +, Filipe David Borba Manana wrote:
From the compiler:
fs/btrfs/file.c: In function ‘prepare_pages.isra.18’:
fs/btrfs/file.c:1265:6: warning: ‘err’ may be used uninitialized in this
function [-Wuninitialized]
My gcc 4.8.1 does not see this warning,
On Mon, Dec 16, 2013 at 02:33:11PM +0200, saeed bishara wrote:
On Thu, Dec 12, 2013 at 1:38 AM, Chandra Seetharaman
sekha...@us.ibm.com wrote:
In order to handle a blocksize that is smaller than the
PAGE_SIZE, we need align all IOs to PAGE_SIZE.
This patch defines a new macro
Hans-Kristian Bakke posted on Mon, 16 Dec 2013 11:55:40 +0100 as
excerpted:
Stupid me, I completely forgot that you can run multidisk arrays with
just block level partitions, just like with md raid! It will introduce a
rather significant management overhead in my case though, as managing
On Fri, Dec 13, 2013 at 06:09:39PM +0800, Miao Xie wrote:
Signed-off-by: Miao Xie mi...@cn.fujitsu.com
---
fs/btrfs/delayed-inode.c | 2 +-
fs/btrfs/transaction.c | 6 --
fs/btrfs/transaction.h | 2 --
3 files changed, 1 insertion(+), 9 deletions(-)
diff --git
On Mon, 2013-12-16 at 15:25 +0800, Liu Bo wrote:
Inode cache is similar to free space cache and in fact shares the same
code, however, we don't load inode cache unless we're about to allocate
inode id, then there is a case where we only commit the transaction during
other operations, such as
On Sun, 2013-12-15 at 03:35 +0100, Hans-Kristian Bakke wrote:
I have done some more testing. I turned off everything using the disk
and only did defrag. I have created a script that gives me a list of
the files with the most extents. I started from the top to improve the
fragmentation of the
On Fri, Dec 13, 2013 at 11:31:47AM -0500, Gene Czarcinski wrote:
On 12/13/2013 11:07 AM, Gene Czarcinski wrote:
I have hunted high and low for the information but, if it is out there,
it is well hidden.
What I want to do: I want to relocate (move) a subvolume from one BTRFS
volume to a
On Fri, Dec 13, 2013 at 09:51:42AM +0900, Tsutomu Itoh wrote:
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -4974,10 +4974,17 @@ static void btrfs_dentry_release(struct dentry
*dentry)
static struct dentry *btrfs_lookup(struct inode *dir, struct dentry *dentry,
The reference roots have to be read-only otherwise the results of send
stream could be undefined if the roots accidentaly change during the
process.
Signed-off-by: David Sterba dste...@suse.cz
---
cmds-send.c | 24
1 file changed, 24 insertions(+)
diff --git
- send accepts multiple subvolumes
- add missing option -e to man
- minor man formating fix
Signed-off-by: David Sterba dste...@suse.cz
---
cmds-send.c| 6 +++---
man/btrfs.8.in | 14 --
2 files changed, 11 insertions(+), 9 deletions(-)
diff --git a/cmds-send.c b/cmds-send.c
On Mon, 2013-12-16 at 14:32 +0200, saeed bishara wrote:
On Thu, Dec 12, 2013 at 1:38 AM, Chandra Seetharaman
sekha...@us.ibm.com wrote:
In order to handle multiple extent buffers per page, first we
need to create a way to handle all the extent buffers that
are attached to a page.
This
On Mon, 2013-12-16 at 15:48 +0100, David Sterba wrote:
On Mon, Dec 16, 2013 at 02:33:11PM +0200, saeed bishara wrote:
On Thu, Dec 12, 2013 at 1:38 AM, Chandra Seetharaman
sekha...@us.ibm.com wrote:
In order to handle a blocksize that is smaller than the
PAGE_SIZE, we need align all IOs
Ok, I guess the essence have been lost in the meta discussion.
Basically I get blocking for more than 120 seconds during these workloads:
- defragmenting several large fragmentet files in succession (leaving
time for btrfs to finish writing each file). This have *always*
happened in my array,
Unused since ed2590953bd06b892f0411fc94e19175d32f197a
Btrfs: stop using vfs_read in send.
Signed-off-by: David Sterba dste...@suse.cz
---
fs/btrfs/send.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c
index 2c1dcab7420d..95f0fae4ce59
First two patches are trivial cleanups that I've spotted while reading send.c,
can be applied independently.
The third patch contains the important bits, safety checks that the subvolumes
involved in send do not accidentally lose the RO status. I haven't seen this
documented anywhere that this is
All the subvolues that are involved in send must be read-only during the
whole operation. The ioctl SUBVOL_SETFLAGS could be used to change the
status to read-write and the result of send stream is undefined if the
data change unexpectedly.
Fix that by adding a refcount for all involved roots and
Remove ifdefed code:
- tlv_put for 8, 16 and 32, add a generic tempalte if needed in future
- tlv_put_timespec - the btrfs_timespec fields are used
- fs_path_remove obsoleted long ago
Signed-off-by: David Sterba dste...@suse.cz
---
fs/btrfs/send.c | 58
Thanks all for feedback, the changes sound ok to me. I'll send v4.
david
--
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
On Mon, Dec 16, 2013 at 2:34 PM, David Sterba dste...@suse.cz wrote:
On Fri, Dec 13, 2013 at 07:39:34PM +, Filipe David Borba Manana wrote:
From the compiler:
fs/btrfs/file.c: In function ‘prepare_pages.isra.18’:
fs/btrfs/file.c:1265:6: warning: ‘err’ may be used uninitialized in this
On Mon, 2013-12-16 at 17:32 +0100, Hans-Kristian Bakke wrote:
Ok, I guess the essence have been lost in the meta discussion.
Basically I get blocking for more than 120 seconds during these workloads:
- defragmenting several large fragmentet files in succession (leaving
time for btrfs to
I have explicitly set compression=lzo, and later noatime just to test
now, else it's just default 3.12.4 options (or 3.13-rc2 when I tested
that).
To make sure, here is my btrfs mounts from /proc/mounts:
/dev/sdl /btrfs btrfs rw,noatime,compress=lzo,space_cache 0 0
/dev/sdl /storage/storage-vol0
I need to create a fake tree to test qgroups and I don't want to have to setup a
fake btree_inode. The fact is we only use the radix tree for the fs_info, so
everybody else who allocates an extent_io_tree is just wasting the space anyway.
This patch moves the radix tree and its lock into
On Mon, 2013-12-16 at 19:22 +0100, Hans-Kristian Bakke wrote:
I have explicitly set compression=lzo, and later noatime just to test
now, else it's just default 3.12.4 options (or 3.13-rc2 when I tested
that).
To make sure, here is my btrfs mounts from /proc/mounts:
/dev/sdl /btrfs btrfs
No problem. You have to wait a bit though, as the volume is currently
going through a reduction in the number of drives from 8 to 7 and I do
not feel comfortable stalling the volume while that is happening. I
will report back with the logs later on.
Mvh
Hans-Kristian Bakke
On 16 December 2013
Hello guys,
I'm just using btrfs for the first time. I tried to create subvolumes
before installing Ubuntu but the installer doesn't give the
opportunity to configure as we want. So it creates by default 2
subvolumes : @ for the root and @home for the home directories.
So now, I wanted to delete
On Dec 16, 2013, at 4:20 PM, Nicolas Michel be.nicolas.mic...@gmail.com wrote:
So now, I wanted to delete the old subvolumes I created. I ran into
problems: the Ubuntu installer set the default subvolume to @ (id=5).
This is normal. id=0 maps to id=5
So when I try to delete my previous
OK. thanks for your pretty fast answer :)
Now my last question is: in this case it was easy as I know that I
created all these subvolumes as parts of volume 0. But in the btrfs
subv list / I don't see any information that tells me they belongs to
id 0. If I have to debug a server/desktop and I
On Dec 16, 2013, at 4:54 PM, Nicolas Michel be.nicolas.mic...@gmail.com wrote:
OK. thanks for your pretty fast answer :)
Now my last question is: in this case it was easy as I know that I
created all these subvolumes as parts of volume 0. But in the btrfs
subv list / I don't see any
On Mon, Dec 16, 2013 at 01:26:26PM -0500, Josef Bacik wrote:
I need to create a fake tree to test qgroups and I don't want to have to
setup a
fake btree_inode. The fact is we only use the radix tree for the fs_info, so
everybody else who allocates an extent_io_tree is just wasting the space
Hi, David,
On 2013/12/17 0:27, David Sterba wrote:
On Fri, Dec 13, 2013 at 09:51:42AM +0900, Tsutomu Itoh wrote:
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -4974,10 +4974,17 @@ static void btrfs_dentry_release(struct dentry *dentry)
static struct dentry *btrfs_lookup(struct inode *dir,
On Tue, 2013-12-17 at 02:06 +0100, David Sterba wrote:
On Mon, Dec 16, 2013 at 01:26:26PM -0500, Josef Bacik wrote:
I need to create a fake tree to test qgroups and I don't want to have to
setup a
fake btree_inode. The fact is we only use the radix tree for the fs_info,
so
everybody
Wang Shilong wangshilong1...@gmail.com writes:
Hello Michael,
I built the new btrfs-progs 3.12 recently. I note that the version
information doesn't seem to match this:
# ./btrfs --version
Btrfs v0.20-rc1-358-g194aa4a
Regardless, I was trying to use btrfs send (which worked in
On 12/17/2013 09:28 AM, Michael Welsh Duggan wrote:
Wang Shilong wangshilong1...@gmail.com writes:
Hello Michael,
I built the new btrfs-progs 3.12 recently. I note that the version
information doesn't seem to match this:
# ./btrfs --version
Btrfs v0.20-rc1-358-g194aa4a
Regardless,
a clean up patch, the BTRFS_STRIPE_LEN is been duplicated across
btrfs-progs, the kernel defines it in volume.h so do the same
for progs.
Signed-off-by: Anand Jain anand.j...@oracle.com
---
v2: commit update
---
btrfs-convert.c | 19 +--
chunk-recover.c | 1 -
cmds-chunk.c|
Signed-off-by: Anand Jain anand.j...@oracle.com
---
v2: commit update
---
btrfs-convert.c | 2 +-
btrfs-image.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/btrfs-convert.c b/btrfs-convert.c
index 65fe707..df20c15 100644
--- a/btrfs-convert.c
+++ b/btrfs-convert.c
@@
this patch will handle the strerror reporting of the error instead of
printing errno, and also replaced the BUG_ON with the error handling
Signed-off-by: Anand Jain anand.j...@oracle.com
---
v2: commit update
---
cmds-device.c | 7 +++
cmds-replace.c | 10 --
mkfs.c | 9
Thanks Wang.
s/btrfs_progs/btrfs-progs?
yeah. updated my script.
+
+zero_dev_error:
+if (ret) {
+ret 0 ?
+fprintf(stderr, ERROR: failed to zero device start '%s' -
%s\n,
+file, strerror(-ret)) :
+fprintf(stderr, ERROR:
We met the following oops when doing space balance:
kobject (88081b590278): tried to init an initialized object, something is
seriously wrong.
...
Call Trace:
[81937262] dump_stack+0x49/0x5f
[8137d259] kobject_init+0x89/0xa0
[8137d36a]
Leonidas Spyropoulos posted on Mon, 16 Dec 2013 23:22:54 + as
excerpted:
I assume there's performance degragation from having all the chunks
allocated in a volume. Is there a recomendation on how often once should
run the balance operation? If on the other hand no performance is
decreased
I'm getting these quite easily after upgrading to 3.13-rc4.
The server is no longer responsive and needs to be hard rebooted:
[ 822.391224] [ cut here ]
[ 822.391282] WARNING: CPU: 1 PID: 8115 at fs/btrfs/extent_io.c:219
free_extent_state.part.36+0x21/0x34 [btrfs]()
[
On Dec 16, 2013, at 10:55 PM, Tomasz Chmielewski t...@wpkg.org wrote:
I'm getting these quite easily after upgrading to 3.13-rc4.
[ 822.392976] CPU: 1 PID: 8115 Comm: rsync Not tainted 3.12.0-rc4 #1
Shows an older kernel. You might try 3.13.rc4 actual and see if it's
reproducible. 3.12.5
61 matches
Mail list logo