Re: [PATCH 1/3] generic/304: fix high offset

2016-02-17 Thread Christoph Hellwig
On Wed, Feb 17, 2016 at 10:04:23AM +, Filipe Manana wrote: > > What does btrfs return here? > > -EINVAL, so this makes the test pass on btrfs. Ok, looks fine to me then: Reviewed-by: Christoph Hellwig -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in

Re: [PATCH 1/3] generic/304: fix high offset

2016-02-17 Thread Filipe Manana
On Wed, Feb 17, 2016 at 9:56 AM, Christoph Hellwig wrote: > On Mon, Feb 15, 2016 at 12:46:14PM -0800, Darrick J. Wong wrote: >> "Invalid argument" is a better response to an impossibly high offset >> dedupe request than "extents don't match", so change the test. > > What does

Re: [PATCH 1/3] generic/304: fix high offset

2016-02-17 Thread Christoph Hellwig
On Mon, Feb 15, 2016 at 12:46:14PM -0800, Darrick J. Wong wrote: > "Invalid argument" is a better response to an impossibly high offset > dedupe request than "extents don't match", so change the test. What does btrfs return here? -- To unsubscribe from this list: send the line "unsubscribe

[PATCH 1/3] generic/304: fix high offset

2016-02-15 Thread Darrick J. Wong
"Invalid argument" is a better response to an impossibly high offset dedupe request than "extents don't match", so change the test. Signed-off-by: Darrick J. Wong --- tests/generic/304.out |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git