Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758239Ab0AOU0K (ORCPT ); Fri, 15 Jan 2010 15:26:10 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1758196Ab0AOU0J (ORCPT ); Fri, 15 Jan 2010 15:26:09 -0500 Received: from mx1.redhat.com ([209.132.183.28]:28661 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758184Ab0AOU0H convert rfc822-to-8bit (ORCPT ); Fri, 15 Jan 2010 15:26:07 -0500 From: Jeff Moyer To: Corrado Zoccolo Cc: "Zhang\, Yanmin" , Shaohua Li , "linux-kernel\@vger.kernel.org" , "jens.axboe\@oracle.com" , "Zhang\, Yanmin" Subject: Re: [PATCH]cfq-iosched: don't take requests with long distence as close References: <20091224005506.GA7879@sli10-desk.sh.intel.com> <20091228092844.GA9710@sli10-desk.sh.intel.com> <4e5e476b1001070544w88a387dkfb48847f4f95a9b1@mail.gmail.com> <1263187209.29897.33.camel@localhost> <4e5e476b1001110705y6c3319ducf6a15c2a2be5670@mail.gmail.com> <1263264218.29897.41.camel@localhost> <4e5e476b1001151132l406054f6xcc47b94e817d3af0@mail.gmail.com> <4e5e476b1001151224x5c94382cnf2a6696f10c9800d@mail.gmail.com> X-PGP-KeyID: 1F78E1B4 X-PGP-CertKey: F6FE 280D 8293 F72C 65FD 5A58 1FF8 A7CA 1F78 E1B4 X-PCLoadLetter: What the f**k does that mean? Date: Fri, 15 Jan 2010 15:26:00 -0500 In-Reply-To: <4e5e476b1001151224x5c94382cnf2a6696f10c9800d@mail.gmail.com> (Corrado Zoccolo's message of "Fri, 15 Jan 2010 21:24:23 +0100") Message-ID: User-Agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Corrado Zoccolo writes: > On Fri, Jan 15, 2010 at 8:45 PM, Jeff Moyer wrote: >> Corrado Zoccolo writes: >> >>> Hi Jeff, >>> I think this patch has the same flaw as Shaohua's. >>> The seekiness check that you introduce in cfq_rq_close is already >>> present in its caller, cfq_close_cooperator, so it is not effective. >> >> I don't think so.  There are two queues, here.  One queue is checked by >> the caller, and that is the cur_cfqq.  The __cfqq needs to also be >> checked. > The other one, i.e. the returned one, is also checked by the caller, > some lines below. haha. OK, I guess I wrote it right the first time? ;-) -Jeff -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/