Re: [PATCH 1/3] dm: log writes target V2

2015-03-24 Thread Mike Snitzer
On Fri, Mar 20 2015 at 10:50am -0400,
Josef Bacik jba...@fb.com wrote:

 This creates a new target that is meant for file system developers to test 
 file
 system integrity at particular points in the life of a file system.  We 
 capture
 all write requests and the data and log the requests and the data to a 
 separate
 device for later replay.  There is a userspace utility to do this replay.  The
 idea behind this is to give file system developers to verify that the file
 system is always consistent.  Thanks,
 
 Signed-off-by: Josef Bacik jba...@fb.com

Hey Josef,

Nice job with this target, you need to contribute to DM more ;)

I've staged your target for the 4.1 merge in linux-dm.git's 'for-next'
branch (with a few small fixes for nits/typos).

FYI, I'll likely rebase 'for-next' once more for 4.1 given a conflicting
4.0 change to DM core still needs to land upstream (you can see the
merge conflict in 'for-next' at the moment).  But that won't impact your
target at all (other than changing the 4.1 commit id).

Thanks,
Mike
--
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


Re: [PATCH 1/3] dm: log writes target V2

2015-03-20 Thread Zach Brown
On Fri, Mar 20, 2015 at 10:50:37AM -0400, Josef Bacik wrote:
 This creates a new target that is meant for file system developers to test 
 file
 system integrity at particular points in the life of a file system.  We 
 capture
 all write requests and the data and log the requests and the data to a 
 separate
 device for later replay.  There is a userspace utility to do this replay.  The
 idea behind this is to give file system developers to verify that the file
 system is always consistent.  Thanks,
 
 Signed-off-by: Josef Bacik jba...@fb.com
 ---
 V1-V2: fixed up stuff based on Zachs review.

Cool,

Reviewed-by: Zach Brown z...@zabbo.net

- z
--
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