From: Jeff Layton
At this point, we know that "now" and the file times may differ, and we
suspect that the i_version has been flagged to be bumped. Attempt to
bump the i_version, and only mark the inode dirty if that actually
occurred or if one of the times was updated.
From: Jeff Layton
We only really need to update i_version if someone has queried for it
since we last incremented it. By doing that, we can avoid having to
update the inode if the times haven't changed.
If the times have changed, then we go ahead and forcibly increment the
From: Anand Jain
Now as the there is path in arg, so instead of reading the path from
cur_device just get it from the caller, and so the purpose of cur_device
is to skip the device, so rename it to skip_dev. Also drop the comment
about different path being used for the
From: Jeff Layton
Since i_version is mostly treated as an opaque value, we can exploit that
fact to avoid incrementing it when no one is watching. With that change,
we can avoid incrementing the counter on writes, unless someone has
queried for it since it was last
There is no need to check for btrfs_fs_devices::seeding when we
have checked for btrfs_fs_devices::opened, because we can't sprout
without its seed FS being opened.
Signed-off-by: Anand Jain
---
fs/btrfs/volumes.c | 2 --
1 file changed, 2 deletions(-)
diff --git
v2->v3:
@ 6/6:
add btrfs_free_stale_device() fn description, suggested by Nikolay
Fix line with longer than 80 char
v1->v2:
@ 6/6:
btrfs_device::name is null when we have missing device and
unmounted. So we still need to check for dev->name.
We can reuse the function
On 2018-01-09 03:33, Marat Khalili wrote:
On 08/01/18 19:34, Austin S. Hemmelgarn wrote:
A: While not strictly necessary, running regular filtered balances
(for example `btrfs balance start -dusage=50 -dlimit=2 -musage=50
-mlimit=4`, see `man btrfs-balance` for more info on what the options
On 2018-01-08 16:43, Tom Worster wrote:
On 01/08/2018 04:55 PM, Austin S. Hemmelgarn wrote:
On 2018-01-08 11:20, ein wrote:
> On 01/08/2018 04:55 PM, Austin S. Hemmelgarn wrote:
>
> > [...]
> >
> > And here's the FAQ entry:
> >
> > Q: Do I need to run a balance regularly?
> >
> > A: In
Gentle ping
2017-12-19 13:02 GMT+03:00 Timofey Titovets :
> 1st patch, remove 16MiB restriction from extent_same ioctl(),
> by doing iterations over passed range.
>
> I did not see much difference in performance, so it's just remove
> logic restriction.
>
> 2-3 pathes,
Pavel Raiskup wrote:
> On Tuesday, January 9, 2018 8:59:06 AM CET Paul Eggert wrote:
> > Pavel Raiskup wrote:
> > > So what about special casing that filesystem, where we can lseek() for
> > > holes anyway?
> >
> > If we can lseek for holes, then why not just do that?
>
>
Paul Eggert wrote:
> If we can lseek for holes, then why not just do that? We shouldn't need
> special-case code for btrfs per se. Any filesystem where we can lseek for
> holes
> should take advantage of that optimization.
This is what star uses since 13 years ;-)
Jörg
On Mon, Jan 8, 2018 at 10:39 PM, Jeff Layton wrote:
>>
>
> Got it, that's helpful. Does this patch help (on top of the others) ?
>
> 8<--
>
> SQUASH: nfs: compare raw iversion counter since that's what's
> being stored
>
Did
On 08/01/18 19:34, Austin S. Hemmelgarn wrote:
A: While not strictly necessary, running regular filtered balances
(for example `btrfs balance start -dusage=50 -dlimit=2 -musage=50
-mlimit=4`, see `man btrfs-balance` for more info on what the options
mean) can help keep a volume healthy by
On Tuesday, January 9, 2018 8:59:06 AM CET Paul Eggert wrote:
> Pavel Raiskup wrote:
> > So what about special casing that filesystem, where we can lseek() for
> > holes anyway?
>
> If we can lseek for holes, then why not just do that?
Checking whether lseek() actually works costs some
Pavel Raiskup wrote:
So
what about special casing that filesystem, where we can lseek() for holes
anyway?
If we can lseek for holes, then why not just do that? We shouldn't need
special-case code for btrfs per se. Any filesystem where we can lseek for holes
should take advantage of that
101 - 115 of 115 matches
Mail list logo