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=-8.0 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 8FF4BC2BA1A for ; Fri, 24 Apr 2020 08:59:51 +0000 (UTC) Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (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 6076B20724 for ; Fri, 24 Apr 2020 08:59:51 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=xen.org header.i=@xen.org header.b="EXEM0Fdo" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6076B20724 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=xen.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=xen-devel-bounces@lists.xenproject.org Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1jRuBR-00078s-Fo; Fri, 24 Apr 2020 08:59:41 +0000 Received: from us1-rack-iad1.inumbo.com ([172.99.69.81]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1jRuBQ-00078h-25 for xen-devel@lists.xenproject.org; Fri, 24 Apr 2020 08:59:40 +0000 X-Inumbo-ID: ee1868d8-8609-11ea-9e09-bc764e2007e4 Received: from mail.xenproject.org (unknown [104.130.215.37]) by us1-rack-iad1.inumbo.com (Halon) with ESMTPS id ee1868d8-8609-11ea-9e09-bc764e2007e4; Fri, 24 Apr 2020 08:59:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=xen.org; s=20200302mail; h=Content-Transfer-Encoding:Content-Type:In-Reply-To: MIME-Version:Date:Message-ID:From:References:Cc:To:Subject:Sender:Reply-To: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=l9hYPuC1aPcKdkz4teGV0XrjqcxoMQyZm3WpG2h4448=; b=EXEM0Fdo0ejY6vq/SHetM604eu 9S2GDR/juXFhe8zz3WN3vXRq15YgOOcsJAWLG1BFZMov2gTyNwV8eeS0Rw4EJ4QnyLsLoaxvcR8CQ /a7t9EsWt0nsi+d4cinY1t5aRRT5PJnc62SJjSROXk6uOGlR7LeSEmEjcIn71LKI/OIs=; Received: from xenbits.xenproject.org ([104.239.192.120]) by mail.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1jRuBO-0003Nn-82; Fri, 24 Apr 2020 08:59:38 +0000 Received: from [54.239.6.188] (helo=a483e7b01a66.ant.amazon.com) by xenbits.xenproject.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.89) (envelope-from ) id 1jRuBN-0003cO-WB; Fri, 24 Apr 2020 08:59:38 +0000 Subject: Re: [PATCH 1/6] x86_64/mm: map and unmap page tables in cleanup_frame_table To: Hongyan Xia , xen-devel@lists.xenproject.org References: <12c4fe0c0c05b9f76377c085d8a6558beae64003.1587116799.git.hongyxia@amazon.com> From: Julien Grall Message-ID: Date: Fri, 24 Apr 2020 09:59:36 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: <12c4fe0c0c05b9f76377c085d8a6558beae64003.1587116799.git.hongyxia@amazon.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit X-BeenThere: xen-devel@lists.xenproject.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Cc: Andrew Cooper , Wei Liu , Jan Beulich , =?UTF-8?Q?Roger_Pau_Monn=c3=a9?= Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" (resending) On 17/04/2020 10:52, Hongyan Xia wrote: > From: Wei Liu > > Also fix a weird indentation. > > Signed-off-by: Wei Liu > Signed-off-by: Hongyan Xia > --- > xen/arch/x86/x86_64/mm.c | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/xen/arch/x86/x86_64/mm.c b/xen/arch/x86/x86_64/mm.c > index e85ef449f3..18210405f4 100644 > --- a/xen/arch/x86/x86_64/mm.c > +++ b/xen/arch/x86/x86_64/mm.c > @@ -737,8 +737,8 @@ static void cleanup_frame_table(struct mem_hotadd_info *info) > > while (sva < eva) > { > - l3e = l4e_to_l3e(idle_pg_table[l4_table_offset(sva)])[ > - l3_table_offset(sva)]; > + l3e = l3e_from_l4e(idle_pg_table[l4_table_offset(sva)], > + l3_table_offset(sva)); This macro doesn't exist yet in the tree. It would be good to spell out the dependencies in the cover letter so this doesn't get merged before the dependency is merged. Reviewed-by: Julien Grall Cheers, -- Julien Grall