[PATCH] Btrfs: fix up seek_hole/seek_data handling V2

2013-10-30 Thread Josef Bacik
Whoever wrote this was braindead. Also it doesn't work right if you have VACANCY's since we assumed you would only have that at the end of the file, which won't be the case in the near future. I tested this with generic/285 and generic/286 as well as the btrfs tests that use fssum since it uses s

Re: [PATCH] Btrfs: fix up seek_hole/seek_data handling

2013-10-18 Thread Zach Brown
On Fri, Oct 18, 2013 at 10:11:15PM +0600, Roman Mamedov wrote: > On Fri, 18 Oct 2013 11:48:21 -0400 > Josef Bacik wrote: > > > Whoever wrote this was braindead. > > You do realize you sent a number of people googling for "Delalloc is such a > pain. If we have a hole and we have pending" just out

Re: [PATCH] Btrfs: fix up seek_hole/seek_data handling

2013-10-18 Thread Josef Bacik
On Fri, Oct 18, 2013 at 10:11:15PM +0600, Roman Mamedov wrote: > On Fri, 18 Oct 2013 11:48:21 -0400 > Josef Bacik wrote: > > > Whoever wrote this was braindead. > > You do realize you sent a number of people googling for "Delalloc is such a > pain. If we have a hole and we have pending" just out

Re: [PATCH] Btrfs: fix up seek_hole/seek_data handling

2013-10-18 Thread Roman Mamedov
On Fri, 18 Oct 2013 11:48:21 -0400 Josef Bacik wrote: > Whoever wrote this was braindead. You do realize you sent a number of people googling for "Delalloc is such a pain. If we have a hole and we have pending" just out of curiosity? :)) -- With respect, Roman signature.asc Description: PGP

[PATCH] Btrfs: fix up seek_hole/seek_data handling

2013-10-18 Thread Josef Bacik
Whoever wrote this was braindead. Also it doesn't work right if you have VACANCY's since we assumed you would only have that at the end of the file, which won't be the case in the near future. I tested this with generic/285 and generic/286 as well as the btrfs tests that use fssum since it uses s