Re: Please add more info to dmesg output on I/O error

2017-03-01 Thread Timofey Titovets
2017-03-02 3:40 GMT+03:00 Chris Murphy : > On Wed, Mar 1, 2017 at 12:38 PM, Kai Krakow wrote: >> Am Wed, 1 Mar 2017 19:04:26 +0300 >> schrieb Timofey Titovets : >> >>> Hi, today i try move my FS from old HDD to new SSD >>> While processing i catch I/O error and device remove operation was >>> canc

Re: Please add more info to dmesg output on I/O error

2017-03-01 Thread Chris Murphy
On Wed, Mar 1, 2017 at 12:38 PM, Kai Krakow wrote: > Am Wed, 1 Mar 2017 19:04:26 +0300 > schrieb Timofey Titovets : > >> Hi, today i try move my FS from old HDD to new SSD >> While processing i catch I/O error and device remove operation was >> canceled >> >> Dmesg: >> [ 1015.010241] blk_update_re

Re: [4.7.2] btrfs_run_delayed_refs:2963: errno=-17 Object already exists

2017-03-01 Thread Qu Wenruo
At 02/02/2017 08:01 PM, Marc Joliet wrote: On Sunday 28 August 2016 15:29:08 Kai Krakow wrote: Hello list! Hi list [kernel message snipped] Btrfs --repair refused to repair the filesystem telling me something about compressed extents and an unsupported case, wanting me to take an image a

raid1 degraded mount still produce single chunks, writeable mount not allowed

2017-03-01 Thread Chris Murphy
[1717713.408675] BTRFS warning (device dm-8): missing devices (1) exceeds the limit (0), writeable mount is not allowed [1717713.446453] BTRFS error (device dm-8): open_ctree failed [chris@f25s ~]$ uname -r 4.9.8-200.fc25.x86_64 I thought this was fixed. I'm still getting a one time degraded rw m

Re: Please add more info to dmesg output on I/O error

2017-03-01 Thread Chris Murphy
On Wed, Mar 1, 2017 at 9:04 AM, Timofey Titovets wrote: > Hi, today i try move my FS from old HDD to new SSD > While processing i catch I/O error and device remove operation was canceled > > Dmesg: > [ 1015.010241] blk_update_request: I/O error, dev sda, sector 81353664 > [ 1015.010246] BTRFS erro

Re: [PATCH 1/8] nowait aio: Introduce IOCB_FLAG_NOWAIT

2017-03-01 Thread Christoph Hellwig
On Wed, Mar 01, 2017 at 10:57:17AM -0600, Goldwyn Rodrigues wrote: > RWF_* ? Isn't that kernel space flags? Or did you intend to say > IOCB_FLAG_*? No, they are the flags for preadv2/pwritev2. > If yes, we maintain two flag fields? aio_reserved1 (perhaps > renamed to aio_flags2) and aio_flags? Y

Re: assertion failed: last_size == new_size, file: fs/btrfs/inode.c

2017-03-01 Thread Dave Jones
On Tue, Feb 28, 2017 at 05:12:01PM -0800, Liu Bo wrote: > On Mon, Feb 27, 2017 at 11:23:42AM -0500, Dave Jones wrote: > > On Mon, Feb 27, 2017 at 07:53:48AM -0800, Liu Bo wrote: > > > On Sun, Feb 26, 2017 at 07:18:42PM -0500, Dave Jones wrote: > > > > Hitting this fairly frequently.. I'm not

Re: Please add more info to dmesg output on I/O error

2017-03-01 Thread Kai Krakow
Am Wed, 1 Mar 2017 19:04:26 +0300 schrieb Timofey Titovets : > Hi, today i try move my FS from old HDD to new SSD > While processing i catch I/O error and device remove operation was > canceled > > Dmesg: > [ 1015.010241] blk_update_request: I/O error, dev sda, sector 81353664 > [ 1015.010246] BT

Re: [4.7.2] btrfs_run_delayed_refs:2963: errno=-17 Object already exists

2017-03-01 Thread Marc Joliet
On Wednesday 01 March 2017 19:14:07 Marc Joliet wrote: > > > Also, the image is complete, so I only need to find somewhere where I > > > can > > > upload a 9.4 GB file. > > > > > > > > Is it a compressed dump? Dumped with btrfs-image -c9? > > It was created with: > > btrfs-image -s -w /dev/sdb2

Re: [PATCH] btrfs-progs: docs: fix many typos, plus three edits for clarit

2017-03-01 Thread David Sterba
On Tue, Feb 21, 2017 at 06:14:37PM -0500, Nicholas D Steeves wrote: > Hi David, > > Please see attached a reasonably thorough patch for all the typos I > could find in btrfs-progs documentation. The three edits are very > minor and look larger than they are because I had to reflow the > paragraph

Re: [4.7.2] btrfs_run_delayed_refs:2963: errno=-17 Object already exists

2017-03-01 Thread Marc Joliet
On Wednesday 01 March 2017 19:14:07 Marc Joliet wrote: > > And btrfs check --mode=lowmem is also recommended as in some rare case, > > low mem mode can detect bug which original mode doesn't. > > I did see differences in output the last time around (again, see my > previous messages in this threa

Re: [4.7.2] btrfs_run_delayed_refs:2963: errno=-17 Object already exists

2017-03-01 Thread Marc Joliet
On Wednesday 01 March 2017 17:32:35 Qu Wenruo wrote: > At 03/01/2017 04:23 PM, Marc Joliet wrote: > > On Tuesday 28 February 2017 23:14:54 Marc Joliet wrote: > >> I think I'm at that point now myself, unless anybody has any other ideas. > > > > For example, could the --init-extent-tree option to b

Re: [PATCH 1/8] nowait aio: Introduce IOCB_FLAG_NOWAIT

2017-03-01 Thread Goldwyn Rodrigues
On 03/01/2017 09:56 AM, Christoph Hellwig wrote: > On Wed, Mar 01, 2017 at 07:36:48AM -0800, Christoph Hellwig wrote: >> Given that we aren't validating aio_flags in older kernels we can't >> just add this flag as it will be a no-op in older kernels. I think >> we will have to add IOCB_CMD_PREAD

Re: [PATCH v3 00/12] lowmem mode fixes

2017-03-01 Thread David Sterba
On Tue, Feb 21, 2017 at 04:34:26PM +0800, Qu Wenruo wrote: > Patches can be fetch from github: > https://github.com/adam900710/btrfs-progs/tree/lowmem_fixes > > Thanks for reports from Chris Murphy and Christoph Anton Mitterer, > several new bugs are exposed for lowmem mode fsck. > > Special than

Re: [PATCH v3 10/12] btrfs-progs: fsck-test: Add new test case for file extent false alerts

2017-03-01 Thread David Sterba
On Tue, Feb 21, 2017 at 04:34:36PM +0800, Qu Wenruo wrote: > +check_global_prereq xfs_io > + run_check xfs_io -c "pwrite 0 64K" "$TEST_MNT/file" > + run_check xfs_io -f -c "pwrite 0 1K" "$TEST_MNT/file" I've converted this to dd as we don't want to depend on xfsprogs installed for the te

Please add more info to dmesg output on I/O error

2017-03-01 Thread Timofey Titovets
Hi, today i try move my FS from old HDD to new SSD While processing i catch I/O error and device remove operation was canceled Dmesg: [ 1015.010241] blk_update_request: I/O error, dev sda, sector 81353664 [ 1015.010246] BTRFS error (device sdb1): bdev /dev/sda1 errs: wr 0, rd 23, flush 0, corrupt

Re: [PATCH] btrfs-progs: send-dump: add missing newlines

2017-03-01 Thread David Sterba
On Wed, Feb 22, 2017 at 10:32:46PM +0100, Benedikt Morbach wrote: > make sure to include newlines after commands that have only one > argument, such as 'unlink' or 'mkfile' > > changes > > unlink ./baz.0/file_autimes ./baz.0/ > atime=2017-02-22T11:59:

Re: [PATCH 1/8] nowait aio: Introduce IOCB_FLAG_NOWAIT

2017-03-01 Thread Christoph Hellwig
On Wed, Mar 01, 2017 at 07:36:48AM -0800, Christoph Hellwig wrote: > Given that we aren't validating aio_flags in older kernels we can't > just add this flag as it will be a no-op in older kernels. I think > we will have to add IOCB_CMD_PREADV2/IOCB_CMD_WRITEV2 opcodes that > properly validate all

Re: [PATCH v3] btrfs: remove btrfs_err_str function from uapi/linux/btrfs.h

2017-03-01 Thread David Sterba
On Wed, Mar 01, 2017 at 02:12:50AM +0300, Dmitry V. Levin wrote: > btrfs_err_str function is not called from anywhere and is replicated > in the userspace headers for btrfs-progs. > > It's removal also fixes the following linux/btrfs.h userspace > compilation error: > > /usr/include/linux/btrfs.h

Re: [PATCH 7/8] nowait aio: xfs

2017-03-01 Thread Christoph Hellwig
> @@ -528,12 +528,17 @@ xfs_file_dio_aio_write( > ((iocb->ki_pos + count) & mp->m_blockmask)) { > unaligned_io = 1; > iolock = XFS_IOLOCK_EXCL; > + if (iocb->ki_flags & IOCB_NOWAIT) > + return -EAGAIN; So all unaligned I/O will

Re: [PATCH 3/8] nowait aio: return if direct write will trigger writeback

2017-03-01 Thread Christoph Hellwig
On Tue, Feb 28, 2017 at 07:46:06PM -0800, Matthew Wilcox wrote: > Ugh, this is pretty inefficient. If that's all you want to know, then > using the radix tree directly will be far more efficient than spinning > up all the pagevec machinery only to discard the pages found. > > But what's going to

Re: [PATCH 1/8] nowait aio: Introduce IOCB_FLAG_NOWAIT

2017-03-01 Thread Christoph Hellwig
On Tue, Feb 28, 2017 at 05:36:03PM -0600, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > This flag informs kernel to bail out if an AIO request will block > for reasons such as file allocations, or a writeback triggered, > or would block while allocating requests while performing > direct

Re: [PATCH 2/8] nowait aio: Return if cannot get hold of i_rwsem

2017-03-01 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig -- 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: [PULL] Btrfs cleanups for 4.11, part 2

2017-03-01 Thread David Sterba
On Wed, Mar 01, 2017 at 09:36:38AM +0200, Nikolay Borisov wrote: > Kbuild reported the following warning: > > fs/btrfs/scrub.c: In function 'check_extent_to_block': > >> fs/btrfs/scrub.c:4259:24: error: passing argument 1 of 'btrfs_get_extent' > >> from incompatible pointer type [-Werror=incomp

Re: [4.7.2] btrfs_run_delayed_refs:2963: errno=-17 Object already exists

2017-03-01 Thread Qu Wenruo
At 03/01/2017 04:23 PM, Marc Joliet wrote: On Tuesday 28 February 2017 23:14:54 Marc Joliet wrote: I think I'm at that point now myself, unless anybody has any other ideas. For example, could the --init-extent-tree option to btrfs-check help, given that I needed to pass -w to btrfs-image?

Re: [PATCH 2/2] Btrfs: remove start_pos

2017-03-01 Thread Qu Wenruo
At 03/01/2017 09:04 AM, Liu Bo wrote: @pos, not aligned @start_pos, should be used to check whether the eof page needs to be marked as readonly, thus @start_pos can be removed. Signed-off-by: Liu Bo --- fs/btrfs/file.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a

Re: [PULL] Btrfs cleanups for 4.11, part 2

2017-03-01 Thread Nikolay Borisov
On 1.03.2017 00:35, Chris Mason wrote: > > > On 02/28/2017 10:09 AM, David Sterba wrote: >> Hi, >> >> this is the second half of the 4.11 batch, the rest of the cleanups. >> Please >> pull, thanks. >> >> The following changes since commit >> 6288d6eabc7505f42dda34a2c2962f91914be3a4: >> >> Bt

Re: [4.7.2] btrfs_run_delayed_refs:2963: errno=-17 Object already exists

2017-03-01 Thread Marc Joliet
On Tuesday 28 February 2017 23:14:54 Marc Joliet wrote: > I think I'm at that point now myself, unless anybody has any other ideas. For example, could the --init-extent-tree option to btrfs-check help, given that I needed to pass -w to btrfs-image? Also, the image is complete, so I only need to