Btrfs doesn't support FIEMAP_FLAG_XATTR, which is enabled by
-x option of filefrag, and will fail with
'FIBMAP: Invalid argument'
for 'filefrag -vx'. 'filefrag -vx' fails on btrfs with
'FIEMAP failed with unsupported flags 2'
Remove the '-x' option.
Signed-off-by: Wang Sheng-Hui
---
Something I've yet to repeat managed to leak a whole bunch of memory
while I was travelling, and locked up my workstation.
When I got home, this was the last thing printed out before it locked up
(it did make it into the logs thankfully) after a bunch of instances of
the oom-killers handywork.
On Wed, 27 Feb 2013 13:23:23 +1100
"Fajar A. Nugraha" wrote:
> Not to mention the hassle in accessing the data if it resides on a
> partition inside the file (e.g. you need losetup + kpartx to access it,
> and you must remember to do the reverse when you're finished with it).
>
> In zfsonlinux
Segmentation fault occurred in the following command.
# btrfs check /dev/sdc7
No valid Btrfs found on /dev/sdc7
Segmentation fault (core dumped)
Fix it.
Signed-off-by: Tsutomu Itoh
---
cmds-check.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/cmds-check.c b/cmds
On Tue, Feb 26, 2013 at 9:30 PM, Martin Steigerwald wrote:
> Am Dienstag, 26. Februar 2013 schrieb Fajar A. Nugraha:
>> On Tue, Feb 26, 2013 at 11:59 AM, Mike Fleetwood
>>
>> wrote:
>> > On 25 February 2013 23:35, Suman C wrote:
>> >> Hi,
>> >>
>> >> I think it would be great if there is a lvm v
On Sat, Feb 23, 2013 at 01:46:03AM +0100, Fredrik Tolf wrote:
> Dear list,
>
> I'm still in the process of transferring all the data I have to the
> btrfs filesystem I have had your help in debugging in a previous
> thread, and I have a slight question, if you will humour me.
>
> I have the data
On Tue, Feb 26, 2013 at 09:20:43AM -0500, Josef Bacik wrote:
> On Sun, Feb 24, 2013 at 07:55:46PM -0700, Marc MERLIN wrote:
> > Is this useful to anyone?
> >
> > Got this after a crash/reboot:
> >
> > if (block_rsv) {
> > WARN_ON(block_rsv->size > 0); <<
> >
The coverity runs had a false positive complaining that
save_ptr is uninitialized in the call to strtok_r.
Turns out that under the covers glibc was doing enough
to confuse the checker about what was being called.
Just to keep the noise down, do a harmless initialization,
with a comment as to why
Hey, I was looking at the wiki 'unclaimed projects' list and I thought I'd
take a look at what might be needed to use the swap-over-n{fs,bd}
functionality in order to implement swapfile support. It *looks* like it's
surprisingly uncomplicated - to the point where I suspect I'm missing
something
Am Dienstag, 26. Februar 2013 schrieb Goffredo Baroncelli:
> > And I teach not to in my trainings as well.
> >
> >
> >
> > Everyone who uses rm -rf by default even just for deleting a single
> > file does it as long as he or she deleted his / her home directory or
> > something.
>
> Unfortunatel
On 02/26/2013 09:17 PM, Eric Sandeen wrote:
> On 2/26/13 12:50 PM, Goffredo Baroncelli wrote:
>> On 02/25/2013 11:54 PM, Eric Sandeen wrote:
>>> cmds-qgroup.c contained a parse_limit() function which
>>> duplicates much of the functionality of parse_size.
>>> The only unique behavior is to handle "
On Tue, Feb 26, 2013 at 02:46:30PM -0600, Eric Sandeen wrote:
> On 2/26/13 2:40 PM, Ilya Dryomov wrote:
> > On Tue, Feb 26, 2013 at 02:20:30PM -0600, Eric Sandeen wrote:
> >> The coverity runs had a false positive complaining that save_ptr
> >> is uninitialized in the call to strtok_r.
> >>
> >> We
On 02/26/2013 09:10 PM, Eric Sandeen wrote:
> On 2/26/13 12:46 PM, Goffredo Baroncelli wrote:
>> Hi Eric,
>>
>> On 02/25/2013 11:54 PM, Eric Sandeen wrote:
>>> Rearrange cmd_subvol_set_default() slightly so we
>>> don't have to close the fd on an error return.
>>>
>>> While we're at it, fix whitesp
On 2/26/13 2:40 PM, Ilya Dryomov wrote:
> On Tue, Feb 26, 2013 at 02:20:30PM -0600, Eric Sandeen wrote:
>> The coverity runs had a false positive complaining that save_ptr
>> is uninitialized in the call to strtok_r.
>>
>> We could initialize it, but Zach points out that just using
>> strsep is a l
I added an "-f" option to mkfs.btrfs to force overwrite
of an existing filesystem. Now on an xfstests run, new
mkfs.btrfs requires it, and old mkfs.btrfs cannot accept
it.
So, add a helper which works out whether -f is needed,
and add it to the MKFS_BTRFS_PROG env. var as necessary,
so that it is
On Tue, Feb 26, 2013 at 02:20:30PM -0600, Eric Sandeen wrote:
> The coverity runs had a false positive complaining that save_ptr
> is uninitialized in the call to strtok_r.
>
> We could initialize it, but Zach points out that just using
> strsep is a lot simpler if there's only one delimiter,
> so
On 2/26/13 12:46 PM, Goffredo Baroncelli wrote:
> Hi Eric,
>
> On 02/25/2013 11:54 PM, Eric Sandeen wrote:
>> Rearrange cmd_subvol_set_default() slightly so we
>> don't have to close the fd on an error return.
>>
>> While we're at it, fix whitespace & remove magic
>> return values.
>>
>> Signed-of
On 2/20/13 9:37 AM, Stefan Behrens wrote:
...
> This means that it is now required to change all occurrences of
> "mkfs.btrfs" to "mkfs.btrfs -f" everywhere. Can't we first establish a
> time period of 100 years where the -f option is tolerated and ignored,
> and then in 2113 we require that the
The coverity runs had a false positive complaining that save_ptr
is uninitialized in the call to strtok_r.
We could initialize it, but Zach points out that just using
strsep is a lot simpler if there's only one delimiter,
so just switch to that.
Signed-off-by: Eric Sandeen
---
V2: Remove accide
On 2/26/13 12:50 PM, Goffredo Baroncelli wrote:
> On 02/25/2013 11:54 PM, Eric Sandeen wrote:
>> cmds-qgroup.c contained a parse_limit() function which
>> duplicates much of the functionality of parse_size.
>> The only unique behavior is to handle "none"; then we
>> can just pass it off to parse_si
On 2/26/13 12:47 PM, Goffredo Baroncelli wrote:
> On 02/25/2013 11:54 PM, Eric Sandeen wrote:
>> The coverity had a false positive complaining that save_ptr
>> is uninitialized in the call to strtok_r.
>>
>> We could initialize it, but Zach points out that just using
>> strsep is a lot simpler if t
> > The equivalent consistent update mechanism in btrfs is cow tree updates.
> > The superblock that references new tree blocks written to free space is
> > itself only written once all those blocks are stable on disk. If the
> > tree block writes are interrupted then the superblock isn't updated
Hi Anand,
On 02/25/2013 05:39 AM, Anand Jain wrote:
>
>
> You may need to update the minimum version of
> libblkid-dev that is required. Since.
>
> latest btrfs-progs needs
> blkid_probe_get_wholedisk_devno()
> from the libblkid-dev.
> found libblkid-dev version 2.17.2 does not work,
> and 2.
On 02/26/2013 11:37 AM, Martin Steigerwald wrote:
> Am Dienstag, 26. Februar 2013 schrieb Tsutomu Itoh:
Therefore I want you to revert
commit:2a2d8e1962e8b6cda7b0a7584f6d2fb95d442cb6.
btrfs-progs: require mkfs -f force option to overwrite filesystem
or partition table
>>
On Tue, Feb 26, 2013 at 10:24:42AM -0800, Zach Brown wrote:
> > Am I wrong when saying that ending up with replay journals that have
> > unexpected data and that can't be replayed is just inevitable and something
> > any journalling filesystem must deal with?
>
> If by journal you mean the btrfs l
> Sometimes these open-coded alignment is not so easy to understand for
> newbie like me.
Thanks, this is a nice change.
You might have mentioned in the commit message that some of the changes
aren't *strictly* equivalent but that they're still safe.
> - iosize = (iosize + blocksize
On 02/25/2013 11:54 PM, Eric Sandeen wrote:
> cmds-qgroup.c contained a parse_limit() function which
> duplicates much of the functionality of parse_size.
> The only unique behavior is to handle "none"; then we
> can just pass it off to parse_size().
>
> Signed-off-by: Eric Sandeen
> ---
> cmds-
On 02/25/2013 11:54 PM, Eric Sandeen wrote:
> The coverity had a false positive complaining that save_ptr
> is uninitialized in the call to strtok_r.
>
> We could initialize it, but Zach points out that just using
> strsep is a lot simpler if there's only one delimiter,
> so just switch to that.
>
Hi Eric,
On 02/25/2013 11:54 PM, Eric Sandeen wrote:
> Rearrange cmd_subvol_set_default() slightly so we
> don't have to close the fd on an error return.
>
> While we're at it, fix whitespace & remove magic
> return values.
>
> Signed-off-by: Eric Sandeen
> ---
> cmds-subvolume.c | 17 ++
> Am I wrong when saying that ending up with replay journals that have
> unexpected data and that can't be replayed is just inevitable and something
> any journalling filesystem must deal with?
If by journal you mean the btrfs log then yes, strictly speaking, you're
wrong. btrfs does deal with th
On Thu, Feb 07, 2013 at 03:38:34PM -0600, Mitch Harder wrote:
> @@ -361,6 +361,7 @@ static match_table_t tokens = {
> {Opt_check_integrity, "check_int"},
> {Opt_check_integrity_including_extent_data, "check_int_data"},
> {Opt_check_integrity_print_mask, "check_int_print_mask=%d"},
On Tue, Feb 26, 2013 at 09:23:00AM -0500, Josef Bacik wrote:
> So how did you reproduce it? I'll take a fs_image, but being able to
> reproduce
> the problem is more valuable. Thanks,
Here's the image: http://marc.merlins.org/tmp/fs_image
I just wrote details in the other message I just Cced y
On Tue, Feb 26, 2013 at 03:02:01PM +0800, Liu Bo wrote:
> On Sun, Feb 24, 2013 at 06:55:46PM -0800, Marc MERLIN wrote:
> > Is this useful to anyone?
> >
>
> Hi Marc,
>
> Thanks for the report, of course they're useful.
Thanks. I wasn't sure since I haven't seen the real problem of crashes
duri
On Tue, Feb 26, 2013 at 05:33:00AM -0700, David Sterba wrote:
> On Tue, Feb 26, 2013 at 08:11:17AM +0800, Liu Bo wrote:
> > On Mon, Feb 25, 2013 at 01:56:47PM -0500, Josef Bacik wrote:
> > > On Sun, Feb 24, 2013 at 09:04:42PM -0700, Liu Bo wrote:
> > > > Before we forced to change a file's NOCOW an
On Mon, Feb 25, 2013 at 11:51:02PM -0700, Marc MERLIN wrote:
> TL;DR;
> WARNING: at fs/btrfs/tree-log.c:1984 walk_down_log_tree+0x51/0x307()
> WARNING: at fs/btrfs/tree-log.c:1988 walk_down_log_tree+0x6c/0x307()
> kernel BUG at fs/btrfs/volumes.c:3753!
>
> It's way time for btrfs to stop crashing
On Sun, Feb 24, 2013 at 07:55:46PM -0700, Marc MERLIN wrote:
> Is this useful to anyone?
>
> Got this after a crash/reboot:
>
> if (block_rsv) {
> WARN_ON(block_rsv->size > 0); <<
> btrfs_free_block_rsv(root, block_rsv);
> }
>
Fixed i
Am Dienstag, 26. Februar 2013 schrieb Martin Steigerwald:
> Am Dienstag, 26. Februar 2013 schrieb Gareth Pye:
> > On Tue, Feb 26, 2013 at 10:09 PM, Martin Steigerwald
> >
> > wrote:
> > > I´d still like that for df, whose output is quite bogus in certain
> > > BTRFS setups at the moment and does
On 02/26/2013 12:04 PM, Zhi Yong Wu wrote:
HI,
It's a bit long so that i haven't read its whole, but i want to know
if it has any collision with my ongoing feature "btrfs hot
relocation/migration"?
It will utilize the hot track patch set you're been creating for VFS. I
think the methods are c
Am Dienstag, 26. Februar 2013 schrieb Gareth Pye:
> On Tue, Feb 26, 2013 at 10:09 PM, Martin Steigerwald
>
> wrote:
> > I´d still like that for df, whose output is quite bogus in certain
> > BTRFS setups at the moment and does not give applications a realistic
> > estimate at all. One example is
Am Samstag, 23. Februar 2013 schrieb Goffredo Baroncelli:
> Hi all,
Hi Goffredo,
> I updates my previous patches [1] to add support for raid5/6.
> These patches update the btrfs fi df command and add two new commands:
> - btrfs filesystem disk-usage
> - btrfs device disk-usage
Tested-By: Mart
On Tue, Feb 26, 2013 at 08:39:52AM +0900, Tsutomu Itoh wrote:
> >This means that it is now required to change all occurrences of
> >"mkfs.btrfs" to "mkfs.btrfs -f" everywhere. Can't we first establish a
>
> I also think so.
> It means -f is not significant to me, I think.
> (Most of my test script
On Tue, Feb 26, 2013 at 08:11:17AM +0800, Liu Bo wrote:
> On Mon, Feb 25, 2013 at 01:56:47PM -0500, Josef Bacik wrote:
> > On Sun, Feb 24, 2013 at 09:04:42PM -0700, Liu Bo wrote:
> > > Before we forced to change a file's NOCOW and COMPRESS flag due to
> > > the parent directory's, but this ends up
On Tue, Feb 26, 2013 at 10:09 PM, Martin Steigerwald
wrote:
> I´d still like that for df, whose output is quite bogus in certain BTRFS
> setups at the moment and does not give applications a realistic estimate at
> all. One example is raid 1 with 10 GB each disk. Shows 20 GB free. An
> application
Am Montag, 25. Februar 2013 schrieb Zach Brown:
> > I updates my previous patches [1] to add support for raid5/6.
> > These patches update the btrfs fi df command and add two new commands:
> > - btrfs filesystem disk-usage
> > - btrfs device disk-usage
>
> This seems like a ton of code.
>
> Her
HI,
It's a bit long so that i haven't read its whole, but i want to know
if it has any collision with my ongoing feature "btrfs hot
relocation/migration"?
On Thu, Feb 21, 2013 at 12:46 AM, Matias Bjorling wrote:
> Here is a short proposal for the hybrid storage cache idea with
> introduction/mot
Am Dienstag, 26. Februar 2013 schrieb Tsutomu Itoh:
> >> Therefore I want you to revert
> >> commit:2a2d8e1962e8b6cda7b0a7584f6d2fb95d442cb6.
> >>
> >>btrfs-progs: require mkfs -f force option to overwrite filesystem
> >>or partition table
> >>
> >> How do you think about it?
> >
> > What if y
Am Dienstag, 26. Februar 2013 schrieb Fajar A. Nugraha:
> On Tue, Feb 26, 2013 at 11:59 AM, Mike Fleetwood
>
> wrote:
> > On 25 February 2013 23:35, Suman C wrote:
> >> Hi,
> >>
> >> I think it would be great if there is a lvm volume or zfs zvol type
> >> support in btrfs.
> >
> > Btrfs alread
On 2013/02/26 16:05, Dave Chinner wrote:
On Tue, Feb 26, 2013 at 01:25:11PM +0900, Tsutomu Itoh wrote:
On 2013/02/26 13:06, Eric Sandeen wrote:
On 2/25/13 9:55 PM, Tsutomu Itoh wrote:
EXPERIMENTAL... It's certainly so.
However, I think that we should not add the option that it troubles
a lot o
Though most of the btrfs codes are using ALIGN macro for page alignment,
there are still some codes using open-coded alignment like the
following:
--
u64 mask = ((u64)root->stripesize - 1);
u64 ret = (val + mask) & ~mask;
--
Or even hidden one:
--
num_bytes = (en
49 matches
Mail list logo
