Cover Letter Detail
Show a cover letter.
GET /api/covers/816441/?format=api
{ "id": 816441, "url": "http://patchwork.ozlabs.org/api/covers/816441/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-imx/cover/20170920201714.19817-1-pasha.tatashin@oracle.com/", "project": { "id": 19, "url": "http://patchwork.ozlabs.org/api/projects/19/?format=api", "name": "Linux IMX development", "link_name": "linux-imx", "list_id": "linux-imx-kernel.lists.patchwork.ozlabs.org", "list_email": "linux-imx-kernel@lists.patchwork.ozlabs.org", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170920201714.19817-1-pasha.tatashin@oracle.com>", "list_archive_url": null, "date": "2017-09-20T20:17:02", "name": "[v9,00/12] complete deferred page initialization", "submitter": { "id": 71010, "url": "http://patchwork.ozlabs.org/api/people/71010/?format=api", "name": "Pavel Tatashin", "email": "pasha.tatashin@oracle.com" }, "mbox": "http://patchwork.ozlabs.org/project/linux-imx/cover/20170920201714.19817-1-pasha.tatashin@oracle.com/mbox/", "series": [ { "id": 4221, "url": "http://patchwork.ozlabs.org/api/series/4221/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-imx/list/?series=4221", "date": "2017-09-20T20:17:02", "name": "complete deferred page initialization", "version": 9, "mbox": "http://patchwork.ozlabs.org/series/4221/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/816441/comments/", "headers": { "Return-Path": "<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>", "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=<UNKNOWN>)", "ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"gdLK3VkD\"; 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 3xy9zy0cfqz9s7h\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 06:18:38 +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 1dulS4-0001LY-Tf; Wed, 20 Sep 2017 20:18:32 +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 1dulRy-0001DD-9Y for linux-arm-kernel@lists.infradead.org;\n\tWed, 20 Sep 2017 20:18:28 +0000", "from aserv0021.oracle.com (aserv0021.oracle.com [141.146.126.233])\n\tby aserp1040.oracle.com (Sentrion-MTA-4.3.2/Sentrion-MTA-4.3.2)\n\twith ESMTP id v8KKHLqD011273\n\t(version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256\n\tverify=OK); Wed, 20 Sep 2017 20:17:21 GMT", "from aserv0121.oracle.com (aserv0121.oracle.com [141.146.126.235])\n\tby aserv0021.oracle.com (8.14.4/8.14.4) with ESMTP id\n\tv8KKHLww020827\n\t(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256\n\tverify=OK); Wed, 20 Sep 2017 20:17:21 GMT", "from abhmp0008.oracle.com (abhmp0008.oracle.com [141.146.116.14])\n\tby aserv0121.oracle.com (8.14.4/8.13.8) with ESMTP id\n\tv8KKHJmo007108; Wed, 20 Sep 2017 20:17:19 GMT", "from xakep.us.oracle.com (/10.154.127.176)\n\tby default (Oracle Beehive Gateway v4.0)\n\twith ESMTP ; Wed, 20 Sep 2017 13:17:19 -0700" ], "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:MIME-Version:List-Subscribe:List-Help:\n\tList-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date:Subject:To:\n\tFrom:Reply-To:Cc:Content-ID:Content-Description:Resent-Date:Resent-From:\n\tResent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:\n\tList-Owner; bh=d4iu17covsDucVkZd/8QB80tey28lZ/Pnl6+iujX9Rw=;\n\tb=gdLK3VkDla+3e7\n\tnKiuI1vY0MW0Y4uDrRKpBDHN9VsBUAU44E854oM+LoaAq3k8i5M4QLroqaNTicZ+o7m6Xxv5j1iFI\n\tmjAlIhX0jE+48Bt3Lu4XVuZXEV4ABxYb2hwNDjhxOVZcqa0aafjLJ54+WEWCRDZwgNkg2WNQO4+pw\n\t4nG5lcR38V1dwVKVTTMqF+4dXMdtbROZ8cAN7WVgQCMoeX4WU1mzgLBnpQ7ERPCSsxPWN4LOeEfdO\n\twopb1puYg4zC0FBjXq90ApsPafQYnL5XM9bIXtFa50zSufsyGtw9U+NQAiK2rAFexVV2A2Oho2Th8\n\tJngYPHr4Rw9xT3ibZkiQ==;", "From": "Pavel Tatashin <pasha.tatashin@oracle.com>", "To": "linux-kernel@vger.kernel.org, sparclinux@vger.kernel.org,\n\tlinux-mm@kvack.org, linuxppc-dev@lists.ozlabs.org,\n\tlinux-s390@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\tx86@kernel.org, kasan-dev@googlegroups.com, borntraeger@de.ibm.com,\n\theiko.carstens@de.ibm.com, davem@davemloft.net, willy@infradead.org, \n\tmhocko@kernel.org, ard.biesheuvel@linaro.org, mark.rutland@arm.com,\n\twill.deacon@arm.com, catalin.marinas@arm.com, sam@ravnborg.org,\n\tmgorman@techsingularity.net, steven.sistare@oracle.com,\n\tdaniel.m.jordan@oracle.com, bob.picco@oracle.com", "Subject": "[PATCH v9 00/12] complete deferred page initialization", "Date": "Wed, 20 Sep 2017 16:17:02 -0400", "Message-Id": "<20170920201714.19817-1-pasha.tatashin@oracle.com>", "X-Mailer": "git-send-email 2.14.1", "X-Source-IP": "aserv0021.oracle.com [141.146.126.233]", "X-CRM114-Version": "20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ", "X-CRM114-CacheID": "sfid-20170920_131826_509414_87DBC99E ", "X-CRM114-Status": "GOOD ( 10.83 )", "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\n\t0.0 UNPARSEABLE_RELAY Informational: message has unparseable relay\n\tlines", "X-BeenThere": "linux-arm-kernel@lists.infradead.org", "X-Mailman-Version": "2.1.21", "Precedence": "list", "List-Unsubscribe": "<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>", "List-Archive": "<http://lists.infradead.org/pipermail/linux-arm-kernel/>", "List-Post": "<mailto:linux-arm-kernel@lists.infradead.org>", "List-Help": "<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>", "List-Subscribe": "<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Sender": "\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>", "Errors-To": "linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org", "List-Id": "linux-imx-kernel.lists.patchwork.ozlabs.org" }, "content": "Changelog:\nv9 - v8\n- Addressed comments raised by Mark Rutland and Ard Biesheuvel: changed\n kasan implementation. Added a new function: kasan_map_populate() that\n zeroes the allocated and mapped memory\n\nv8 - v7\n- Added Acked-by's from Dave Miller for SPARC changes\n- Fixed a minor compiling issue on tile architecture reported by kbuild\n\nv7 - v6\n- Addressed comments from Michal Hocko\n- memblock_discard() patch was removed from this series and integrated\n separately\n- Fixed bug reported by kbuild test robot new patch:\n mm: zero reserved and unavailable struct pages\n- Removed patch\n x86/mm: reserve only exiting low pages\n As, it is not needed anymore, because of the previous fix\n- Re-wrote deferred_init_memmap(), found and fixed an existing bug, where\n page variable is not reset when zone holes present.\n- Merged several patches together per Michal request\n- Added performance data including raw logs\n\nv6 - v5\n- Fixed ARM64 + kasan code, as reported by Ard Biesheuvel\n- Tested ARM64 code in qemu and found few more issues, that I fixed in this\n iteration\n- Added page roundup/rounddown to x86 and arm zeroing routines to zero the\n whole allocated range, instead of only provided address range.\n- Addressed SPARC related comment from Sam Ravnborg\n- Fixed section mismatch warnings related to memblock_discard().\n\nv5 - v4\n- Fixed build issues reported by kbuild on various configurations\n\nv4 - v3\n- Rewrote code to zero sturct pages in __init_single_page() as\n suggested by Michal Hocko\n- Added code to handle issues related to accessing struct page\n memory before they are initialized.\n\nv3 - v2\n- Addressed David Miller comments about one change per patch:\n * Splited changes to platforms into 4 patches\n * Made \"do not zero vmemmap_buf\" as a separate patch\n\nv2 - v1\n- Per request, added s390 to deferred \"struct page\" zeroing\n- Collected performance data on x86 which proofs the importance to\n keep memset() as prefetch (see below).\n\nSMP machines can benefit from the DEFERRED_STRUCT_PAGE_INIT config option,\nwhich defers initializing struct pages until all cpus have been started so\nit can be done in parallel.\n\nHowever, this feature is sub-optimal, because the deferred page\ninitialization code expects that the struct pages have already been zeroed,\nand the zeroing is done early in boot with a single thread only. Also, we\naccess that memory and set flags before struct pages are initialized. All\nof this is fixed in this patchset.\n\nIn this work we do the following:\n- Never read access struct page until it was initialized\n- Never set any fields in struct pages before they are initialized\n- Zero struct page at the beginning of struct page initialization\n\n\n==========================================================================\nPerformance improvements on x86 machine with 8 nodes:\nIntel(R) Xeon(R) CPU E7-8895 v3 @ 2.60GHz and 1T of memory:\n TIME SPEED UP\nbase no deferred: 95.796233s\nfix no deferred: 79.978956s 19.77%\n\nbase deferred: 77.254713s\nfix deferred: 55.050509s 40.34%\n==========================================================================\nSPARC M6 3600 MHz with 15T of memory\n TIME SPEED UP\nbase no deferred: 358.335727s\nfix no deferred: 302.320936s 18.52%\n\nbase deferred: 237.534603s\nfix deferred: 182.103003s 30.44%\n==========================================================================\nRaw dmesg output with timestamps:\nx86 base no deferred: https://hastebin.com/ofunepurit.scala\nx86 base deferred: https://hastebin.com/ifazegeyas.scala\nx86 fix no deferred: https://hastebin.com/pegocohevo.scala\nx86 fix deferred: https://hastebin.com/ofupevikuk.scala\nsparc base no deferred: https://hastebin.com/ibobeteken.go\nsparc base deferred: https://hastebin.com/fariqimiyu.go\nsparc fix no deferred: https://hastebin.com/muhegoheyi.go\nsparc fix deferred: https://hastebin.com/xadinobutu.go\n\nPavel Tatashin (12):\n x86/mm: setting fields in deferred pages\n sparc64/mm: setting fields in deferred pages\n mm: deferred_init_memmap improvements\n sparc64: simplify vmemmap_populate\n mm: defining memblock_virt_alloc_try_nid_raw\n mm: zero struct pages during initialization\n sparc64: optimized struct page zeroing\n mm: zero reserved and unavailable struct pages\n mm/kasan: kasan specific map populate function\n x86/kasan: use kasan_map_populate()\n arm64/kasan: use kasan_map_populate()\n mm: stop zeroing memory during allocation in vmemmap\n\n arch/arm64/include/asm/pgtable.h | 3 +\n arch/arm64/mm/kasan_init.c | 12 +--\n arch/sparc/include/asm/pgtable_64.h | 30 ++++++\n arch/sparc/mm/init_64.c | 31 +++---\n arch/x86/mm/init_64.c | 9 +-\n arch/x86/mm/kasan_init_64.c | 8 +-\n include/linux/bootmem.h | 27 +++++\n include/linux/kasan.h | 2 +\n include/linux/memblock.h | 16 +++\n include/linux/mm.h | 26 +++++\n mm/kasan/kasan_init.c | 67 ++++++++++++\n mm/memblock.c | 60 +++++++++--\n mm/page_alloc.c | 207 ++++++++++++++++++++----------------\n mm/sparse-vmemmap.c | 15 ++-\n mm/sparse.c | 6 +-\n 15 files changed, 380 insertions(+), 139 deletions(-)" }