Hi Chris,
This fairly trivial patch missed the 3.4 merge window:
http://lkml.indiana.edu/hypermail/linux/kernel/1202.3/00764.html
Do you have it queued for the 3.5 merge window now in play, or
would you rather I push the changeset myself directly?
Thanks,
Paul.
--
To unsubscribe from this list:
On Fri, May 25, 2012 at 10:42 PM, Alexander Block
wrote:
> On Fri, May 25, 2012 at 10:27 PM, Peter Maloney
> wrote:
>> On 05/25/2012 09:10 PM, Alexander Block wrote:
>>> Just to show some numbers I made a simple test on a fresh btrfs fs. I
>>> copied my hosts /usr (4 gig) folder to that fs and ch
On Fri, May 25, 2012 at 10:27 PM, Peter Maloney
wrote:
> On 05/25/2012 09:10 PM, Alexander Block wrote:
>> Just to show some numbers I made a simple test on a fresh btrfs fs. I
>> copied my hosts /usr (4 gig) folder to that fs and checked metadata
>> usage with "btrfs fi df /mnt", which was around
On 05/25/12 19:49, Stefan Behrens wrote:
It would be helpful if already the generic block layer would offer
device error counters. Then btrfs could read them, add own counters for
its checksum detected errors, and store everything persistently in the
filesystem.
I take it that you not only cou
On 05/25/2012 09:10 PM, Alexander Block wrote:
> Just to show some numbers I made a simple test on a fresh btrfs fs. I
> copied my hosts /usr (4 gig) folder to that fs and checked metadata
> usage with "btrfs fi df /mnt", which was around 300m. Then I created
> 10 snapshots and checked metadata usa
On Fri, May 25, 2012 at 5:35 PM, Alexander Block
wrote:
> Hello,
>
> (this is a resend with proper CC for linux-fsdevel and linux-kernel)
>
> I would like to start a discussion on atime in Btrfs (and other
> filesystems with snapshot support).
>
> As atime is updated on every access of a file or d
It would be helpful if already the generic block layer would offer
device error counters. Then btrfs could read them, add own counters for
its checksum detected errors, and store everything persistently in the
filesystem.
The goal is to replace disks that have an increased error rate with
spa
On Fri, May 25, 2012 at 9:25 AM, Alexander Block
wrote:
> On Fri, May 25, 2012 at 5:15 PM, Alexander Block
> wrote:
>> Hello,
>>
>> I would like to start a discussion on atime in Btrfs (and possibly
>> other filesystems with snapshot support).
>>
>> As atime is updated on every access of a file o
On 2012-05-25, at 9:59, Alexander Block wrote:
> On Fri, May 25, 2012 at 5:42 PM, Josef Bacik wrote:
>> On Fri, May 25, 2012 at 05:35:37PM +0200, Alexander Block wrote:
>>> Hello,
>>>
>>> (this is a resend with proper CC for linux-fsdevel and linux-kernel)
>>>
>>> I would like to start a discu
On Fri, May 25, 2012 at 6:32 PM, Freddie Cash wrote:
>
> On May 25, 2012 9:00 AM, "Alexander Block" wrote:
>>
>> On Fri, May 25, 2012 at 5:42 PM, Josef Bacik wrote:
>> > On Fri, May 25, 2012 at 05:35:37PM +0200, Alexander Block wrote:
>> >> Hello,
>> >>
>> >> (this is a resend with proper CC for
On Fri, May 25, 2012 at 6:28 PM, Andreas Dilger wrote:
> On 2012-05-25, at 9:59, Alexander Block wrote:
>
> Are you talking about the atime for the primary copy, or the atime for the
> snapshots? IMHO, the atime should not be updated for a snapshot unless it is
> explicitly mounted r/w, or it
On Fri, May 25, 2012 at 5:42 PM, Josef Bacik wrote:
> On Fri, May 25, 2012 at 05:35:37PM +0200, Alexander Block wrote:
>> Hello,
>>
>> (this is a resend with proper CC for linux-fsdevel and linux-kernel)
>>
>> I would like to start a discussion on atime in Btrfs (and other
>> filesystems with snap
On Fri, May 25, 2012 at 05:35:37PM +0200, Alexander Block wrote:
> Hello,
>
> (this is a resend with proper CC for linux-fsdevel and linux-kernel)
>
> I would like to start a discussion on atime in Btrfs (and other
> filesystems with snapshot support).
>
> As atime is updated on every access of
Hello,
(this is a resend with proper CC for linux-fsdevel and linux-kernel)
I would like to start a discussion on atime in Btrfs (and other
filesystems with snapshot support).
As atime is updated on every access of a file or directory, we get
many changes to the trees in btrfs that as always tri
On Fri, May 25, 2012 at 5:15 PM, Alexander Block
wrote:
> Hello,
>
> I would like to start a discussion on atime in Btrfs (and possibly
> other filesystems with snapshot support).
>
> As atime is updated on every access of a file or directory, we get
> many changes to the trees in btrfs that as al
Can you explain why the device error counters should be in a filesystem
instead of generic block layer code?
On Fri, May 25, 2012 at 04:06:07PM +0200, Stefan Behrens wrote:
> Changes v1-v2:
> - Remove restriction that BTRFS_IOC_GET_DEVICE_STATS is a privileged
> operation
> - Cast u64 to unsigne
Hello,
I would like to start a discussion on atime in Btrfs (and possibly
other filesystems with snapshot support).
As atime is updated on every access of a file or directory, we get
many changes to the trees in btrfs that as always trigger cow
operations. This is no problem as long as the change
The linux-joystick mailing list has been superseded by the linux-input
list run at vger.kernel.org. See http://vger.kernel.org/vger-lists.html for
information on the new list server (or consult your local oracle).
Yours virtually,
Martin Mar
If cow_file_range_inline fails with ENOSPC we abort the transaction which
isn't very nice. This really shouldn't be happening anyways but there's no
sense in making it a horrible error when we can easily just go allocate
normal data space for this stuff. Thanks,
Signed-off-by: Josef Bacik
---
Ceph was hitting this race where we would remove an inode from the per-root
orphan list before we would release the space we had reserved for the inode.
We actually don't need a list or anything, we just need to make sure the
root doesn't try to free up the orphan reserve until after the inodes hav
Miao pointed this out while I was working on an orphan problem that messing
with a bitfield where different ranges are protected by different locks
doesn't work out right. Turns out we've been doing this forever where we
have different parts of the bit field protected by either no lock at all or
d
"btrfs device stats" is used to retrieve and print the device stats.
"btrfs device stats -z" is used to atomically retrieve, reset and
print the stats.
Signed-off-by: Stefan Behrens
---
cmds-device.c | 118
ctree.h|6 +++
ioc
Two convenient utility functions that have so far been local to scrub are
moved to utils.c.
They will be used in the device stats code in a following commit.
Signed-off-by: Stefan Behrens
---
cmds-scrub.c | 72 ++
utils.c | 66
This is a preparation step to add support for device stats. The definition
of the function open_file_or_dir() is moved from common.c to utils.c in
order to be able to share some common code between scrub and the device
stats in the following step. That common code uses open_file_or_dir().
Since ope
"btrfs device stats" is used to retrieve and print the device stats.
"btrfs device stats -z" is used to atomically retrieve, reset and
print the stats.
In order to share two utility functions between scrub and the dev stats
code, these two functions are moved to utils.c and renamed.
Since these fu
The goal is to detect when drives start to get an increased error rate,
when drives should be replaced soon. Therefore statistic counters are
added that count IO errors (read, write and flush). Additionally, the
software detected errors like checksum errors and corrupted blocks are
counted.
Signed
The device statistics are written into the device tree with each
transaction commit. Only modified statistics are written.
When a filesystem is mounted, the device statistics for each involved
device are read from the device tree and used to initialize the
counters.
Signed-off-by: Stefan Behrens
Changes v1-v2:
- Remove restriction that BTRFS_IOC_GET_DEVICE_STATS is a privileged
operation
- Cast u64 to unsigned long long for printf()
Changes v2-v3:
- Rebased on Chris' current master
Changes v3-v4:
- Add padding at end of ioctl structure
Changes v4-v5:
- The statistic members in the ioc
An ioctl interface is added to get the device statistic counters.
A second ioctl is added to atomically get and reset these counters.
Signed-off-by: Stefan Behrens
---
fs/btrfs/ioctl.c | 26 ++
fs/btrfs/ioctl.h | 14 ++
fs/btrfs/volumes.c | 34 ++
Before the update_time inode operation was indroduced, it was
not possible to prevent updates of atime on RO subvolumes.
btrfs_update_time does now check if the root is RO and skip
updating of atime.
This patch requires the update_time patches from Josef
Bacik.
Signed-off-by: Alexander Block
---
David just pointed me to the update_time patches from Josef. When these
get into the kernel, the RO snapshot + atime problem could be easily fixed.
Any idea when those get into the kernel?
On Thu, May 24, 2012 at 7:12 PM, Alexander Block
wrote:
> Hello,
>
> if a snapshot was created with -r and
31 matches
Mail list logo