{"id":757923,"url":"http://patchwork.ozlabs.org/api/patches/757923/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/87o9vauvyk.fsf@concordia.ellerman.id.au/","project":{"id":2,"url":"http://patchwork.ozlabs.org/api/projects/2/?format=json","name":"Linux PPC development","link_name":"linuxppc-dev","list_id":"linuxppc-dev.lists.ozlabs.org","list_email":"linuxppc-dev@lists.ozlabs.org","web_url":"https://github.com/linuxppc/wiki/wiki","scm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git","webscm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/","list_archive_url":"https://lore.kernel.org/linuxppc-dev/","list_archive_url_format":"https://lore.kernel.org/linuxppc-dev/{}/","commit_url_format":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?id={}"},"msgid":"<87o9vauvyk.fsf@concordia.ellerman.id.au>","list_archive_url":"https://lore.kernel.org/linuxppc-dev/87o9vauvyk.fsf@concordia.ellerman.id.au/","date":"2017-05-03T09:32:35","name":"powerpc/mm: Fix mapped range information print during boot","commit_ref":"6deb6b474bda4d2d3fbee066f20561d3dc7f8b30","pull_url":null,"state":"accepted","archived":false,"hash":"35345282bfb3ab12c12840a91a095a241a122f83","submitter":{"id":46580,"url":"http://patchwork.ozlabs.org/api/people/46580/?format=json","name":"Michael Ellerman","email":"mpe@ellerman.id.au"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/87o9vauvyk.fsf@concordia.ellerman.id.au/mbox/","series":[],"comments":"http://patchwork.ozlabs.org/api/patches/757923/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/757923/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org","linuxppc-dev@ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3wHtJM03LJz9s3w\n\tfor ;\n\tWed, 3 May 2017 19:33:39 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3wHtJL6J4nzDqJ7\n\tfor ;\n\tWed, 3 May 2017 19:33:38 +1000 (AEST)","from ozlabs.org (ozlabs.org [103.22.144.67])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3wHtHD3qzqzDqBH\n\tfor ;\n\tWed, 3 May 2017 19:32:40 +1000 (AEST)","by ozlabs.org (Postfix)\n\tid 3wHtHD37XFz9s7v; Wed, 3 May 2017 19:32:40 +1000 (AEST)","from authenticated.ozlabs.org (localhost [127.0.0.1])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPSA id 3wHtHD1dPlz9s3w;\n\tWed, 3 May 2017 19:32:40 +1000 (AEST)"],"From":"Michael Ellerman ","To":"Anshuman Khandual , linuxppc-dev@ozlabs.org","Subject":"Re: [PATCH] powerpc/mm: Fix mapped range information print during\n\tboot","In-Reply-To":"<20170428142445.17615-1-khandual@linux.vnet.ibm.com>","References":"<20170428142445.17615-1-khandual@linux.vnet.ibm.com>","User-Agent":"Notmuch/0.21 (https://notmuchmail.org)","Date":"Wed, 03 May 2017 19:32:35 +1000","Message-ID":"<87o9vauvyk.fsf@concordia.ellerman.id.au>","MIME-Version":"1.0","Content-Type":"text/plain","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"aneesh.kumar@linux.vnet.ibm.com","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"},"content":"Anshuman Khandual writes:\n\n> This is a trivial fix patch regarding mapped ranges in radix MMU\n> environment during boot. No functional change.\n>\n> Before the patch:\n>\n> $dmesg | grep Mapped\n> [ 0.000000] Mapped range 0x0 - 0x2000000000 with 0x40000000\n> [ 0.000000] Mapped range 0x200000000000 - 0x202000000000 with 0x40000000\n>\n> After the patch:\n>\n> $dmesg | grep Mapped\n> [ 0.000000] Mapped range 0x0000000000000000 - 0x0000002000000000 with 1024 MB\n> [ 0.000000] Mapped range 0x0000200000000000 - 0x0000202000000000 with 1024 MB\n\nIt's a bit better, how about this instead?\n\n\n\ncheers","diff":"diff --git a/arch/powerpc/mm/pgtable-radix.c b/arch/powerpc/mm/pgtable-radix.c\nindex c28165d8970b..519cfef569d1 100644\n--- a/arch/powerpc/mm/pgtable-radix.c\n+++ b/arch/powerpc/mm/pgtable-radix.c\n@@ -8,9 +8,14 @@\n * as published by the Free Software Foundation; either version\n * 2 of the License, or (at your option) any later version.\n */\n+\n+#define pr_fmt(fmt) \"radix-mmu: \" fmt\n+\n+#include \n #include \n #include \n #include \n+#include \n \n #include \n #include \n@@ -112,10 +117,14 @@ static inline void __meminit print_mapping(unsigned long start,\n \t\t\t\t\t unsigned long end,\n \t\t\t\t\t unsigned long size)\n {\n+\tchar buf[10];\n+\n \tif (end <= start)\n \t\treturn;\n \n-\tpr_info(\"Mapped range 0x%lx - 0x%lx with 0x%lx\\n\", start, end, size);\n+\tstring_get_size(size, 1, STRING_UNITS_2, buf, sizeof(buf));\n+\n+\tpr_info(\"Mapped 0x%016lx-0x%016lx with %s pages\\n\", start, end, buf);\n }\n \n static int __meminit create_physical_mapping(unsigned long start,\n","prefixes":[]}