Re: [PATCH] btrfs: Use iocb to derive pos instead of passing a separate parameter

2018-06-24 Thread Misono Tomohiro
So, this is the updated version of https://patchwork.kernel.org/patch/10063039/ This time xfstest is ok and Reviewed-by: Misono Tomohiro On 2018/06/18 2:39, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > struct kiocb carries the ki_pos, so there is no need to pass it as > a separate

Re: [PATCH v2] btrfs: return EUCLEAN if extent_inline_ref type is invalid

2018-06-24 Thread Su Yue
On 06/25/2018 09:28 AM, Su Yue wrote: On 06/22/2018 05:40 PM, David Sterba wrote: On Fri, Jun 22, 2018 at 04:18:01PM +0800, Su Yue wrote: If type of extent_inline_ref found is not expected, filesystem may have been corrupted, should return EUCLEAN instead of EINVAL. No functional changes.

Re: [PATCH v2] btrfs: return EUCLEAN if extent_inline_ref type is invalid

2018-06-24 Thread Su Yue
On 06/22/2018 05:40 PM, David Sterba wrote: On Fri, Jun 22, 2018 at 04:18:01PM +0800, Su Yue wrote: If type of extent_inline_ref found is not expected, filesystem may have been corrupted, should return EUCLEAN instead of EINVAL. No functional changes. Signed-off-by: Su Yue --- Changelog:

Re: unsolvable technical issues?

2018-06-24 Thread Goffredo Baroncelli
On 06/23/2018 07:11 AM, Duncan wrote: > waxhead posted on Fri, 22 Jun 2018 01:13:31 +0200 as excerpted: > >> According to this: >> >> https://stratis-storage.github.io/StratisSoftwareDesign.pdf Page 4 , >> section 1.2 >> >> It claims that BTRFS still have significant technical issues that may >>

Re: unsolvable technical issues?

2018-06-24 Thread Ferry Toth
waxhead wrote: > Jukka Larja wrote: >> waxhead wrote 24.6.2018 klo 1.01: >>> Nikolay Borisov wrote: On 22.06.2018 02:13, waxhead wrote: > According to this: > > https://stratis-storage.github.io/StratisSoftwareDesign.pdf > Page 4 , section 1.2 > > It claims that

Re: unsolvable technical issues?

2018-06-24 Thread waxhead
Jukka Larja wrote: waxhead wrote 24.6.2018 klo 1.01: Nikolay Borisov wrote: On 22.06.2018 02:13, waxhead wrote: According to this: https://stratis-storage.github.io/StratisSoftwareDesign.pdf Page 4 , section 1.2 It claims that BTRFS still have significant technical issues that may never be