From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ronald Moesbergen Subject: Re: [RESEND] [PATCH] readahead:add blk_run_backing_dev Date: Thu, 16 Jul 2009 09:32:48 +0200 Message-ID: References: <4A3CD62B.1020407@vlnb.net> <4A5395FD.2040507@vlnb.net> <4A5493A8.2000806@vlnb.net> <4A56FF32.2060303@vlnb.net> <4A570981.5080803@vlnb.net> <4A5CD3E2.2060307@vlnb.net> <4A5D7794.2070607@vlnb.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: fengguang.wu@intel.com, linux-kernel@vger.kernel.org, akpm@linux-foundation.org, kosaki.motohiro@jp.fujitsu.com, Alan.Brunelle@hp.com, linux-fsdevel@vger.kernel.org, jens.axboe@oracle.com, randy.dunlap@oracle.com, Bart Van Assche To: Vladislav Bolkhovitin Return-path: Received: from fg-out-1718.google.com ([72.14.220.159]:40150 "EHLO fg-out-1718.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751311AbZGPHct (ORCPT ); Thu, 16 Jul 2009 03:32:49 -0400 In-Reply-To: <4A5D7794.2070607@vlnb.net> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: 2009/7/15 Vladislav Bolkhovitin : >> The drop with 64 max_sectors_kb on the client is a consequence of how CFQ >> is working. I can't find the exact code responsible for this, but from all >> signs, CFQ stops delaying requests if amount of outstanding requests exceeds >> some threshold, which is 2 or 3. With 64 max_sectors_kb and 5 SCST I/O >> threads this threshold is exceeded, so CFQ doesn't recover order of >> requests, hence the performance drop. With default 512 max_sectors_kb and >> 128K RA the server sees at max 2 requests at time. >> >> Ronald, can you perform the same tests with 1 and 2 SCST I/O threads, >> please? Ok. Should I still use the file-on-xfs testcase for this, or should I go back to using a regular block device? The file-over-iscsi is quite uncommon I suppose, most people will export a block device over iscsi, not a file. > With context-RA patch, please, in those and future tests, since it should > make RA for cooperative threads much better. > >> You can limit amount of SCST I/O threads by num_threads parameter of >> scst_vdisk module. Ok, I'll try that and include the blk_run_backing_dev, readahead-context and io_context patches. Ronald.