Re: [PATCH] Remove certain calls for releasing page cache

2014-07-31 Thread Hugo Mills
On Wed, Jul 30, 2014 at 10:05:16PM -0400, Nick Krause wrote: On Wed, Jul 30, 2014 at 7:30 PM, Dave Airlie airl...@gmail.com wrote: This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in the compression rountines of btrfs.

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-31 Thread Nick Krause
On Thu, Jul 31, 2014 at 6:11 AM, Hugo Mills h...@carfax.org.uk wrote: On Wed, Jul 30, 2014 at 10:05:16PM -0400, Nick Krause wrote: On Wed, Jul 30, 2014 at 7:30 PM, Dave Airlie airl...@gmail.com wrote: This patch removes the lines for releasing the page cache in certain files as this may aid

[PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Nicholas Krause
This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in the compression rountines of btrfs. Please note that this patch has not been tested on my own hardware due to no compression based btrfs volumes of my own. Signed-off-by:

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Josef Bacik
On 07/30/2014 04:42 PM, Nicholas Krause wrote: This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in the compression rountines of btrfs. Please note that this patch has not been tested on my own hardware due to no compression

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Zach Brown
On Wed, Jul 30, 2014 at 04:47:12PM -0400, Josef Bacik wrote: On 07/30/2014 04:42 PM, Nicholas Krause wrote: This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in the compression rountines of btrfs. Please note that this patch

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Nick Krause
On Wed, Jul 30, 2014 at 4:51 PM, Zach Brown z...@zabbo.net wrote: On Wed, Jul 30, 2014 at 04:47:12PM -0400, Josef Bacik wrote: On 07/30/2014 04:42 PM, Nicholas Krause wrote: This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Josef Bacik
So I'm going to list all the things that are wrong with this in the off chance that you will actually improve and then go back to ignoring your emails. 1) You test your patches before you send them. Xfstests is how you test. 2) You seem to think page_cache_release releases the page cache. It

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Dave Airlie
This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in the compression rountines of btrfs. Please note that this patch has not been tested on my own hardware due to no compression based btrfs volumes of my own. For all that is

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Nick Krause
On Wed, Jul 30, 2014 at 7:30 PM, Dave Airlie airl...@gmail.com wrote: This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in the compression rountines of btrfs. Please note that this patch has not been tested on my own hardware

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Dave Airlie
On 31 July 2014 12:05, Nick Krause xerofo...@gmail.com wrote: On Wed, Jul 30, 2014 at 7:30 PM, Dave Airlie airl...@gmail.com wrote: This patch removes the lines for releasing the page cache in certain files as this may aid in perfomance with writes in the compression rountines of btrfs. Please

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Nick Krause
On Wed, Jul 30, 2014 at 11:57 PM, Dave Airlie airl...@gmail.com wrote: On 31 July 2014 12:05, Nick Krause xerofo...@gmail.com wrote: On Wed, Jul 30, 2014 at 7:30 PM, Dave Airlie airl...@gmail.com wrote: This patch removes the lines for releasing the page cache in certain files as this may aid

Re: [PATCH] Remove certain calls for releasing page cache

2014-07-30 Thread Duncan
Nick Krause posted on Thu, 31 Jul 2014 00:14:50 -0400 as excerpted: I am wondering if some one can give me a pointer on where to download the git repos for the btrfs user spaces tools as it seems on the btrfs tools on the wiki.