From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS, USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 12B2FC43441 for ; Fri, 16 Nov 2018 14:06:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id CE3EB2089D for ; Fri, 16 Nov 2018 14:06:24 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="Za9bifmV" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CE3EB2089D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-block-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728124AbeKQAS5 (ORCPT ); Fri, 16 Nov 2018 19:18:57 -0500 Received: from mail.kernel.org ([198.145.29.99]:47768 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728088AbeKQAS4 (ORCPT ); Fri, 16 Nov 2018 19:18:56 -0500 Received: from localhost (unknown [207.164.22.36]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id AF0512086C; Fri, 16 Nov 2018 14:06:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1542377183; bh=0f/qJuc87VEwBPpSIY/eXHUqZvojAGi3ynPHswTY3BQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Za9bifmVpjAUWFAFoHY5CWPaZZ5Ybn8wRoQMBUlFuAi6kNkZp4ttd/yQjjO2Zm+wV QyY55s1kwms9moIcMeeTh70ZM44SsQ8bhmKtywJRoiYooqowjz2fgx37qHUqnsrZ/W hMnzSr9S59NNDAf4FEDVZeuU1qUOEgThuO4+Dy+o= Date: Fri, 16 Nov 2018 06:06:23 -0800 From: Greg Kroah-Hartman To: Ming Lei Cc: Jens Axboe , linux-block@vger.kernel.org, "jianchao.wang" , Guenter Roeck Subject: Re: [PATCH V2 for-4.21 2/2] blk-mq: alloc q->queue_ctx as normal array Message-ID: <20181116140623.GC4595@kroah.com> References: <20181116112311.4117-1-ming.lei@redhat.com> <20181116112311.4117-3-ming.lei@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181116112311.4117-3-ming.lei@redhat.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-block-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-block@vger.kernel.org On Fri, Nov 16, 2018 at 07:23:11PM +0800, Ming Lei wrote: > Now q->queue_ctx is just one read-mostly table for query the > 'blk_mq_ctx' instance from one cpu index, it isn't necessary > to allocate it as percpu variable. One simple array may be > more efficient. "may be", have you run benchmarks to be sure? If so, can you add the results of them to this changelog? If there is no measurable difference, then why make this change at all? thanks, greg k-h