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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B8C17C7EE23 for ; Wed, 17 May 2023 06:20:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230127AbjEQGUS (ORCPT ); Wed, 17 May 2023 02:20:18 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36668 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232441AbjEQGUN (ORCPT ); Wed, 17 May 2023 02:20:13 -0400 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A4DD32728 for ; Tue, 16 May 2023 23:20:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=Uiam3MV1AIHx2JXhBYlardHpiKPPLXRwMMxlhPQWj/8=; b=lCMxRrDHuojZ0UPd/0UgioRB86 2fhjUmfD78FcEyAu/uz3qh+YysEvBlNn3xRL8Tsf6EbZ3JjOPy6WMGzZ6RC48vvcs2bufdXIbItkI QKDCuWVkywiVvHZ+0vIGTEaNwHLPKEoKjT+qbxKoNaOVlewfVtaLHA028AScpKU449bGeo8DpmSq4 cQtDOvl7rY08GfxHMJlOMTuitFwEx8Tde4zleVg3eoI6f+RJX2KGDrRE5MndVj+WaA6OXP010uY55 3MqslUAkzi3wJzPq957lHtT+WX0rw+ZkskTevkhRhbJUEDcY6KS57uu+alDGFxf0qPsuqmi3hmEFe v57ygwTA==; Received: from hch by bombadil.infradead.org with local (Exim 4.96 #2 (Red Hat Linux)) id 1pzAW7-008PAq-1I; Wed, 17 May 2023 06:20:07 +0000 Date: Tue, 16 May 2023 23:20:07 -0700 From: Christoph Hellwig To: Lorenzo Stoakes Cc: Ryan Roberts , Andrew Morton , "Matthew Wilcox (Oracle)" , "Kirill A. Shutemov" , SeongJae Park , linux-kernel@vger.kernel.org, linux-mm@kvack.org, damon@lists.linux.dev, Christoph Hellwig , Uladzislau Rezki Subject: Re: [RESEND PATCH v1 1/5] mm: vmalloc must set pte via arch code Message-ID: References: <20230511132113.80196-1-ryan.roberts@arm.com> <20230511132113.80196-2-ryan.roberts@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, May 13, 2023 at 02:14:18PM +0100, Lorenzo Stoakes wrote: > > Fixes: 3e9a9e256b1e ("mm: add a vmap_pfn function") > > Not sure if this is really 'fixing' anything, I mean ostensibly, but not > sure if the tag is relevant here, that is more so for a bug being > introduced, and unless an issue has arisen not sure if it's > appropriate. But this might be a nit, again! vmap_pfn was factored out of i915-specific code, which probably never ran on anything but x86 at that time. Now that intel builds plug in GPUs is probably could. But independent of that core mm code should use the proper APIs.