Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp2181235pxb; Tue, 12 Oct 2021 00:43:03 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwJnINic4Qimt8SdrmV6ggX6SfNvuxVhxq4zWjTm85i5CaBV/sAXGct9F3WS0hzzUakbAaN X-Received: by 2002:a17:907:168c:: with SMTP id hc12mr914719ejc.570.1634024583652; Tue, 12 Oct 2021 00:43:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634024583; cv=none; d=google.com; s=arc-20160816; b=M3lxq3OeVHIZVWufSq0nmV59dWKM+Q6kbxJf9AYy3oA8c5vE7MXDLKbapfc0LmaCE1 iPZ8c3clfX5F2hKZCyrJHoe20+H8JyfPQlGrpmkHIVLakjxzd/tqlifirE9eSzWPUTbG 6hu9OjGZqRDx2b0qM1PPX+hfq5CI4hiGv+C9CUhsNYam2KjQ+EeoI5Il0GIp7EgY5Mkb LFlHjgXN5e4uQtEkjykDCUKvc6UADNyA9WiHJQDKuZ8KlG1UXg41WTmmMZcg6uMsgK9Z WoLPVen4tB7w4Pzj0g4I6NDh5jflMvJa3bEk5uyTOKo+WvhxQCvBb8Rla8nPmMmhjhTg DWbg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=DVWmp7ETySRZN2bTXV464m5d6VkNArMaleejegyuVsQ=; b=T07jk1ZBf+2RurcsiUF2n511TgXxbmB3+AuLiP4mzqyKQ5mJNw1+Jwn42OUZOTQ9/v hzGyFV3SvbGaBPW9zGtCJ0Mm4hLsagtpEV7NPRU0ZIFycOn/x+3AvgYR7FdGirzIZpZa 226E3BAyOn4sB5Pw0yq+lA73d3/zdr8c8CqefyyfMYGo0R27EwhIRp4QJ3cXaFZwC/ZV KOOrS+vAI9jRpExL2fKHfZCYs2OQqhs7lTWVhmzQJxHR15w711btFOKJOa+4lFqgBkEu 7yIkRs1JCtZvZLUhkn/bFXplXAM9nIIUhcgI5WYyRZ5mA67k4XtwnnvuO2AQPQyZrGO9 GHHQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ne34si24522367ejc.482.2021.10.12.00.42.39; Tue, 12 Oct 2021 00:43:03 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234082AbhJLHk1 (ORCPT + 99 others); Tue, 12 Oct 2021 03:40:27 -0400 Received: from szxga03-in.huawei.com ([45.249.212.189]:25177 "EHLO szxga03-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232565AbhJLHkZ (ORCPT ); Tue, 12 Oct 2021 03:40:25 -0400 Received: from dggemv711-chm.china.huawei.com (unknown [172.30.72.54]) by szxga03-in.huawei.com (SkyGuard) with ESMTP id 4HT6v021wXz8tWb; Tue, 12 Oct 2021 15:37:16 +0800 (CST) Received: from dggpemm500005.china.huawei.com (7.185.36.74) by dggemv711-chm.china.huawei.com (10.1.198.66) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.8; Tue, 12 Oct 2021 15:38:21 +0800 Received: from [10.69.30.204] (10.69.30.204) by dggpemm500005.china.huawei.com (7.185.36.74) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.2308.8; Tue, 12 Oct 2021 15:38:15 +0800 Subject: Re: [PATCH net-next -v5 3/4] mm: introduce __get_page() and __put_page() To: Ilias Apalodimas , Jesper Dangaard Brouer CC: John Hubbard , , , , , , , , , , , , , , , , , , , , , , , , , , References: <20211009093724.10539-1-linyunsheng@huawei.com> <20211009093724.10539-4-linyunsheng@huawei.com> <62106771-7d2a-3897-c318-79578360a88a@nvidia.com> <89bcc42a-ad95-e729-0748-bf394bf770be@redhat.com> From: Yunsheng Lin Message-ID: <3bba942e-eefd-7ac2-7a8c-b6c349641dd4@huawei.com> Date: Tue, 12 Oct 2021 15:38:15 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.69.30.204] X-ClientProxiedBy: dggeme702-chm.china.huawei.com (10.1.199.98) To dggpemm500005.china.huawei.com (7.185.36.74) X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2021/10/11 20:29, Ilias Apalodimas wrote: > On Mon, Oct 11, 2021 at 02:25:08PM +0200, Jesper Dangaard Brouer wrote: >> >> >> On 09/10/2021 21.49, John Hubbard wrote: >>> So in case it's not clear, I'd like to request that you drop this one >>> patch from your series. >> >> In my opinion as page_pool maintainer, you should also drop patch 4/4 from >> this series. >> >> I like the first two patches, and they should be resend and can be applied >> without too much further discussion. > > +1 Ok, it seems there is a lot of contention about how to avoid calling compound_head() now. Will send out the uncontroversial one first. > That's what I hinted on the previous version. The patches right now go way > beyond the spec of page pool. We are starting to change core networking > functions and imho we need a lot more people involved in this discussion, > than the ones participating already. > > As a general note and the reason I am so hesitant, is that we are starting > to violate layers here (at least in my opinion). When the recycling was > added, my main concern was to keep the network stack unaware (apart from > the skb bit). Now suddenly we need to teach frag_ref/unref internal page Maybe the skb recycle bit is a clever way to avoid dealing with the network stack directly. But that bit might also introduce or hide some problem, like the data race as pointed out by Alexander, and the odd using of page pool in mlx5 driver. > pool counters and that doesn't feel right. We first need to prove the race > can actually happen, before starting to change things. As the network stack is adding a lot of performance improvement, such as sockmap for BPF, which may cause problem for them, will dig more to prove that. > > Regards > /Ilias >> >> --Jesper >> > . >