Re: [RFC] fsblock

2007-06-27 Thread Chris Mason
On Wed, Jun 27, 2007 at 07:32:45AM +0200, Nick Piggin wrote: > On Tue, Jun 26, 2007 at 08:34:49AM -0400, Chris Mason wrote: > > On Tue, Jun 26, 2007 at 07:23:09PM +1000, David Chinner wrote: > > > On Tue, Jun 26, 2007 at 01:55:11PM +1000, Nick Piggin wrote: > > > >

Re: [RFC] fsblock

2007-06-28 Thread Chris Mason
On Thu, Jun 28, 2007 at 04:44:43AM +0200, Nick Piggin wrote: > On Thu, Jun 28, 2007 at 08:35:48AM +1000, David Chinner wrote: > > On Wed, Jun 27, 2007 at 07:50:56AM -0400, Chris Mason wrote: > > > Lets look at a typical example of how IO actually gets done today, > > &

Re: [PATCH RFC] extent mapped page cache

2007-07-18 Thread Chris Mason
On Thu, 12 Jul 2007 00:00:28 -0700 Daniel Phillips <[EMAIL PROTECTED]> wrote: > On Tuesday 10 July 2007 14:03, Chris Mason wrote: > > This patch aims to demonstrate one way to replace buffer heads with > > a few extent trees... > > Hi Chris, > > Quite terse

[ANNOUNCE] seekwatcher IO graphing v0.2

2007-07-23 Thread Chris Mason
Hello everyone, Since doing the initial Btrfs benchmarks, I've made my blktrace graphing utility a little more generic and tossed it out on oss.oracle.com. This new version can easily graph two different runs, and has a few other tweaks that make the graphs look nicer. Docs, examples and other

Re: [PATCH 0/6] writeback time order/delay fixes take 3

2007-08-21 Thread Chris Mason
On Sun, 12 Aug 2007 17:11:20 +0800 Fengguang Wu <[EMAIL PROTECTED]> wrote: > Andrew and Ken, > > Here are some more experiments on the writeback stuff. > Comments are highly welcome~ I've been doing benchmarks lately to try and trigger fragmentation, and one of them is a simulation of make -j

Re: [PATCH 0/6] writeback time order/delay fixes take 3

2007-08-22 Thread Chris Mason
On Wed, 22 Aug 2007 09:18:41 +0800 Fengguang Wu <[EMAIL PROTECTED]> wrote: > On Tue, Aug 21, 2007 at 08:23:14PM -0400, Chris Mason wrote: > > On Sun, 12 Aug 2007 17:11:20 +0800 > > Fengguang Wu <[EMAIL PROTECTED]> wrote: > > > > > Andrew and Ken, &

Re: [PATCH 0/6] writeback time order/delay fixes take 3

2007-08-23 Thread Chris Mason
On Thu, 23 Aug 2007 12:47:23 +1000 David Chinner <[EMAIL PROTECTED]> wrote: > On Wed, Aug 22, 2007 at 08:42:01AM -0400, Chris Mason wrote: > > I think we should assume a full scan of s_dirty is impossible in the > > presence of concurrent writers. We want to be able to

Re: [PATCH 0/6] writeback time order/delay fixes take 3

2007-08-24 Thread Chris Mason
On Fri, 24 Aug 2007 21:24:58 +0800 Fengguang Wu <[EMAIL PROTECTED]> wrote: > > 2) s_dirty and s_io both become radix trees. s_dirty is indexed by > > a sequence number that corresponds to age. It is treated as a big > > circular indexed list that can wrap around over time. Radix tree > > tags

Re: how do versioning filesystems take snapshot of opened files?

2007-07-03 Thread Chris Mason
On Tue, 3 Jul 2007 01:28:57 -0400 "Xin Zhao" <[EMAIL PROTECTED]> wrote: > Hi, > > > If a file is already opened when snapshot command is issued, the file > itself could be in an inconsistent state already. Before the file is > closed, maybe part of the file contains old data, the rest contains

Re: Versioning file system

2007-07-05 Thread Chris Mason
On Thu, 5 Jul 2007 09:57:40 -0400 "John Stoffel" <[EMAIL PROTECTED]> wrote: > > "Erik" == Erik Mouw <[EMAIL PROTECTED]> writes: > > Erik> (sorry for the late reply, just got back from holiday) > Erik> On Mon, Jun 18, 2007 at 01:29:56PM -0400, Theodore Tso wrote: > >> As I mentioned in my

Re: [PATCH 0/3] [RFC][PATCH] clustered writeback

2007-08-27 Thread Chris Mason
On Mon, 27 Aug 2007 05:03:36 -0700 Arjan van de Ven <[EMAIL PROTECTED]> wrote: > On Mon, 27 Aug 2007 19:21:52 +0800 > > > > Because it does the work in small batches of 10 inodes, when the > > system has <=10 dirty inodes, its behavior will reduce to: > > - do a full sweep *at once* on every 25s

Re: [PATCH 0/6] writeback time order/delay fixes take 3

2007-08-28 Thread Chris Mason
: > > > > On Tue, Aug 21, 2007 at 08:23:14PM -0400, Chris Mason wrote: > > > > Notes: > > > > (1) I'm not sure inode number is correlated to disk location in > > > > filesystems other than ext2/3/4. Or parent dir? > > > > > &

Re: [PATCH 0/6] writeback time order/delay fixes take 3

2007-08-28 Thread Chris Mason
On Wed, 29 Aug 2007 02:33:08 +1000 David Chinner <[EMAIL PROTECTED]> wrote: > On Tue, Aug 28, 2007 at 11:08:20AM -0400, Chris Mason wrote: > > > > > > > > I wonder if XFS can benefit any more from the general writeback > > > > cluster

[PATCH RFC] extent mapped page cache

2007-07-24 Thread Chris Mason
On Tue, 10 Jul 2007 17:03:26 -0400 Chris Mason <[EMAIL PROTECTED]> wrote: > This patch aims to demonstrate one way to replace buffer heads with a > few extent trees. Buffer heads provide a few different features: > > 1) Mapping of logical file offset to blocks on disk &g

[PATCH RFC] extent mapped page cache main code

2007-07-24 Thread Chris Mason
Core Extentmap implementation diff -r 126111346f94 -r 53cabea328f7 fs/Makefile --- a/fs/Makefile Mon Jul 09 10:53:57 2007 -0400 +++ b/fs/Makefile Tue Jul 24 15:40:27 2007 -0400 @@ -11,7 +11,7 @@ obj-y := open.o read_write.o file_table. attr.o bad_inode.o file.o

[PATCH RFC] ext2 extentmap support

2007-07-24 Thread Chris Mason
mount -o extentmap to use the new stuff diff -r 126111346f94 -r 53cabea328f7 fs/ext2/ext2.h --- a/fs/ext2/ext2.hMon Jul 09 10:53:57 2007 -0400 +++ b/fs/ext2/ext2.hTue Jul 24 15:40:27 2007 -0400 @@ -1,5 +1,6 @@ #include #include +#include /* * ext2 mount options @@ -65,6 +66,7 @@

Re: [PATCH RFC] extent mapped page cache

2007-07-24 Thread Chris Mason
On Tue, 24 Jul 2007 23:25:43 +0200 Peter Zijlstra <[EMAIL PROTECTED]> wrote: > On Tue, 2007-07-24 at 16:13 -0400, Trond Myklebust wrote: > > On Tue, 2007-07-24 at 16:00 -0400, Chris Mason wrote: > > > On Tue, 10 Jul 2007 17:03:26 -0400 > > > Chri

Re: [PATCH RFC] extent mapped page cache

2007-07-25 Thread Chris Mason
On Wed, 25 Jul 2007 04:32:17 +0200 Nick Piggin <[EMAIL PROTECTED]> wrote: > On Tue, Jul 24, 2007 at 07:25:09PM -0400, Chris Mason wrote: > > On Tue, 24 Jul 2007 23:25:43 +0200 > > Peter Zijlstra <[EMAIL PROTECTED]> wrote: > > > > The tree is a cri

Re: [PATCH RFC] extent mapped page cache

2007-07-25 Thread Chris Mason
On Thu, 26 Jul 2007 03:37:28 +0200 Nick Piggin <[EMAIL PROTECTED]> wrote: > > > One advantage to the state tree is that it separates the state from > > the memory being described, allowing a simple kmap style interface > > that covers subpages, highmem and superpages. > > I suppose so,

Re: [PATCH RFC] extent mapped page cache

2007-07-26 Thread Chris Mason
On Thu, 26 Jul 2007 04:36:39 +0200 Nick Piggin <[EMAIL PROTECTED]> wrote: [ are state trees a good idea? ] > > One thing it gains us is finding the start of the cluster. Even if > > called by kswapd, the state tree allows writepage to find the start > > of the cluster and send down a big bio

[ANNOUNCE] seekwatcher v0.3 IO graphing an animation

2007-07-27 Thread Chris Mason
Hello everyone, I've tossed out seekwatcher v0.3. The major changes are using rolling averages to smooth out the seek and throughput graphs, and it can generate mpgs of the IO done by a given trace. Here's a sample of the smoother graphs (creating 20 kernel trees):

Re: [PATCH 00/23] per device dirty throttling -v8

2007-08-06 Thread Chris Mason
On Sun, 5 Aug 2007 11:00:29 -0400 Theodore Tso <[EMAIL PROTECTED]> wrote: > On Sun, Aug 05, 2007 at 02:26:53AM +0200, Andi Kleen wrote: > > I always thought the right solution would be to just sync atime only > > very very lazily. This means if a inode is only dirty because of an > > atime update

Re: [PATCH] rd: Use a private inode for backing storage

2007-10-22 Thread Chris Mason
On Sun, 21 Oct 2007 12:39:30 -0600 [EMAIL PROTECTED] (Eric W. Biederman) wrote: > Nick Piggin <[EMAIL PROTECTED]> writes: > > > On Sunday 21 October 2007 18:23, Eric W. Biederman wrote: > >> Christian Borntraeger <[EMAIL PROTECTED]> writes: > > > >> Let me put it another way. Looking at

Re: [PATCH] reiserfs: don't drop PG_dirty when releasing sub-page-sized dirty file

2007-10-23 Thread Chris Mason
On Tue, 23 Oct 2007 19:56:20 +0800 Fengguang Wu <[EMAIL PROTECTED]> wrote: > On Tue, Oct 23, 2007 at 12:07:07PM +0200, Peter Zijlstra wrote: > > [ adding reiserfs devs to the CC ] > > Thank you. > > This fix is kind of crude - even when it fixed Maxim's problem, and > survived my stress testing

[CFP] 2008 Linux Storage and Filesystem Workshop

2007-10-24 Thread Chris Mason
Hello everyone, We are organizing another filesystem and storage workshop in San Jose next Feb 25 and 26. You can find some great writeups of last year's conference on LWN: http://lwn.net/Articles/226351/ This year we're trying to concentrate on more problem solving sessions, short term

Re: dio_get_page() lockdep complaints

2007-11-09 Thread Chris Mason
On Fri, 09 Nov 2007 09:48:22 -0800 Zach Brown <[EMAIL PROTECTED]> wrote: > > >> So reiser and NFS need to be fixed. No? > > > > Actually, it is rather mmap() needs to be fixed. > > Sure, I'm willing to have that demonstrated. My point was that DIO > getting the mmap_sem inside i_mutex is

Re: dio_get_page() lockdep complaints

2007-11-09 Thread Chris Mason
On Fri, 09 Nov 2007 10:35:04 -0800 Zach Brown <[EMAIL PROTECTED]> wrote: > > Without getting into a huge patch, the best fix would just be > > switching to try lock. If the tail doesn't get packed, the world > > doesn't end. > > So, something like this? Re

Re: dio_get_page() lockdep complaints

2007-11-09 Thread Chris Mason
On Fri, 9 Nov 2007 13:53:27 -0500 Chris Mason <[EMAIL PROTECTED]> wrote: > On Fri, 09 Nov 2007 10:35:04 -0800 > Zach Brown <[EMAIL PROTECTED]> wrote: > > > > Without getting into a huge patch, the best fix would just be > > > switching to try lock. If

Re: dio_get_page() lockdep complaints

2007-11-09 Thread Chris Mason
On Fri, 09 Nov 2007 11:16:53 -0800 Zach Brown <[EMAIL PROTECTED]> wrote: > > Ugh, I thought the preallocation was getting freed elsewhere, but it > > looks like I was wrong. We can't just skip the i_mutex after all, > > sorry. > > Ah, so none of those tests at the top will stop tail packing if

More Large blocksize benchmarks

2007-10-15 Thread Chris Mason
Hello everyone, I'm stealing the cc list and reviving and old thread because I've finally got some numbers to go along with the Btrfs variable blocksize feature. The basic idea is to create a read/write interface to map a range of bytes on the address space, and use it in Btrfs for all metadata

Re: More Large blocksize benchmarks

2007-10-16 Thread Chris Mason
On Tue, 2007-10-16 at 12:36 +1000, David Chinner wrote: > On Mon, Oct 15, 2007 at 08:22:31PM -0400, Chris Mason wrote: > > Hello everyone, > > > > I'm stealing the cc list and reviving and old thread because I've > > finally got some numbers to go along with t

Re: [PATCH] rd: Mark ramdisk buffers heads dirty

2007-10-17 Thread Chris Mason
On Wed, 2007-10-17 at 11:57 -0600, Eric W. Biederman wrote: > Christian Borntraeger <[EMAIL PROTECTED]> writes: > > > Eric, > > > > Am Dienstag, 16. Oktober 2007 schrieb Christian Borntraeger: > >> Am Dienstag, 16. Oktober 2007 schrieb Eric W. Biederman: > >> > >> > fs/buffer.c |3 +++ > >> >

Re: [PATCH] rd: Mark ramdisk buffers heads dirty

2007-10-17 Thread Chris Mason
On Wed, 2007-10-17 at 14:29 -0600, Eric W. Biederman wrote: > Chris Mason <[EMAIL PROTECTED]> writes: > > > In this case, the commit block isn't allowed to be dirty before reiserfs > > decides it is safe to write it. The journal code expects it is the only > > spo

Re: [PATCH] rd: Mark ramdisk buffers heads dirty

2007-10-17 Thread Chris Mason
On Wed, 2007-10-17 at 15:30 -0600, Eric W. Biederman wrote: > Chris Mason <[EMAIL PROTECTED]> writes: > > >> Thinking about it. I don't believe anyone has ever intentionally built > >> a filesystem tool that depends on being able to modify a file systems &g

Re: [PATCH] rd: Mark ramdisk buffers heads dirty

2007-10-17 Thread Chris Mason
On Wed, 2007-10-17 at 17:28 -0600, Eric W. Biederman wrote: > Chris Mason <[EMAIL PROTECTED]> writes: > > > So, the problem is using the Dirty bit to indicate pinned. You're > > completely right that our current setup of buffer heads and pages and > > f

Re: [patch 4/6][RFC] Attempt to plug race with truncate

2007-10-29 Thread Chris Mason
On Fri, 26 Oct 2007 16:37:36 -0700 Mike Waychison <[EMAIL PROTECTED]> wrote: > Attempt to deal with races with truncate paths. > > I'm not really sure on the locking here, but these seem to be taken > by the truncate path. BKL is left as some filesystem may(?) still > require it. > >

Re: [patch 5/6][RFC] Introduce FIBMAP64

2007-10-29 Thread Chris Mason
On Fri, 26 Oct 2007 16:37:37 -0700 Mike Waychison <[EMAIL PROTECTED]> wrote: > Introduce FIBMAP64. This is the same as FIBMAP, but takes a u64. If we're adding new ioctls, I'd rather see the FIEMAP stuff go in, a quick search found discussions but has it died off? -chris - To unsubscribe from

Re: [patch 0/6][RFC] Cleanup FIBMAP

2007-10-29 Thread Chris Mason
On Sat, 27 Oct 2007 18:57:06 +0100 Anton Altaparmakov <[EMAIL PROTECTED]> wrote: > Hi, > > ->bmap is ugly and horrible! If you have to do this at the very > least please cause ->bmap64 to be able to return error values in case > the file system failed to get the information or indeed such >

Re: [patch 0/6][RFC] Cleanup FIBMAP

2007-10-29 Thread Chris Mason
On Mon, 29 Oct 2007 12:18:22 -0700 Mike Waychison <[EMAIL PROTECTED]> wrote: > Zach Brown wrote: > >>> And another of my pet peeves with ->bmap is that it uses 0 to > >>> mean "sparse" which causes a conflict on NTFS at least as block > >>> zero is part of the $Boot system file so it is a real,

2008 Linux Storage and Filesystem Workshop

2007-11-05 Thread Chris Mason
Hello everyone, The position statement submission system for the 2008 storage and filesystem workshop is now online. This is how you let us know you're interested in attending and what topics are most important for discussion. For all the details, please see:

Re: [3.10] Oopses in kmem_cache_allocate() via prepare_creds()

2013-08-19 Thread Chris Mason
Quoting Linus Torvalds (2013-08-19 17:16:36) > On Mon, Aug 19, 2013 at 1:29 PM, Christoph Lameter wrote: > > On Mon, 19 Aug 2013, Simon Kirby wrote: > > > >>[... ] The > >> alloc/free traces are always the same -- always alloc_pipe_info and > >> free_pipe_info. This is seen on 3.10 and (now)

[GIT PULL] Btrfs

2013-08-10 Thread Chris Mason
Hi Linus Please pull my for-linus branch: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus These are assorted fixes, mostly from Josef nailing down xfstests runs. Zach also has a long standing fix for problems with readdir wrapping f_pos (or ctx->pos) These patches

Re: Boot failure on Arndale with next-20131105

2013-11-05 Thread Chris Mason
Quoting Olof Johansson (2013-11-05 15:23:51) > On Tue, Nov 5, 2013 at 11:33 AM, Jens Axboe wrote: [ horrible crashes fixed by removing my patch ] > > Very weird! What file system is being used? > > Most of my failures have happened on regular MMC cards with ext4 > filesystems on them. > >

Re: Boot failure on Arndale with next-20131105

2013-11-05 Thread Chris Mason
Quoting Olof Johansson (2013-11-05 15:38:33) > On Tue, Nov 5, 2013 at 12:33 PM, Chris Mason wrote: > > Quoting Olof Johansson (2013-11-05 15:23:51) > >> On Tue, Nov 5, 2013 at 11:33 AM, Jens Axboe wrote: > > > > [ horrible crashes fixed by removing my patch ] &g

Re: Boot failure on Arndale with next-20131105

2013-11-05 Thread Chris Mason
Quoting Olof Johansson (2013-11-05 17:41:42) > On Tue, Nov 5, 2013 at 2:06 PM, Stephen Warren wrote: > > On 11/05/2013 01:56 PM, Chris Mason wrote: > >> Quoting Olof Johansson (2013-11-05 15:38:33) > >>> On Tue, Nov 5, 2013 at 12:33 PM, Chris Mason > >>

Re: [PATCH] block: Revert bio_clone() default behaviour

2013-11-06 Thread Chris Mason
Quoting Kent Overstreet (2013-11-05 22:48:41) > This patch reverts the default behaviour introduced by > 9fc6286f347d00528adcdcf12396d220f47492ed - bio_clone_biovec() no clonger > shares the source bio's biovec, cloning the biovec is once again the > default. > > Instead, we add a new

Re: [PATCH] block: Revert bio_clone() default behaviour

2013-11-06 Thread Chris Mason
Quoting Kent Overstreet (2013-11-06 15:02:22) > On Wed, Nov 06, 2013 at 11:11:30AM -0500, Chris Mason wrote: > > Quoting Kent Overstreet (2013-11-05 22:48:41) > > > This patch reverts the default behaviour introduced by > > > 9fc6286f347d00528adcdcf12396d220f47492ed - bi

Re: block: Revert bio_clone() default behaviour

2013-11-06 Thread Chris Mason
Quoting Mike Snitzer (2013-11-06 15:36:40) > On Wed, Nov 06 2013 at 3:22pm -0500, > Chris Mason wrote: > > > Quoting Kent Overstreet (2013-11-06 15:02:22) > > > On Wed, Nov 06, 2013 at 11:11:30AM -0500, Chris Mason wrote: > > > > > > > > I thin

Re: [PATCH] block: Revert bio_clone() default behaviour

2013-11-06 Thread Chris Mason
Quoting Kent Overstreet (2013-11-06 15:57:34) > On Wed, Nov 06, 2013 at 03:22:36PM -0500, Chris Mason wrote: > > Quoting Kent Overstreet (2013-11-06 15:02:22) [ ... nods, thanks! ... ] > OTOH - with regards to just the ordering requirements, the more I look at > various code the

[GIT PULL] Btrfs

2013-10-18 Thread Chris Mason
Hi Linus, My for-linus branch has a one line fix: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus Sage hit a deadlock with ceph on btrfs, and Josef tracked it down to a regression in our initial rc1 pull. When doing nocow writes we were sometimes starting a

[GIT PULL] Btrfs

2013-11-14 Thread Chris Mason
Hi Linus, Please pull my for-linus branch: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus This is our usual merge window set of bug fixes, performance improvements and cleanups. Miao Xie has some really nice optimizations for writeback. Josef also expanded our

Re: [RFC PATCH] futex: Remove requirement for lock_page in get_futex_key

2013-10-29 Thread Chris Mason
Quoting Mel Gorman (2013-10-29 13:38:14) > Thomas Gleixner and Peter Zijlstra discussed off-list that real-time users > currently have a problem with the page lock being contended for unbounded > periods of time during futex operations. The three of us discussed the > possibiltity that the page

[PATCH] Btrfs: update bi_remaining to relfect our bio endio chaining

2013-10-31 Thread Chris Mason
Btrfs is sometimes calling bio_endio twice on the same bio while we chain things. This makes sure we don't trip over new assertions in fs/bio.c Signed-off-by: Chris Mason diff --git a/fs/btrfs/check-integrity.c b/fs/btrfs/check-integrity.c index 7fcac70..5b30360 100644 --- a/fs/btrfs/check

Re: [GIT PULL] Btrfs

2013-11-15 Thread Chris Mason
Quoting Heiko Carstens (2013-11-15 06:32:16) > On Thu, Nov 14, 2013 at 12:19:52PM -0500, Chris Mason wrote: > > Hi Linus, > > > > Please pull my for-linus branch: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git > > for-linus

Re: [GIT PULL] Btrfs

2013-11-15 Thread Chris Mason
Quoting Chris Mason (2013-11-15 07:21:31) > Quoting Heiko Carstens (2013-11-15 06:32:16) > > On Thu, Nov 14, 2013 at 12:19:52PM -0500, Chris Mason wrote: > > > Hi Linus, > > > > > > Please pull my for-linus branch: > > > > > > git://git.ker

[GIT PULL] Btrfs

2013-11-15 Thread Chris Mason
Hi Linus, This pull fixes the empty_zero_page bug that Heiko reported, and includes one more cleanup from Al Viro. Please grab my for-linus: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus Chris Mason (1) commits (+7/-2): btrfs: fix empty_zero_page misusage

[GIT PULL] Btrfs Maintainer update

2013-12-04 Thread Chris Mason
for Josef as well. Instead of diffstat etc, I've put the patch below. The commit is also signed, my updated key should be floating around the pgp servers now. commit c0778e2534b81be6b85b5ee07c0e15ff774f7136 Author: Chris Mason Date: Tue Dec 3 20:16:03 2013 -0500 Btrfs: update

Re: [GIT PULL] Btrfs Maintainer update

2013-12-05 Thread Chris Mason
Quoting Linus Torvalds (2013-12-05 12:53:26) > On Wed, Dec 4, 2013 at 9:53 AM, Chris Mason wrote: > > > > Instead of diffstat etc, I've put the patch below. The commit is also > > signed, my updated key should be floating around the pgp servers now. > > You've don

[GIT PULL] Btrfs

2013-12-12 Thread Chris Mason
Hi Linus, Please pull my for-linus branch (or for-linus signed tag): git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus This is a small collection of fixes. It was rebased this morning, but I was just fixing signed-off-by tags with the wrong email. Wang Shilong (3)

[GIT PULL] Btrfs fixes

2013-03-17 Thread Chris Mason
Hi Linus, My for-linus branch has some btrfs fixes: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus Eric's rcu barrier patch fixes a long standing problem with our unmount code hanging on to devices in workqueue helpers. Liu Bo nailed down a difficult assertion

[GIT PULL] Btrfs fixes

2013-02-06 Thread Chris Mason
in btrfs_file_aio_write() (+2/-1) Jan Schmidt (1) commits (+10/-12): Btrfs: fix EDQUOT handling in btrfs_delalloc_reserve_metadata Liu Bo (1) commits (+38/-9): Btrfs: fix race between snapshot deletion and getting inode Chris Mason (1) commits (+4/-1): Btrfs: move d_instantiate outside

Re: Oops when mounting btrfs partition

2013-02-02 Thread Chris Mason
Hi Arnd, First things first, nospace_cache is a safe thing to use. It is slow because it's finding free extents, but it's just a cache and always safe to discard. With your other errors, I'd just mount it readonly and then you won't waste time on atime updates. I'll take a look at the BUG you

Re: [BUG REPORT] Kernel panic on 3.9.0-rc7-4-gbb33db7

2013-04-19 Thread Chris Mason
Quoting Tejun Heo (2013-04-19 01:57:54) > > Ewweehh > > No wonder this thing crashes. Chris, can't the original bio carry > bbio in bi_private and let end_bio_extent_readpage() free the bbio > instead of abusing bi_bdev like this? Yes, we can definitely carry bbio up

Re: [BUG REPORT] Kernel panic on 3.9.0-rc7-4-gbb33db7

2013-04-19 Thread Chris Mason
Quoting Jens Axboe (2013-04-19 09:32:50) > > > > No wonder this thing crashes. Chris, can't the original bio carry > > bbio in bi_private and let end_bio_extent_readpage() free the bbio > > instead of abusing bi_bdev like this? > > Ugh, wtf. > > Chris, time for a swim in the bay :-) Yeah, I

Re: [GIT PULL] SLAB changes for v3.10

2013-05-08 Thread Chris Mason
[ Sorry if I break the threading on this, I had to pull it off gmane ] On Tue, 7 May 2013, Tony Lindgren wrote: > OK got it narrowed down to CONFIG_DEBUG_SPINLOCK=y causing the problem > with commit 8a965b3b. Ain't nothing like bisecting and booting and then > diffing .config files on top of

Re: [GIT PULL] SLAB changes for v3.10

2013-05-08 Thread Chris Mason
Quoting Christoph Lameter (2013-05-08 14:25:49) > On Wed, 8 May 2013, Chris Mason wrote: > > > This patch fixes things for me, but to maintain the rules from > > Christoph's patch, kmalloc_caches[2] should have been created whenever > > kmalloc_caches[7] was done. >

[PATCH] Fix crash during slab init

2013-05-08 Thread Chris Mason
at index 1 or 2 might not get created at all. This patch makes sure none of the slabs get skipped. Tony Lindgren bisected this down to the offending commit, which really helped because bisect kept bringing me to almost but not quite this one. Signed-off-by: Chris Mason Acked-by: Christoph

[GIT PULL] Btrfs

2013-05-09 Thread Chris Mason
n missed error case Simon Kirby (1) commits (+133/-109): Btrfs: Include the device in most error printk()s Nathaniel Yazdani (1) commits (+1/-1): btrfs: fix minor typo in comment Chris Mason (1) commits (+5/-0): Btrfs: allow superblock mismatch from older mkfs Vincent (1) commits (+3/

[GIT PULL] Btrfs

2013-03-02 Thread Chris Mason
e null fs_info in btrfs_panic() (+7/-4) btrfs: fix varargs in __btrfs_std_error (+7/-7) btrfs: list_entry can't return NULL (+0/-2) Chris Mason (7) commits (+561/-30): Btrfs: reduce CPU contention while waiting for delayed extent operations (+70/-5) Btrfs: remove conflicting check for minim

Re: [GIT PULL] Btrfs

2013-03-02 Thread Chris Mason
On Sat, Mar 02, 2013 at 05:45:41PM -0700, Linus Torvalds wrote: > On Sat, Mar 2, 2013 at 7:15 AM, Chris Mason wrote: > > > > Our set of btrfs features, fixes and cleanups are in my for-linus > > branch: > > I *really* wish that big pull requests like this would com

Re: [PATCH] btrfs/raid56: Add missing #include

2013-03-03 Thread Chris Mason
On Sun, Mar 03, 2013 at 04:44:41AM -0700, Geert Uytterhoeven wrote: > tilegx_defconfig: > > fs/btrfs/raid56.c: In function 'btrfs_alloc_stripe_hash_table': > fs/btrfs/raid56.c:206:3: error: implicit declaration of function 'vzalloc' > [-Werror=implicit-function-declaration] >

[GIT PULL] Btrfs fixup

2013-03-03 Thread Chris Mason
Hi Linus, Geert and James both sent this one in, sorry guys. git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus Geert Uytterhoeven (1) commits (+1/-0): btrfs/raid56: Add missing #include Total: (1) commits (+1/-0) fs/btrfs/raid56.c | 1 + 1 file changed, 1

[GIT PULL] One more btrfs

2013-04-13 Thread Chris Mason
Hi Linus My for-linus branch: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus Has a recent fix from Josef for our tree log replay code. It fixes problems where the inode counter for the number of bytes in the file wasn't getting updated properly during fsync

[GIT PULL] Btrfs

2014-02-16 Thread Chris Mason
offset for compressed extents Mitch Harder (1) commits (+1/-1): Btrfs: fix max_inline mount option Chris Mason (1) commits (+0/-17): Revert "btrfs: add ioctl to export size of global metadata reservation" Josef Bacik (1) commits (+9/-2): Btrfs: unset DCACHE_DISCONNECTED whe

Re: [RFC PATCH] fs: xattr-based FS_IOC_[GS]ETFLAGS interface

2014-01-07 Thread Chris Mason
On Tue, 2014-01-07 at 11:43 -0800, Darrick J. Wong wrote: > On Tue, Jan 07, 2014 at 12:04:30PM -0500, Theodore Ts'o wrote: > > On Tue, Jan 07, 2014 at 07:49:35AM -0800, Christoph Hellwig wrote: > > > On Tue, Jan 07, 2014 at 01:48:31PM +0100, Jan Kara wrote: > > > > I have to say I'm not thrilled

Re: [block:for-3.14/core] kernel BUG at fs/bio.c:1748

2014-01-07 Thread Chris Mason
s this against? Just Jens or some extra code too? I'll run some tests here. My original patch is below (it's slightly different from Muthu's). Btrfs is sometimes calling bio_endio twice on the same bio while we chain things. This makes sure we don't trip over new assertions in fs/bio.c Signed-off

Re: [RFC PATCH] fs: xattr-based FS_IOC_[GS]ETFLAGS interface

2014-01-07 Thread Chris Mason
On Tue, 2014-01-07 at 14:02 -0800, Darrick J. Wong wrote: > On Tue, Jan 07, 2014 at 07:59:15PM +0000, Chris Mason wrote: > > On Tue, 2014-01-07 at 11:43 -0800, Darrick J. Wong wrote: > > > On Tue, Jan 07, 2014 at 12:04:30PM -0500, Theodore Ts'o wrote: > > > > On T

Re: [block:for-3.14/core] kernel BUG at fs/bio.c:1748

2014-01-08 Thread Chris Mason
On Tue, 2014-01-07 at 13:23 -0800, Muthu Kumar wrote: > Chris, > This is based off of Jens block tree, for-3.14/core branch... > Ok, Kent did pull in one of my hunks, one was a comment and the third was effectively the same as your patch. I tried to test the end result today, but get these on

Re: [block:for-3.14/core] kernel BUG at fs/bio.c:1748

2014-01-08 Thread Chris Mason
On Wed, 2014-01-08 at 11:54 -0800, Muthu Kumar wrote: > Chris, > > [8.336061] WARNING: CPU: 0 PID: 0 at fs/bio.c:1778 bio_endio+0xbe/0x100() > [8.336062] bio_endio: bio for (unknown) without endio > > This is my recent change to avoid memory leak in bio_endio. But I > think the problem

Re: [block:for-3.14/core] kernel BUG at fs/bio.c:1748

2014-01-08 Thread Chris Mason
On Wed, 2014-01-08 at 12:40 -0800, Muthu Kumar wrote: > On Wed, Jan 8, 2014 at 12:16 PM, Chris Mason wrote: > > On Wed, 2014-01-08 at 11:54 -0800, Muthu Kumar wrote: > >> Chris, > >> > >> [8.336061] WARNING: CPU: 0 PID: 0 at fs/bio.c:1778 > >&

Re: [block:for-3.14/core] kernel BUG at fs/bio.c:1748

2014-01-08 Thread Chris Mason
On Wed, 2014-01-08 at 13:01 -0800, Muthu Kumar wrote: > On Wed, Jan 8, 2014 at 12:51 PM, Chris Mason wrote: > > On Wed, 2014-01-08 at 12:40 -0800, Muthu Kumar wrote: > >> On Wed, Jan 8, 2014 at 12:16 PM, Chris Mason wrote: > >> > On Wed, 2014-01-08 at

Re: [block:for-3.14/core] kernel BUG at fs/bio.c:1748

2014-01-08 Thread Chris Mason
guang wu > CC: Kent Overstreet > CC: Jens Axboe > CC: Chris Mason > Signed-off-by: Muthukumar Ratty > Reviewed-by: Chris Mason Jens, please pull this one in. > --- > fs/btrfs/volumes.c |8 ++-- > 1 files changed, 6 insertions(+), 2 deletions(-

Re: [btrfs] BUG: unable to handle kernel NULL pointer dereference at 0000000000000038

2014-02-07 Thread Chris Mason
On Fri 07 Feb 2014 07:10:38 AM EST, Fengguang Wu wrote: On Fri, Feb 07, 2014 at 02:13:59AM -0800, David Rientjes wrote: On Fri, 7 Feb 2014, Fengguang Wu wrote: [1.625020] BTRFS: selftest: Running btrfs_split_item tests [1.627004] BTRFS: selftest: Running find delalloc tests [

[GIT PULL] Btrfs

2014-02-09 Thread Chris Mason
Hi Linus, Please pull my for-linus branch: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus This is a small collection of fixes Josef Bacik (2) commits (+4/-5): Btrfs: don't loop forever if we can't run because of the tree mod log (+1/-0) Btrfs: fix

Re: 1be41b78: +18% increased btrfs write throughput

2014-01-03 Thread Chris Mason
On Fri, 2014-01-03 at 23:54 +0800, fengguang...@intel.com wrote: > Hi Josef, > > FYI. We are doing 0day performance tests and happen to notice that > btrfs write throughput increased considerably during v3.10-11 time > frame: > > v3.10 v3.11

Re: [RFC v2 2/2] fs: btrfs: Extends btrfs/raid56 to support up to six parities

2014-01-06 Thread Chris Mason
On Mon, 2014-01-06 at 10:31 +0100, Andrea Mazzoleni wrote: > This patch changes btrfs/raid56.c to use the new raid interface and > extends its support to an arbitrary number of parities. > > More in details, the two faila/failb failure indexes are now replaced > with a fail[] vector that keeps

Re: Build failures due to commit 416161db (btrfs: offline dedupe)

2013-09-13 Thread Chris Mason
Quoting Guenter Roeck (2013-09-13 12:35:35) > On Fri, Sep 13, 2013 at 03:52:43PM +0200, Geert Uytterhoeven wrote: > > On Fri, Sep 13, 2013 at 3:33 PM, Guenter Roeck wrote: > > > fs/btrfs/ioctl.c: In function 'btrfs_ioctl_file_extent_same': > > > fs/btrfs/ioctl.c:2802:3: error: implicit

Re: Build failures due to commit 416161db (btrfs: offline dedupe)

2013-09-13 Thread Chris Mason
Quoting Mark Fasheh (2013-09-13 13:58:01) > On Fri, Sep 13, 2013 at 01:00:22PM -0400, Chris Mason wrote: > > Quoting Guenter Roeck (2013-09-13 12:35:35) > > I'm happy to fix this with a bigger put of the info struct, just > > let me know the preferred arch-happy solution. >

[GIT PULL] Btrfs

2013-10-11 Thread Chris Mason
Hi Linus, We've got more bug fixes in my for-linus branch: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus One of these fixes another corner of the compression oops from last time. Miao nailed down some problems with concurrent snapshot deletion and drive

[GIT PULL] Btrfs

2013-09-22 Thread Chris Mason
Hi Linus, Please pull my for-linus branch: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus These are mostly bug fixes and a two small performance fixes. The most important of the bunch are Josef's fix for a snapshotting regression and Mark's update to fix compile

[GIT PULL] Btrfs

2013-09-12 Thread Chris Mason
Hi Linus, Please pull my for-linus branch: git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git for-linus This is against 3.11-rc7, but was pulled and tested against your tree as of yesterday. We do have two small incrementals queued up, but I wanted to get this bunch out the

Re: [PATCH v2 0/4] ipc: reduce ipc lock contention

2013-03-07 Thread Chris Mason
On Thu, Mar 07, 2013 at 01:45:33AM -0700, Peter Zijlstra wrote: > On Tue, 2013-03-05 at 15:53 -0500, Rik van Riel wrote: > > > Indeed. Though how well my patches will work with Oracle will > > depend a lot on what kind of semctl syscalls they are doing. > > > > Does Oracle typically do one

Re: [PATCH v2 0/4] ipc: reduce ipc lock contention

2013-03-07 Thread Chris Mason
On Thu, Mar 07, 2013 at 08:54:55AM -0700, Dave Kleikamp wrote: > On 03/07/2013 06:55 AM, Chris Mason wrote: > > On Thu, Mar 07, 2013 at 01:45:33AM -0700, Peter Zijlstra wrote: > >> On Tue, 2013-03-05 at 15:53 -0500, Rik van Riel wrote: > >> > >>> Indeed

[GIT PULL] Btrfs updates

2013-03-08 Thread Chris Mason
reloc roots (+2/-0) Btrfs: avoid deadlock on transaction waiting list (+7/-0) Btrfs: free all recorded tree blocks on error (+6/-3) Btrfs: do not BUG_ON on aborted situation (+12/-3) Btrfs: do not BUG_ON in prepare_to_reloc (+9/-1) Chris Mason (2) commits (+96/-63): Btrfs

Re: btrfs triggered lockdep WARN.

2013-06-27 Thread Chris Mason
Quoting Dave Jones (2013-06-27 10:58:24) > Another bug caused by this script. > https://github.com/kernelslacker/io-tests/blob/master/setup.sh I'm still struggling to reproduce that one here. I've tried every variation I can think of but I'll try again. I really hope you don't already have

Re: btrfs triggered lockdep WARN.

2013-06-27 Thread Chris Mason
Quoting Dave Jones (2013-06-27 11:19:22) > On Thu, Jun 27, 2013 at 11:01:30AM -0400, Chris Mason wrote: > > Quoting Dave Jones (2013-06-27 10:58:24) > > > Another bug caused by this script. > https://github.com/kernelslacker/io-tests/blob/master/setup.sh > >

Re: [3.10-rc6] WARNING: at fs/btrfs/inode.c:7961 btrfs_destroy_inode+0x265/0x2e0 [btrfs]()

2013-06-17 Thread Chris Mason
Quoting Dave Jones (2013-06-17 09:49:55) > Hit this while running this script in a loop.. > https://github.com/kernelslacker/io-tests/blob/master/setup.sh > [34385.251507] [ cut here ] > [34385.254068] WARNING: at fs/btrfs/inode.c:7961 > btrfs_destroy_inode+0x265/0x2e0

Re: [3.10-rc6] WARNING: at fs/btrfs/inode.c:7961 btrfs_destroy_inode+0x265/0x2e0 [btrfs]()

2013-06-17 Thread Chris Mason
Quoting Dave Jones (2013-06-17 14:20:06) > On Mon, Jun 17, 2013 at 01:39:42PM -0400, Chris Mason wrote: > > Quoting Dave Jones (2013-06-17 09:49:55) > > > Hit this while running this script in a loop.. > > > https://github.com/kernelslacker/io-tests/blob/master/se

Re: [3.10-rc6] WARNING: at fs/btrfs/inode.c:7961 btrfs_destroy_inode+0x265/0x2e0 [btrfs]()

2013-06-19 Thread Chris Mason
Quoting Dave Jones (2013-06-17 14:58:10) > On Mon, Jun 17, 2013 at 02:42:27PM -0400, Chris Mason wrote: > > Quoting Dave Jones (2013-06-17 14:20:06) > > > On Mon, Jun 17, 2013 at 01:39:42PM -0400, Chris Mason wrote: > > > > Quoting Dave Jones (2013-06-17 09:49:

Re: [3.10-rc6] WARNING: at fs/btrfs/inode.c:7961 btrfs_destroy_inode+0x265/0x2e0 [btrfs]()

2013-06-19 Thread Chris Mason
Quoting Dave Jones (2013-06-19 14:34:50) > On Wed, Jun 19, 2013 at 02:02:33PM -0400, Chris Mason wrote: > > Quoting Dave Jones (2013-06-17 14:58:10) > > > On Mon, Jun 17, 2013 at 02:42:27PM -0400, Chris Mason wrote: > > > > Quoting Dave Jones (2013-06-17 14:20:

<    6   7   8   9   10   11   12   13   14   15   >