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=-9.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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 5868EC4338F for ; Thu, 5 Aug 2021 19:04:13 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 2622960EEA for ; Thu, 5 Aug 2021 19:04:13 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 2622960EEA Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Cc:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=FB4bhIruhOXibQ3sfBYhURPU/GQO2YgMtayCZh2RBYk=; b=FU4xP4lCgYHvnM SlmwzwvJNsAWuSh/GehZGWMdfd0q03JuQXt/vXk/aoBCIeAO8Slr8QIFk/5O4P6vPbpWNqvkvjzND geeSHj+yYY4/+nQMgCMO/E0xXyYegN3ubpDh8wi/LHgHX7+230wT5OgATOeYBwSje5ChKhLpmbbCT xpBA8AHn89VDjUyZaVOuqGR1golwMSSM3SIINiHv8yA1iM9Fah15w5OgdGqCeInCv2SKy1izUN4Ec rKBbK+ULMUnfRDS/iu11TEJgByCXW7eHeRTcZemzSwTSYh0LL65kyIrG6KOU6AZ33mUzb5CRPwJOZ NTrr2Odcs+hWFMCh4QQQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mBidX-00AbBW-TO; Thu, 05 Aug 2021 19:02:36 +0000 Received: from mail-oi1-f172.google.com ([209.85.167.172]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mBidT-00Ab96-QW for linux-arm-kernel@lists.infradead.org; Thu, 05 Aug 2021 19:02:33 +0000 Received: by mail-oi1-f172.google.com with SMTP id t128so8739231oig.1 for ; Thu, 05 Aug 2021 12:02:30 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=MhAdb2Kihhg6rVG3uFDeKcl+JBG5IM2thW4aNaeXliY=; b=D8HIt3TLSYNbFvxGZwTN+K0lyesKqvQgIl/k3xIy+NTJAYHXgMzK5kQ97B+5niSFQ6 1KCElHmWTbAfNDA2ufrB7VyY3xK7d3HV0S6RnYzFijPgdrYw+vkUGWuigg/8t4tJIGXQ 6tgV7hDzoxE7bpk4qRFs7WFfugWR2DTlRKjkv1y38L1QPE3H5zL5ydOY5mP+hWcP+ZwO dWRX1yzzT5a4cmTTQB7uD3/Emn171KYW5viH3Qgyn+tDOkbn2fhtJsgObLormYC2DuE2 oi3GFv+LO+LTD/Vl7jaZ2f6PImB27fY8M1t3liXD9eAGBRWI85XP8fXTtNXIzCojEL57 ErCA== X-Gm-Message-State: AOAM530gcWozgOThsuXNZRWEfOxFhi9xILSbTXP5PFeS4ATgi5DEe37b OqhKM8mbqjxQsVgptunl79nhMpV2EdosF5tkQess3/XP X-Google-Smtp-Source: ABdhPJxal+2hfkY2GCDIFlz0tFgaMCTJ8e8FxAgZveO2IVjKnwnmsi5xxZOnGFQssFG52p0/Loe7bbGcOFNVextj2GQ= X-Received: by 2002:a05:6808:198c:: with SMTP id bj12mr4752045oib.71.1628190150002; Thu, 05 Aug 2021 12:02:30 -0700 (PDT) MIME-Version: 1.0 References: <20210726100026.12538-1-lorenzo.pieralisi@arm.com> <20210802152359.12623-4-lorenzo.pieralisi@arm.com> <20210802164622.GJ18685@arm.com> <20210803091605.GA9637@lpieralisi> In-Reply-To: <20210803091605.GA9637@lpieralisi> From: "Rafael J. Wysocki" Date: Thu, 5 Aug 2021 21:02:19 +0200 Message-ID: Subject: Re: [PATCH v2 3/3] ACPI: Add memory semantics to acpi_os_map_memory() To: Lorenzo Pieralisi Cc: Catalin Marinas , "Rafael J. Wysocki" , Linux Kernel Mailing List , Hanjun Guo , Ard Biesheuvel , Will Deacon , Sudeep Holla , ACPI Devel Maling List , Linux ARM , Veronika kabatova , Robin Murphy X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210805_120231_899911_5F5DA053 X-CRM114-Status: GOOD ( 25.88 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Aug 3, 2021 at 11:16 AM Lorenzo Pieralisi wrote: > > On Mon, Aug 02, 2021 at 05:46:22PM +0100, Catalin Marinas wrote: > > On Mon, Aug 02, 2021 at 04:23:59PM +0100, Lorenzo Pieralisi wrote: > > > The memory attributes attached to memory regions depend on architecture > > > specific mappings. > > > > > > For some memory regions, the attributes specified by firmware (eg > > > uncached) are not sufficient to determine how a memory region should be > > > mapped by an OS (for instance a region that is define as uncached in > > > firmware can be mapped as Normal or Device memory on arm64) and > > > therefore the OS must be given control on how to map the region to match > > > the expected mapping behaviour (eg if a mapping is requested with memory > > > semantics, it must allow unaligned accesses). > > > > > > Rework acpi_os_map_memory() and acpi_os_ioremap() back-end to split > > > them into two separate code paths: > > > > > > acpi_os_memmap() -> memory semantics > > > acpi_os_ioremap() -> MMIO semantics > > > > > > The split allows the architectural implementation back-ends to detect > > > the default memory attributes required by the mapping in question > > > (ie the mapping API defines the semantics memory vs MMIO) and map the > > > memory accordingly. > > > > > > Link: https://lore.kernel.org/linux-arm-kernel/31ffe8fc-f5ee-2858-26c5-0fd8bdd68702@arm.com > > > Tested-by: Hanjun Guo > > > Signed-off-by: Lorenzo Pieralisi > > > Acked-by: Ard Biesheuvel > > > Cc: Ard Biesheuvel > > > Cc: Will Deacon > > > Cc: Hanjun Guo > > > Cc: Sudeep Holla > > > Cc: Catalin Marinas > > > Cc: "Rafael J. Wysocki" > > > > For the arm64 bits: > > > > Acked-by: Catalin Marinas > > > > I presume this will get merged via the ACPI tree? > > Thank you, I don't know what's the best option in Rafael's opinion > (of course if he is OK with the patches which are mostly touching > ACPI code). Well, I can apply them. I'll queue them up tomorrow, but next week I'm on vacation, so they will show up in linux-next after -rc6. Hopefully, that's not too late. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel