Cover Letter Detail
Show a cover letter.
GET /api/covers/816461/?format=api
{ "id": 816461, "url": "http://patchwork.ozlabs.org/api/covers/816461/?format=api", "web_url": "http://patchwork.ozlabs.org/project/sparclinux/cover/20170920201714.19817-1-pasha.tatashin@oracle.com/", "project": { "id": 10, "url": "http://patchwork.ozlabs.org/api/projects/10/?format=api", "name": "Linux SPARC Development ", "link_name": "sparclinux", "list_id": "sparclinux.vger.kernel.org", "list_email": "sparclinux@vger.kernel.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/sparclinux/cover/20170920201714.19817-1-pasha.tatashin@oracle.com/mbox/", "series": [ { "id": 4223, "url": "http://patchwork.ozlabs.org/api/series/4223/?format=api", "web_url": "http://patchwork.ozlabs.org/project/sparclinux/list/?series=4223", "date": "2017-09-20T20:17:06", "name": "complete deferred page initialization", "version": 9, "mbox": "http://patchwork.ozlabs.org/series/4223/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/816461/comments/", "headers": { "Return-Path": "<sparclinux-owner@vger.kernel.org>", "X-Original-To": "patchwork-incoming@ozlabs.org", "Delivered-To": "patchwork-incoming@ozlabs.org", "Authentication-Results": "ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=sparclinux-owner@vger.kernel.org;\n\treceiver=<UNKNOWN>)", "Received": [ "from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyB3p4BhGz9s7v\n\tfor <patchwork-incoming@ozlabs.org>;\n\tThu, 21 Sep 2017 06:21:58 +1000 (AEST)", "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751484AbdITUSm (ORCPT <rfc822;patchwork-incoming@ozlabs.org>);\n\tWed, 20 Sep 2017 16:18:42 -0400", "from aserp1040.oracle.com ([141.146.126.69]:22841 \"EHLO\n\taserp1040.oracle.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751000AbdITUSk (ORCPT\n\t<rfc822; sparclinux@vger.kernel.org>); Wed, 20 Sep 2017 16:18:40 -0400", "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" ], "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,\n\twilly@infradead.org, mhocko@kernel.org, ard.biesheuvel@linaro.org,\n\tmark.rutland@arm.com, will.deacon@arm.com, catalin.marinas@arm.com,\n\tsam@ravnborg.org, mgorman@techsingularity.net,\n\tsteven.sistare@oracle.com, daniel.m.jordan@oracle.com,\n\tbob.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]", "Sender": "sparclinux-owner@vger.kernel.org", "Precedence": "bulk", "List-ID": "<sparclinux.vger.kernel.org>", "X-Mailing-List": "sparclinux@vger.kernel.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(-)" }