Received: by 2002:a05:6a10:2726:0:0:0:0 with SMTP id ib38csp1080192pxb; Wed, 6 Apr 2022 08:14:40 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzJ24WRz7wWxy0W9RhGacwaHzeNmV0rZiShNMIbyGgtk4A7LYjuNLoK06W3JLP8+s1HsKm6 X-Received: by 2002:a17:902:bb85:b0:154:9a18:4a6d with SMTP id m5-20020a170902bb8500b001549a184a6dmr9030757pls.62.1649258079681; Wed, 06 Apr 2022 08:14:39 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1649258079; cv=none; d=google.com; s=arc-20160816; b=bmhoYxzTiOhyUAKgXP0r1nIFmxmcESWoJeptwqIGyiwVd5Y5PDza9gT38U7pBeilPQ LC5nwzwAZUrqliKwowsIYOqPOJMca6FtXf8OiG8pm38jX9QIHO5l9TgtRJl5CXSHeA8F 0VKpeBalk7w73tH8gt49X5zVmtmFD+CRrQcfsJmFogWOcPxtJIsjdR0+EI1xHq66hBBL JBF2CbkBm8G8z0MMFESd32MsGfE3ushOV3g4gE74Dfp8pljQqMB+e9evVdUK8ZmOlY/W qIF/sA+yveIM+0/myZeFCaQ4QUPw60W75RpykXeUF1xGWDZ2pTthC9Yx8DesUQLNqvwt qwWg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :organization:from:references:cc:to:content-language:subject :user-agent:mime-version:date:message-id:dkim-signature :wdcironportexception:ironport-sdr:ironport-sdr:dkim-signature; bh=hrSKUWKqRSfbyKandFWNO6+rdumsHAXtOlhhIa0b/YA=; b=FICQjjFaLZ04l2HEe5bORzhdyJAcWdgP9xVhwT+h6Z+077fg6hbU/JHcqgKgRAoUgg 0+D8rL08IS63X0c+UwC9RIZf7c6iA0jzcrW4D1u/cIda9oAOBGyA2VtWlz1wIR6902yt w2aDLZQa2yH0am1F7E9w4GNtn7TPr3oz3rNnrJrmdjDV92Q/TlgW6xBxWZgRI2N+ujYC PaxZYSie0YFfA5EBr8xb87HyQxHuNpFByXm76fqVNBYMoEWbAalQ6hojb2oOBMnAqymN VxtdrGteb3u5uqWLC6N678HOIZbi6zU78xXAIGZbOVKJPjaBPHV7JN0R9YtNlNYg84FB mLvg== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=gpvrCNsm; dkim=pass header.i=@opensource.wdc.com header.s=dkim header.b=E4UOnw8B; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=opensource.wdc.com Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [2620:137:e000::1:18]) by mx.google.com with ESMTPS id j14-20020a63b60e000000b003816043f0cdsi15470402pgf.706.2022.04.06.08.14.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Apr 2022 08:14:39 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18; Authentication-Results: mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=gpvrCNsm; dkim=pass header.i=@opensource.wdc.com header.s=dkim header.b=E4UOnw8B; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=opensource.wdc.com Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 25D1B37CD3D; Wed, 6 Apr 2022 06:04:50 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231706AbiDFNG3 (ORCPT + 99 others); Wed, 6 Apr 2022 09:06:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56536 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231888AbiDFNGP (ORCPT ); Wed, 6 Apr 2022 09:06:15 -0400 Received: from esa3.hgst.iphmx.com (esa3.hgst.iphmx.com [216.71.153.141]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BFB3F41FD07 for ; Tue, 5 Apr 2022 18:48:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1649209693; x=1680745693; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=VuoVmwsM1kjAIgmEzd4iWUN0hXYBPqUPYmlYGxx90q0=; b=gpvrCNsmTmfW/cXQ+nQCVquU/8tcBIfxMUbDV/oa/zfnOiTYaS8/Th7x XrJzOykZ74ZB3tUxJySl1pg8ymyWCJkuloAQSih3z4XOKcpNelq/BWx0U VXPLAKfGwDkx0Rnqg78IAxjmhMcH3glMKISWooYmfJZ7KFRJZtOWlTF1Z dzVHjoqMpw2HMZP0GN4LKwZnnDqEzeqgLILFz4rnwVZwFZ16g6Pko5Z7P 37sRKhw2MfHaIIqwnvtVAmC8bxu8qIaH35r/f2wk+5Er24FuQYbAMLaRf Aiceg859qK/byZ0tG8iMhZbsVkweFQY1FkOfEtNAdcZWOn/Fy3X/yUrYQ g==; X-IronPort-AV: E=Sophos;i="5.90,238,1643644800"; d="scan'208";a="202037232" Received: from h199-255-45-15.hgst.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 06 Apr 2022 09:48:06 +0800 IronPort-SDR: YYT6ODKAVJo7ucncVjl9vKjgaQLgQNC8X3mHPcZGuhjT+Qi8m9L5Tgpq9ZOpDYOof3MQbM+buO 6uZCf8ki72FNOYfLb/y6F8c1T+56EMGVfe/6LbhJwYqqz8Myg8yRP4jQh5yRCEQSmaaE8iqjbi uejsfsoag0mKMpIZyM90JoIv2sYE/mOIJr7USwU21OPtIwjMbOFAIZV87fh9Gc1yUl7K+rAIDC GBxOjhWCagcHo7jKeX2IDcVigsyCyvCPjmWt1HHmu+pr0YtcGyhrCO2+6F036Q5gVfmDuthFzr GevYmk98E4dXn7mX0ZVbZYh9 Received: from uls-op-cesaip02.wdc.com ([10.248.3.37]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 05 Apr 2022 18:18:49 -0700 IronPort-SDR: Z8TzV4KQr29ZzqbnvJ6olDsbWoj7lzZVvf3Mj7zPwTBY0NjO5hQexC/CwxuJ8GWNP4RC4SvY6q 6UFfeApI3iuCwbrL4dmo/UbHiCwRvHZMYVVvD4qN+04BnVG6EqCE2uggrkF1KqP7RKJLQFdF94 sdHrrnvl3dBaucfWc61cY+TDSfCNBtk43oFamQ9YNjOhC5xl8adwKdYFc0uH3Lv+8NssonHnVN ILNzvSrBUnVOcWEpt5rRyCRThkNZsTQ5W/Gq19E10ilKc6vYj0y+/0UqG8qSRKIrqP3WgAxUng Afs= WDCIronportException: Internal Received: from usg-ed-osssrv.wdc.com ([10.3.10.180]) by uls-op-cesaip02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 05 Apr 2022 18:48:06 -0700 Received: from usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTP id 4KY6ps4pBfz1SVnx for ; Tue, 5 Apr 2022 18:48:05 -0700 (PDT) Authentication-Results: usg-ed-osssrv.wdc.com (amavisd-new); dkim=pass reason="pass (just generated, assumed good)" header.d=opensource.wdc.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d= opensource.wdc.com; h=content-transfer-encoding:content-type :in-reply-to:organization:from:references:to:content-language :subject:user-agent:mime-version:date:message-id; s=dkim; t= 1649209685; x=1651801686; bh=VuoVmwsM1kjAIgmEzd4iWUN0hXYBPqUPYml YGxx90q0=; b=E4UOnw8BjAuFsxgUNz1JVDSZoX8yQLctisFANLe1Z/7C0dFuEJk LfdRZDbW8aSOdPPRAKOLeHbtE6mmG+YYKwab0H3EYIozRVzMTdKBgE2ik7HJHuwe 1b/tZXQrhQEBKn14uwEUh3KXtkYU3AwYxZb/cftSIWs4sLb5Ii1/2OmbJOA5JmTM pMTGewU2Q4eeocqSTsJgcr3aFdUI/EEHKR3ZwRZqyhl210d7gKEwFqJzkMZnFvWx sVkMFEpQFSvwkCv/cyub/aWGqmuHgeyvyl77QlJzkoPLP94S2ray/pI2ibZtgHr6 MBrtS9l0xhBIHxhWo4d1BYM4S1B2CX7v4tQ== X-Virus-Scanned: amavisd-new at usg-ed-osssrv.wdc.com Received: from usg-ed-osssrv.wdc.com ([127.0.0.1]) by usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id js7mOVg80Fvw for ; Tue, 5 Apr 2022 18:48:05 -0700 (PDT) Received: from [10.149.53.254] (washi.fujisawa.hgst.com [10.149.53.254]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTPSA id 4KY6pr31hmz1Rvlx; Tue, 5 Apr 2022 18:48:04 -0700 (PDT) Message-ID: Date: Wed, 6 Apr 2022 10:48:03 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: [PATCH v2 2/2] libata: Inline ata_qc_new_init() in ata_scsi_qc_new() Content-Language: en-US To: Christoph Hellwig , John Garry Cc: linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, linux-doc@vger.kernel.org, linux-scsi@vger.kernel.org References: <1649083990-207133-1-git-send-email-john.garry@huawei.com> <1649083990-207133-3-git-send-email-john.garry@huawei.com> <20220405055252.GA23698@lst.de> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <20220405055252.GA23698@lst.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-4.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE, T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 4/5/22 14:52, Christoph Hellwig wrote: > On Mon, Apr 04, 2022 at 10:53:10PM +0800, John Garry wrote: >> From: Christoph Hellwig >> >> It is a bit pointless to have ata_qc_new_init() in libata-core.c since it >> pokes scsi internals, so inline it in ata_scsi_qc_new() (in libata-scsi.c). >> >> >> [jpg, Take Christoph's change from list and form into a patch] >> Signed-off-by: John Garry > > Signed-off-by: Christoph Hellwig > > Although I still think merging the two patches into one to avoid all > the churn would be much better. I agree. Let's merge these 2 patches. -- Damien Le Moal Western Digital Research