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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id CA263C4332F for ; Tue, 15 Nov 2022 21:09:13 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 51AB96B0081; Tue, 15 Nov 2022 16:09:13 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 4CA908E0002; Tue, 15 Nov 2022 16:09:13 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3E11D6B0083; Tue, 15 Nov 2022 16:09:13 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 2F40B6B0081 for ; Tue, 15 Nov 2022 16:09:13 -0500 (EST) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id DCA3D80384 for ; Tue, 15 Nov 2022 21:09:12 +0000 (UTC) X-FDA: 80136916944.04.6F15EDB Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) by imf10.hostedemail.com (Postfix) with ESMTP id 68587C0007 for ; Tue, 15 Nov 2022 21:09:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=hgbPXbCYgLBsaW86LPk6uCa9uNA4J884/uF7RVk0T4o=; b=ir/apLSpJxjKbmRtx0X5CXb0L3 kKLJBc5vH0F8unE4m8uKiLIOJxBVXEZHVS1z/Ywxmpct0NEtAHm2pxd4fQB52r+KWGawAPh349yUx /AU3e2xOqSbzZasKV8YW/hHu2paKyrAixKcmoeoN+pgujdkO3+t8nxOwwtYfhKSvq77DAnWdpaDN3 oL3nzzKY+oqeuH4PkY0Ly7/N18tRAdqy0s7Hrf0/fFBtLauVZCQP+zNfkC5NeEm5tbzNvHpkZDdLg aOsvOQbWiVW9YYzkf61d9g+0xfb9Znj1wyycSRXqWMlBzjzQS8ntzZ3afpiF2cvMolUnXYnbwKCK7 FyJiDoPQ==; Received: from mcgrof by bombadil.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1ov3B6-00F1BI-IA; Tue, 15 Nov 2022 21:09:08 +0000 Date: Tue, 15 Nov 2022 13:09:08 -0800 From: Luis Chamberlain To: Song Liu Cc: bpf@vger.kernel.org, linux-mm@kvack.org, peterz@infradead.org, akpm@linux-foundation.org, x86@kernel.org, hch@lst.de, rick.p.edgecombe@intel.com, aaron.lu@intel.com, rppt@kernel.org Subject: Re: [PATCH bpf-next v2 0/5] execmem_alloc for BPF programs Message-ID: References: <20221107223921.3451913-1-song@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1668546551; a=rsa-sha256; cv=none; b=dYe7RfM7M66rh4Z/mBmV5ZZKPyt3EA14zbrNyJN3Lcf99oXlBVPpjsOk6/SAVi/of3lTir VWNwUDs3pZO7RFVRb9skdFW3ouz+MQI7l5Jz8wxO0KmxMPS2/8ICZKJrYiFpfWQJZa7gwj F6J6EfFl0yrDuWLKMaZ2+Gh9VtLRWfQ= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=infradead.org header.s=bombadil.20210309 header.b="ir/apLSp"; dmarc=fail reason="No valid SPF, DKIM not aligned (relaxed)" header.from=kernel.org (policy=none); spf=none (imf10.hostedemail.com: domain of mcgrof@infradead.org has no SPF policy when checking 198.137.202.133) smtp.mailfrom=mcgrof@infradead.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1668546551; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=hgbPXbCYgLBsaW86LPk6uCa9uNA4J884/uF7RVk0T4o=; b=nnb2gkRv8VUOs4XttGDo8/F8ePt4fWtBynrSMG/ALsZvOCTubtQeQSq89tw05Kc60M/DHN OPgd3LubXOR24a5Pix03Qv6z/baIwEN+2SYlgQKgB4iZksrNSRN2lS60tcgKuREQzsr35H h9GaBQLLyYNzTQSvHhkTN16ibJwD2go= X-Stat-Signature: xnf3x7msh8d583aqbwozo63wchgm8e7f X-Rspamd-Queue-Id: 68587C0007 X-Rspam-User: Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=infradead.org header.s=bombadil.20210309 header.b="ir/apLSp"; dmarc=fail reason="No valid SPF, DKIM not aligned (relaxed)" header.from=kernel.org (policy=none); spf=none (imf10.hostedemail.com: domain of mcgrof@infradead.org has no SPF policy when checking 198.137.202.133) smtp.mailfrom=mcgrof@infradead.org X-Rspamd-Server: rspam09 X-HE-Tag: 1668546551-938707 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Mon, Nov 14, 2022 at 05:30:39PM -0800, Song Liu wrote: > On Mon, Nov 7, 2022 at 2:41 PM Song Liu wrote: > > > > [...] > > > > > > > This set enables bpf programs and bpf dispatchers to share huge pages with > > new API: > > execmem_alloc() > > execmem_alloc() > > execmem_fill() > > > > The idea is similar to Peter's suggestion in [1]. > > > > execmem_alloc() manages a set of PMD_SIZE RO+X memory, and allocates these > > memory to its users. execmem_alloc() is used to free memory allocated by > > execmem_alloc(). execmem_fill() is used to update memory allocated by > > execmem_alloc(). > > Sigh, I just realized this thread made through linux-mm@kvack.org, but got > dropped by bpf@vger.kernel.org, so I guess I will have to resend v3. I don't know what is going on with the bpf list but whatever it is, is silly. You should Cc the right folks to ensure proper review if the bpf list is the issue. > Currently, I have got the following action items for v3: > 1. Add unify API to allocate text memory to motivation; > 2. Update Documentation/x86/x86_64/mm.rst; > 3. Allow none PMD_SIZE allocation for powerpc. - I am really exausted of asking again for real performance tests, you keep saying you can't and I keep saying you can, you are not trying hard enough. Stop thinking about your internal benchmark which you cannot publish. There should be enough crap out which you can use. - A new selftest or set of selftests which demonstrates gain in performance - Extensions maybe of lib/test_vmalloc.c or whatever is appropriate to test correctness - Enhance commit logs to justify the *why*, one of which to hightight is providing an API for memory semantics for special memory pages Luis