[{"id":1759794,"web_url":"http://patchwork.ozlabs.org/comment/1759794/","msgid":"<20170829.180844.1866158985287686267.davem@davemloft.net>","list_archive_url":null,"date":"2017-08-30T01:08:44","subject":"Re: [PATCH v7 04/11] sparc64: simplify vmemmap_populate","submitter":{"id":15,"url":"http://patchwork.ozlabs.org/api/people/15/","name":"David Miller","email":"davem@davemloft.net"},"content":"From: Pavel Tatashin \nDate: Mon, 28 Aug 2017 22:02:15 -0400\n\n> Remove duplicating code by using common functions\n> vmemmap_pud_populate and vmemmap_pgd_populate.\n> \n> Signed-off-by: Pavel Tatashin \n> Reviewed-by: Steven Sistare \n> Reviewed-by: Daniel Jordan \n> Reviewed-by: Bob Picco \n\nAcked-by: David S. Miller ","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"C7WLeeza\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xhnTT2Nktz9sP5\n\tfor ;\n\tWed, 30 Aug 2017 11:09:17 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmrVJ-000167-Dq; Wed, 30 Aug 2017 01:09:13 +0000","from shards.monkeyblade.net ([184.105.139.130])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmrVF-00013R-F6 for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 01:09:11 +0000","from localhost (74-93-104-98-Washington.hfc.comcastbusiness.net\n\t[74.93.104.98]) (using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(Client did not present a certificate)\n\t(Authenticated sender: davem-davemloft)\n\tby shards.monkeyblade.net (Postfix) with ESMTPSA id 2483F13DAB50D;\n\tTue, 29 Aug 2017 18:08:45 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:Mime-Version:References:In-Reply-To:\n\tFrom:Subject:To:Message-Id:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=CE98lwXjNstH4jhPcjXXnwQBpcA3XtOIUmqsDAWb1p0=;\n\tb=C7WLeezaqe78/H\n\ti/m+FlPiWSWvP9bsbbZL/oG9pEeeeMXnfSrSCDjYtfX0HobfzT2WkHgguJi6MAeK7qByXaHtjChMG\n\tFCs22tSBXQQ18BFdP6n0YzrXae2ifhrwjVSnGKM1rj+Omu4NaoVvIMfTXcyhUaHe2mx9CL8oOI0CN\n\tZpHJhAZP7EvD6niamfARF3VprJ3HuZLrOPCw67NwV7GDfk104I+ygWVcEO4DStv8lHPjdPz6WX5nA\n\tUMxyu4P7Ad6yjVYRxallXbY8LHE4Y0rj0V8Y5njjjxvCmy+MYGEjanXCM4RmjfRlqm8xo6yeR2dBU\n\tkAnV8tnnd0wMpM3Em9JQ==;","Date":"Tue, 29 Aug 2017 18:08:44 -0700 (PDT)","Message-Id":"<20170829.180844.1866158985287686267.davem@davemloft.net>","To":"pasha.tatashin@oracle.com","Subject":"Re: [PATCH v7 04/11] sparc64: simplify vmemmap_populate","From":"David Miller ","In-Reply-To":"<1503972142-289376-5-git-send-email-pasha.tatashin@oracle.com>","References":"<1503972142-289376-1-git-send-email-pasha.tatashin@oracle.com>\n\t<1503972142-289376-5-git-send-email-pasha.tatashin@oracle.com>","X-Mailer":"Mew version 6.7 on Emacs 25.2 / Mule 6.0 (HANACHIRUSATO)","Mime-Version":"1.0","X-Greylist":"Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12\n\t(shards.monkeyblade.net [149.20.54.216]);\n\tTue, 29 Aug 2017 18:08:45 -0700 (PDT)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170829_180909_521639_CBF4F844 ","X-CRM114-Status":"UNSURE ( 7.44 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linux-s390@vger.kernel.org, ard.biesheuvel@linaro.org,\n\tmgorman@techsingularity.net, sam@ravnborg.org, borntraeger@de.ibm.com,\n\twill.deacon@arm.com, x86@kernel.org, heiko.carstens@de.ibm.com,\n\tlinux-kernel@vger.kernel.org, kasan-dev@googlegroups.com,\n\tmhocko@kernel.org, \n\tlinux-mm@kvack.org, Steven.Sistare@oracle.com, willy@infradead.org,\n\tcatalin.marinas@arm.com, sparclinux@vger.kernel.org,\n\tbob.picco@oracle.com, \n\tdaniel.m.jordan@oracle.com, linuxppc-dev@lists.ozlabs.org,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1759796,"web_url":"http://patchwork.ozlabs.org/comment/1759796/","msgid":"<20170829.180928.896986367806428362.davem@davemloft.net>","list_archive_url":null,"date":"2017-08-30T01:09:28","subject":"Re: [PATCH v7 02/11] sparc64/mm: setting fields in deferred pages","submitter":{"id":15,"url":"http://patchwork.ozlabs.org/api/people/15/","name":"David Miller","email":"davem@davemloft.net"},"content":"From: Pavel Tatashin \nDate: Mon, 28 Aug 2017 22:02:13 -0400\n\n> Without deferred struct page feature (CONFIG_DEFERRED_STRUCT_PAGE_INIT),\n> flags and other fields in \"struct page\"es are never changed prior to first\n> initializing struct pages by going through __init_single_page().\n> \n> With deferred struct page feature enabled there is a case where we set some\n> fields prior to initializing:\n> \n> mem_init() {\n> register_page_bootmem_info();\n> free_all_bootmem();\n> ...\n> }\n> \n> When register_page_bootmem_info() is called only non-deferred struct pages\n> are initialized. But, this function goes through some reserved pages which\n> might be part of the deferred, and thus are not yet initialized.\n> \n> mem_init\n> register_page_bootmem_info\n> register_page_bootmem_info_node\n> get_page_bootmem\n> .. setting fields here ..\n> such as: page->freelist = (void *)type;\n> \n> free_all_bootmem()\n> free_low_memory_core_early()\n> for_each_reserved_mem_region()\n> reserve_bootmem_region()\n> init_reserved_page() <- Only if this is deferred reserved page\n> __init_single_pfn()\n> __init_single_page()\n> memset(0) <-- Loose the set fields here\n> \n> We end-up with similar issue as in the previous patch, where currently we\n> do not observe problem as memory is zeroed. But, if flag asserts are\n> changed we can start hitting issues.\n> \n> Also, because in this patch series we will stop zeroing struct page memory\n> during allocation, we must make sure that struct pages are properly\n> initialized prior to using them.\n> \n> The deferred-reserved pages are initialized in free_all_bootmem().\n> Therefore, the fix is to switch the above calls.\n> \n> Signed-off-by: Pavel Tatashin \n> Reviewed-by: Steven Sistare \n> Reviewed-by: Daniel Jordan \n> Reviewed-by: Bob Picco \n\nAcked-by: David S. Miller ","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"G34rBbig\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xhnVF6mGCz9sN7\n\tfor ;\n\tWed, 30 Aug 2017 11:09:57 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmrVy-0001Rq-Bb; Wed, 30 Aug 2017 01:09:54 +0000","from shards.monkeyblade.net ([184.105.139.130])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmrVt-0001LO-Og for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 01:09:51 +0000","from localhost (74-93-104-98-Washington.hfc.comcastbusiness.net\n\t[74.93.104.98]) (using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(Client did not present a certificate)\n\t(Authenticated sender: davem-davemloft)\n\tby shards.monkeyblade.net (Postfix) with ESMTPSA id BA18613DAB50D;\n\tTue, 29 Aug 2017 18:09:28 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:Mime-Version:References:In-Reply-To:\n\tFrom:Subject:To:Message-Id:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=jkDXMkLu2T/FHyJ6R45WAw7O6asbAsdAmyKMzxF3MaY=;\n\tb=G34rBbigjzWV6T\n\t6E24HVTsdQCuhUHwQN3A1RaS7OK7dd2qmJE1qut8betzHwfNp1uLv6F3vce181+/vRCdUvLtsmG16\n\tdi1kPu0YcqsM5kkp5aDB8Gu/sxoA1yHW8FwMBZOJmrOlVwLytJMi2VPK/FF/LPnj8cwoAlEybqdi5\n\ti19RejGVVhM/L66xOnFCu9i4GWwhOprEmrb1xIaNuKQphAhe6z0zr9xf/txsvPS/6ZSI0LO3PyYjw\n\tuEFdtNwL/ovpXcaYX5OOhuvMD5GMXkcuUwhTxH9UEzAlf4xPP0MuKipTmXbpHS5XlXVnHkUbb+J/b\n\tnOJyJxhuHVPDP/hpyKEg==;","Date":"Tue, 29 Aug 2017 18:09:28 -0700 (PDT)","Message-Id":"<20170829.180928.896986367806428362.davem@davemloft.net>","To":"pasha.tatashin@oracle.com","Subject":"Re: [PATCH v7 02/11] sparc64/mm: setting fields in deferred pages","From":"David Miller ","In-Reply-To":"<1503972142-289376-3-git-send-email-pasha.tatashin@oracle.com>","References":"<1503972142-289376-1-git-send-email-pasha.tatashin@oracle.com>\n\t<1503972142-289376-3-git-send-email-pasha.tatashin@oracle.com>","X-Mailer":"Mew version 6.7 on Emacs 25.2 / Mule 6.0 (HANACHIRUSATO)","Mime-Version":"1.0","X-Greylist":"Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12\n\t(shards.monkeyblade.net [149.20.54.216]);\n\tTue, 29 Aug 2017 18:09:29 -0700 (PDT)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170829_180949_807051_5483F3B1 ","X-CRM114-Status":"GOOD ( 14.00 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linux-s390@vger.kernel.org, ard.biesheuvel@linaro.org,\n\tmgorman@techsingularity.net, sam@ravnborg.org, borntraeger@de.ibm.com,\n\twill.deacon@arm.com, x86@kernel.org, heiko.carstens@de.ibm.com,\n\tlinux-kernel@vger.kernel.org, kasan-dev@googlegroups.com,\n\tmhocko@kernel.org, \n\tlinux-mm@kvack.org, Steven.Sistare@oracle.com, willy@infradead.org,\n\tcatalin.marinas@arm.com, sparclinux@vger.kernel.org,\n\tbob.picco@oracle.com, \n\tdaniel.m.jordan@oracle.com, linuxppc-dev@lists.ozlabs.org,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1759800,"web_url":"http://patchwork.ozlabs.org/comment/1759800/","msgid":"<20170829.181208.171985548699678313.davem@davemloft.net>","list_archive_url":null,"date":"2017-08-30T01:12:08","subject":"Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing","submitter":{"id":15,"url":"http://patchwork.ozlabs.org/api/people/15/","name":"David Miller","email":"davem@davemloft.net"},"content":"From: Pavel Tatashin \nDate: Mon, 28 Aug 2017 22:02:18 -0400\n\n> Add an optimized mm_zero_struct_page(), so struct page's are zeroed without\n> calling memset(). We do eight to ten regular stores based on the size of\n> struct page. Compiler optimizes out the conditions of switch() statement.\n> \n> SPARC-M6 with 15T of memory, single thread performance:\n> \n> BASE FIX OPTIMIZED_FIX\n> bootmem_init 28.440467985s 2.305674818s 2.305161615s\n> free_area_init_nodes 202.845901673s 225.343084508s 172.556506560s\n> --------------------------------------------\n> Total 231.286369658s 227.648759326s 174.861668175s\n> \n> BASE: current linux\n> FIX: This patch series without \"optimized struct page zeroing\"\n> OPTIMIZED_FIX: This patch series including the current patch.\n> \n> bootmem_init() is where memory for struct pages is zeroed during\n> allocation. Note, about two seconds in this function is a fixed time: it\n> does not increase as memory is increased.\n> \n> Signed-off-by: Pavel Tatashin \n> Reviewed-by: Steven Sistare \n> Reviewed-by: Daniel Jordan \n> Reviewed-by: Bob Picco \n\nYou should probably use initializing stores when you are doing 8\nstores and we thus know the page struct is cache line aligned.\n\nBut other than that:\n\nAcked-by: David S. Miller ","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"Ymacm1wZ\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xhnYK2vmcz9sN7\n\tfor ;\n\tWed, 30 Aug 2017 11:12:37 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmrYY-00044n-EC; Wed, 30 Aug 2017 01:12:34 +0000","from shards.monkeyblade.net ([184.105.139.130])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmrYV-00043H-1x for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 01:12:32 +0000","from localhost (74-93-104-98-Washington.hfc.comcastbusiness.net\n\t[74.93.104.98]) (using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(Client did not present a certificate)\n\t(Authenticated sender: davem-davemloft)\n\tby shards.monkeyblade.net (Postfix) with ESMTPSA id 8683713DA6DC0;\n\tTue, 29 Aug 2017 18:12:09 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:Mime-Version:References:In-Reply-To:\n\tFrom:Subject:To:Message-Id:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=l428BdeqAWaW3YsAiI0hz66tA75tdqmsGmRT2iRaYIs=;\n\tb=Ymacm1wZu6vfFm\n\t/FbzoozVGWz9p408cqMEvcYENv7745PjeT9RuBjdHOyBV6U/5vO2ySu3LfY/vA2tP6j4IAlZSKFER\n\tQSuWp+6sf4A3BU7f70gf0e3DotjasIh0jwaHD0WElW38+EBioPsGUORmfPv3lBbW3bEoI3oMJZpeI\n\tdmy61Wen/ncZtQ9fuI8bz+a1lpJ5x3/4/FdsFO04qlQkhyYHEAOETUAClXXwhWsjKIqe78p8QfvUX\n\tijmP4DzRLBm7NImgdXtz/2/F5Jt/7a1BfZDkCGYEDNgPkOGhRztt1p3yrUpFD+vxeTIxRaYSskheX\n\th8qSPReF/DRP/RwgntAA==;","Date":"Tue, 29 Aug 2017 18:12:08 -0700 (PDT)","Message-Id":"<20170829.181208.171985548699678313.davem@davemloft.net>","To":"pasha.tatashin@oracle.com","Subject":"Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing","From":"David Miller ","In-Reply-To":"<1503972142-289376-8-git-send-email-pasha.tatashin@oracle.com>","References":"<1503972142-289376-1-git-send-email-pasha.tatashin@oracle.com>\n\t<1503972142-289376-8-git-send-email-pasha.tatashin@oracle.com>","X-Mailer":"Mew version 6.7 on Emacs 25.2 / Mule 6.0 (HANACHIRUSATO)","Mime-Version":"1.0","X-Greylist":"Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12\n\t(shards.monkeyblade.net [149.20.54.216]);\n\tTue, 29 Aug 2017 18:12:10 -0700 (PDT)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170829_181231_158108_953FBC1D ","X-CRM114-Status":"GOOD ( 11.91 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linux-s390@vger.kernel.org, ard.biesheuvel@linaro.org,\n\tmgorman@techsingularity.net, sam@ravnborg.org, borntraeger@de.ibm.com,\n\twill.deacon@arm.com, x86@kernel.org, heiko.carstens@de.ibm.com,\n\tlinux-kernel@vger.kernel.org, kasan-dev@googlegroups.com,\n\tmhocko@kernel.org, \n\tlinux-mm@kvack.org, Steven.Sistare@oracle.com, willy@infradead.org,\n\tcatalin.marinas@arm.com, sparclinux@vger.kernel.org,\n\tbob.picco@oracle.com, \n\tdaniel.m.jordan@oracle.com, linuxppc-dev@lists.ozlabs.org,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1760147,"web_url":"http://patchwork.ozlabs.org/comment/1760147/","msgid":"","list_archive_url":null,"date":"2017-08-30T13:19:58","subject":"Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing","submitter":{"id":71010,"url":"http://patchwork.ozlabs.org/api/people/71010/","name":"Pavel Tatashin","email":"pasha.tatashin@oracle.com"},"content":"Hi Dave,\n\nThank you for acking.\n\nThe reason I am not doing initializing stores is because they require a \nmembar, even if only regular stores are following (I hoped to do a \nmembar before first load). This is something I was thinking was not \ntrue, but after consulting with colleagues and checking processor \nmanual, I verified that it is the case.\n\nPasha\n\n> \n> You should probably use initializing stores when you are doing 8\n> stores and we thus know the page struct is cache line aligned.\n> \n> But other than that:\n> \n> Acked-by: David S. Miller ","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"R1ivKY2M\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xj5kB6YYFz9sN7\n\tfor ;\n\tWed, 30 Aug 2017 23:21:22 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn2vn-0001w0-Hu; Wed, 30 Aug 2017 13:21:19 +0000","from aserp1040.oracle.com ([141.146.126.69])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn2vk-0001nT-3Z for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 13:21:18 +0000","from userv0021.oracle.com (userv0021.oracle.com [156.151.31.71])\n\tby aserp1040.oracle.com (Sentrion-MTA-4.3.2/Sentrion-MTA-4.3.2) with\n\tESMTP id v7UDK58S010472\n\t(version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256\n\tverify=OK); Wed, 30 Aug 2017 13:20:06 GMT","from userv0121.oracle.com (userv0121.oracle.com [156.151.31.72])\n\tby userv0021.oracle.com (8.14.4/8.14.4) with ESMTP id v7UDK5jf006519\n\t(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256\n\tverify=OK); Wed, 30 Aug 2017 13:20:05 GMT","from abhmp0001.oracle.com (abhmp0001.oracle.com [141.146.116.7])\n\tby userv0121.oracle.com (8.14.4/8.13.8) with ESMTP id v7UDK24n014324; \n\tWed, 30 Aug 2017 13:20:02 GMT","from [192.168.1.10] (/98.216.35.41)\n\tby default (Oracle Beehive Gateway v4.0)\n\twith ESMTP ; Wed, 30 Aug 2017 06:20:02 -0700"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:\n\tContent-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive:\n\tList-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From:\n\tReferences:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner;\n\tbh=e7t7Fo8hPNRSd8YtNSsjnWvUc2FtGfb3aqdZCsqTHpo=;\n\tb=R1ivKY2Mn2Q72t1oRp9SZGs1n\n\tfezOKSEdEwqHRT6vdHNtkW9cQPNl/GdsZ+dwy3Ok/iNx7HnZ07jf4puBkR2P/Cn4FGg54G9+H8yfz\n\tEWubgMRiskfOFHX0g5e/FKvD1ufkJPnLQZB1o/145XlWuoyn0DVlRAcHqwUoHpp+2FLMXpScBZjCf\n\tlB/dWk8gnPgO6bI7nNhOGXdmSUJlMkrtxfiDnwVIslQ1AKFUeo9L9iWrDhZjStpOBNrpAQ3RaSZel\n\t7+nIyuWaSlMYLxbUK3XPCG3UeUnFRs/UeIs/WxbtK2FZITabPi6JI1k+chHck670D4voX/hQBv638\n\tOdXX40fpA==;","Subject":"Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing","To":"David Miller ","References":"<1503972142-289376-1-git-send-email-pasha.tatashin@oracle.com>\n\t<1503972142-289376-8-git-send-email-pasha.tatashin@oracle.com>\n\t<20170829.181208.171985548699678313.davem@davemloft.net>","From":"Pasha Tatashin ","Message-ID":"","Date":"Wed, 30 Aug 2017 09:19:58 -0400","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170829.181208.171985548699678313.davem@davemloft.net>","Content-Language":"en-US","X-Source-IP":"userv0021.oracle.com [156.151.31.71]","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170830_062116_349352_380C3897 ","X-CRM114-Status":"UNSURE ( 7.05 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-4.2 (----)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-4.2 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\n\tmedium trust [141.146.126.69 listed in list.dnswl.org]\n\t-0.0 RCVD_IN_MSPIKE_H4 RBL: Very Good reputation (+4)\n\t[141.146.126.69 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.0 RCVD_IN_MSPIKE_WL Mailspike good senders","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linux-s390@vger.kernel.org, ard.biesheuvel@linaro.org,\n\tmgorman@techsingularity.net, sam@ravnborg.org, borntraeger@de.ibm.com,\n\twill.deacon@arm.com, x86@kernel.org, heiko.carstens@de.ibm.com,\n\tlinux-kernel@vger.kernel.org, kasan-dev@googlegroups.com,\n\tmhocko@kernel.org, \n\tlinux-mm@kvack.org, Steven.Sistare@oracle.com, willy@infradead.org,\n\tcatalin.marinas@arm.com, sparclinux@vger.kernel.org,\n\tbob.picco@oracle.com, \n\tdaniel.m.jordan@oracle.com, linuxppc-dev@lists.ozlabs.org,\n\tlinux-arm-kernel@lists.infradead.org","Content-Transfer-Encoding":"7bit","Content-Type":"text/plain; charset=\"us-ascii\"; Format=\"flowed\"","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1760377,"web_url":"http://patchwork.ozlabs.org/comment/1760377/","msgid":"<20170830.104608.2080967179463692935.davem@davemloft.net>","list_archive_url":null,"date":"2017-08-30T17:46:08","subject":"Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing","submitter":{"id":15,"url":"http://patchwork.ozlabs.org/api/people/15/","name":"David Miller","email":"davem@davemloft.net"},"content":"From: Pasha Tatashin \nDate: Wed, 30 Aug 2017 09:19:58 -0400\n\n> The reason I am not doing initializing stores is because they require\n> a membar, even if only regular stores are following (I hoped to do a\n> membar before first load). This is something I was thinking was not\n> true, but after consulting with colleagues and checking processor\n> manual, I verified that it is the case.\n\nOh yes, that's right, now I remember.","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"T0f7r23g\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xjCcJ3CWmz9sN5\n\tfor ;\n\tThu, 31 Aug 2017 03:46:40 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn74X-0008DC-8A; Wed, 30 Aug 2017 17:46:37 +0000","from shards.monkeyblade.net ([184.105.139.130])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn74U-0008As-A1 for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 17:46:35 +0000","from localhost (74-93-104-98-Washington.hfc.comcastbusiness.net\n\t[74.93.104.98]) (using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(Client did not present a certificate)\n\t(Authenticated sender: davem-davemloft)\n\tby shards.monkeyblade.net (Postfix) with ESMTPSA id 4D3B0124C72B8;\n\tWed, 30 Aug 2017 10:46:10 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:Mime-Version:References:In-Reply-To:\n\tFrom:Subject:To:Message-Id:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=es2q7AsB904O0ZTTNMJj1NDDIt+BiZGYjJuJV1CiB+M=;\n\tb=T0f7r23gJs7B6q\n\tm8bjx4SyDqQwaXdi1ukf5YodgrLtZOjjXn8Ic7t8jLJNL+H04nAZA36nPCd3yv/Iv3kb4liAQnDeM\n\tsJ+jIKxORBLArbJaxgYGMvrZPYz34dwLF4L7Un7Sxgwkw9HmNNCBCu52PRtPXnb/RuMK193z7rHjh\n\tbkVyuMkQBMxHNZpBZ15zCI1XJswSYDoN+LxMxunWmIDYM0Cex0epHafhigF5CmEAEWxVlYvDs8rw3\n\tCzNaWG4UQ6mcrpgn4Ix56nhoz4V23zgDp7WJi11wQ5eogzO0qVFIZ7OP45Kf0Xs2rgn7AgJTf3Ymw\n\ts5hjIhfn+RYv6m6GvQfg==;","Date":"Wed, 30 Aug 2017 10:46:08 -0700 (PDT)","Message-Id":"<20170830.104608.2080967179463692935.davem@davemloft.net>","To":"pasha.tatashin@oracle.com","Subject":"Re: [PATCH v7 07/11] sparc64: optimized struct page zeroing","From":"David Miller ","In-Reply-To":"","References":"<1503972142-289376-8-git-send-email-pasha.tatashin@oracle.com>\n\t<20170829.181208.171985548699678313.davem@davemloft.net>\n\t","X-Mailer":"Mew version 6.7 on Emacs 25.2 / Mule 6.0 (HANACHIRUSATO)","Mime-Version":"1.0","X-Greylist":"Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12\n\t(shards.monkeyblade.net [149.20.54.216]);\n\tWed, 30 Aug 2017 10:46:11 -0700 (PDT)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170830_104634_353999_424F52FF ","X-CRM114-Status":"UNSURE ( 8.72 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linux-s390@vger.kernel.org, ard.biesheuvel@linaro.org,\n\tmgorman@techsingularity.net, sam@ravnborg.org, borntraeger@de.ibm.com,\n\twill.deacon@arm.com, x86@kernel.org, heiko.carstens@de.ibm.com,\n\tlinux-kernel@vger.kernel.org, kasan-dev@googlegroups.com,\n\tmhocko@kernel.org, \n\tlinux-mm@kvack.org, Steven.Sistare@oracle.com, willy@infradead.org,\n\tcatalin.marinas@arm.com, sparclinux@vger.kernel.org,\n\tbob.picco@oracle.com, \n\tdaniel.m.jordan@oracle.com, linuxppc-dev@lists.ozlabs.org,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]