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 DCA6EECAAD2 for ; Sun, 28 Aug 2022 15:12:15 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 40CEC6B0073; Sun, 28 Aug 2022 11:12:15 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 3BB466B0074; Sun, 28 Aug 2022 11:12:15 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 282C7940007; Sun, 28 Aug 2022 11:12:15 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 1719C6B0073 for ; Sun, 28 Aug 2022 11:12:15 -0400 (EDT) Received: from smtpin09.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id E2047140B4D for ; Sun, 28 Aug 2022 15:12:14 +0000 (UTC) X-FDA: 79849342188.09.EA99C60 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by imf25.hostedemail.com (Postfix) with ESMTP id 84F2AA0030 for ; Sun, 28 Aug 2022 15:12:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1661699534; h=from:from: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:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=mpY/8AdUR+vNQIgjLclSZmI7zrqsvBcdHsK7fFs0Xqc=; b=UBZ27l+aqVZVoHQbFln3d5TTAs4DMVp4nuLAv6qSTkwyGF8tnXVvfzyuIm8O/m5SvDG4Pk zMo+UEA+aoozj3udaaTZ5gC2IPVRRK7rac+DHTWC+GGzRYIOSZNmd577LUViJKk1ciZkmY wP8qeTKuOHVRktMBzv/OXT26+/yMrOk= Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-240-SVZOcRTLOPy8iZh4izkDrA-1; Sun, 28 Aug 2022 11:12:12 -0400 X-MC-Unique: SVZOcRTLOPy8iZh4izkDrA-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 0012A38005C0; Sun, 28 Aug 2022 15:12:11 +0000 (UTC) Received: from localhost (ovpn-12-25.pek2.redhat.com [10.72.12.25]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 2A2861121314; Sun, 28 Aug 2022 15:12:10 +0000 (UTC) Date: Sun, 28 Aug 2022 23:12:07 +0800 From: Baoquan He To: Christophe Leroy Cc: "linux-kernel@vger.kernel.org" , "linux-mm@kvack.org" , "akpm@linux-foundation.org" , "hch@infradead.org" , "agordeev@linux.ibm.com" , "wangkefeng.wang@huawei.com" , "linux-arm-kernel@lists.infradead.org" , Brian Cain , "linux-hexagon@vger.kernel.org" Subject: Re: [PATCH v2 05/11] hexagon: mm: Convert to GENERIC_IOREMAP Message-ID: References: <20220820003125.353570-1-bhe@redhat.com> <20220820003125.353570-6-bhe@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Scanned-By: MIMEDefang 2.78 on 10.11.54.3 ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1661699534; a=rsa-sha256; cv=none; b=VQU+CjdOX8h3IjoyPAzS2T1biW3ZdMPfeb4guQPJ9to5NioAUPL2HS6NAxAhS0qEULD8ok T6pYr1VIqlXYSZrWXIN7ctLk97JjdKfvQfLZ7qeGZgl8cWdTVWEUgoWXPjblUijJqeDpun C3HCHk7VjS54WkW4GxeCkSZ1HKZ8s94= ARC-Authentication-Results: i=1; imf25.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=UBZ27l+a; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf25.hostedemail.com: domain of bhe@redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=bhe@redhat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1661699534; h=from:from: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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=mpY/8AdUR+vNQIgjLclSZmI7zrqsvBcdHsK7fFs0Xqc=; b=JkhCDhUzDPAowNX5LwSkuFq9fk2MfN1Octu0/7WQC7iD61hLcYOsKIX1xhHHZyEbjCzMvJ sON61SXjsnCm9b/aFB8xhQU7RhS6kY8sybOR8UmkCV+YJfZetRJ2V1m2TjEKpDcScTdz6F FmbaI2g5ndCGXpk2CGiQktde3o0Xnfk= X-Rspamd-Server: rspam11 X-Rspam-User: X-Rspamd-Queue-Id: 84F2AA0030 Authentication-Results: imf25.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=UBZ27l+a; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf25.hostedemail.com: domain of bhe@redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=bhe@redhat.com X-Stat-Signature: 1p8dhxhoua7s5yu3oiadk5dbaeorz9gq X-HE-Tag: 1661699534-419817 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 08/22/22 at 06:38am, Christophe Leroy wrote: > > > Le 20/08/2022 à 02:31, Baoquan He a écrit : > > With it, the old ioremap() and iounmap() can be perfectly removed > > since they are duplicated with the standard functions. > > I see nothing related to the preparation patches. Could this have been > converted without all previous patches ? In that case I think this one > should go in the begining of the series, it would help see which > architectecture really needs the changes to the generic parts. You are right. Hexagon basically has the same implementation as standard ioremap and iounmap code. I will move this at the beginning of patch series in next round of post. Thanks for point this out. > > > > > Signed-off-by: Baoquan He > > Cc: Brian Cain > > Cc: linux-hexagon@vger.kernel.org > > --- > > arch/hexagon/Kconfig | 1 + > > arch/hexagon/include/asm/io.h | 9 +++++-- > > arch/hexagon/mm/ioremap.c | 44 ----------------------------------- > > 3 files changed, 8 insertions(+), 46 deletions(-) > > delete mode 100644 arch/hexagon/mm/ioremap.c > > > > diff --git a/arch/hexagon/Kconfig b/arch/hexagon/Kconfig > > index 54eadf265178..17afffde1a7f 100644 > > --- a/arch/hexagon/Kconfig > > +++ b/arch/hexagon/Kconfig > > @@ -25,6 +25,7 @@ config HEXAGON > > select NEED_SG_DMA_LENGTH > > select NO_IOPORT_MAP > > select GENERIC_IOMAP > > + select GENERIC_IOREMAP > > select GENERIC_SMP_IDLE_THREAD > > select STACKTRACE_SUPPORT > > select GENERIC_CLOCKEVENTS_BROADCAST > > diff --git a/arch/hexagon/include/asm/io.h b/arch/hexagon/include/asm/io.h > > index c33241425a5c..e2d3091ec9d6 100644 > > --- a/arch/hexagon/include/asm/io.h > > +++ b/arch/hexagon/include/asm/io.h > > @@ -170,8 +170,13 @@ static inline void writel(u32 data, volatile void __iomem *addr) > > #define writew_relaxed __raw_writew > > #define writel_relaxed __raw_writel > > > > -void __iomem *ioremap(unsigned long phys_addr, unsigned long size); > > -#define ioremap_uc(X, Y) ioremap((X), (Y)) > > +/* > > + * I/O memory mapping functions. > > + */ > > +#define _PAGE_IOREMAP (_PAGE_PRESENT | _PAGE_READ | _PAGE_WRITE | \ > > + (__HEXAGON_C_DEV << 6)) > > + > > +#define ioremap_uc(addr, size) ioremap((addr), (size)) > > > > > > #define __raw_writel writel > > diff --git a/arch/hexagon/mm/ioremap.c b/arch/hexagon/mm/ioremap.c > > deleted file mode 100644 > > index 255c5b1ee1a7..000000000000 > > --- a/arch/hexagon/mm/ioremap.c > > +++ /dev/null > > @@ -1,44 +0,0 @@ > > -// SPDX-License-Identifier: GPL-2.0-only > > -/* > > - * I/O remap functions for Hexagon > > - * > > - * Copyright (c) 2010-2011, The Linux Foundation. All rights reserved. > > - */ > > - > > -#include > > -#include > > -#include > > - > > -void __iomem *ioremap(unsigned long phys_addr, unsigned long size) > > -{ > > - unsigned long last_addr, addr; > > - unsigned long offset = phys_addr & ~PAGE_MASK; > > - struct vm_struct *area; > > - > > - pgprot_t prot = __pgprot(_PAGE_PRESENT|_PAGE_READ|_PAGE_WRITE > > - |(__HEXAGON_C_DEV << 6)); > > - > > - last_addr = phys_addr + size - 1; > > - > > - /* Wrapping not allowed */ > > - if (!size || (last_addr < phys_addr)) > > - return NULL; > > - > > - /* Rounds up to next page size, including whole-page offset */ > > - size = PAGE_ALIGN(offset + size); > > - > > - area = get_vm_area(size, VM_IOREMAP); > > - addr = (unsigned long)area->addr; > > - > > - if (ioremap_page_range(addr, addr+size, phys_addr, prot)) { > > - vunmap((void *)addr); > > - return NULL; > > - } > > - > > - return (void __iomem *) (offset + addr); > > -} > > - > > -void iounmap(const volatile void __iomem *addr) > > -{ > > - vunmap((void *) ((unsigned long) addr & PAGE_MASK)); > > -}