From patchwork Fri Dec 4 22:05:59 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Scott Wood X-Patchwork-Id: 552896 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id A18971402B4 for ; Sat, 5 Dec 2015 09:07:14 +1100 (AEDT) Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 87B261A0F62 for ; Sat, 5 Dec 2015 09:07:14 +1100 (AEDT) X-Original-To: linuxppc-dev@lists.ozlabs.org Delivered-To: linuxppc-dev@lists.ozlabs.org Received: from na01-bn1-obe.outbound.protection.outlook.com (mail-bn1on0112.outbound.protection.outlook.com [157.56.110.112]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 7671A1A0519 for ; Sat, 5 Dec 2015 09:06:17 +1100 (AEDT) Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=scottwood@freescale.com; Received: from snotra.am.freescale.net (192.88.168.49) by BN3PR03MB1479.namprd03.prod.outlook.com (10.163.35.142) with Microsoft SMTP Server (TLS) id 15.1.337.19; Fri, 4 Dec 2015 22:06:09 +0000 From: Scott Wood To: Subject: [PATCH] powerpc/e6500: add locking to hugetlb Date: Fri, 4 Dec 2015 16:05:59 -0600 Message-ID: <1449266759-3418-1-git-send-email-scottwood@freescale.com> X-Mailer: git-send-email 2.5.0 MIME-Version: 1.0 X-Originating-IP: [192.88.168.49] X-ClientProxiedBy: BY2PR1001CA0026.namprd10.prod.outlook.com (25.164.163.164) To BN3PR03MB1479.namprd03.prod.outlook.com (25.163.35.142) X-Microsoft-Exchange-Diagnostics: 1; BN3PR03MB1479; 2:jCcHzc2CiEHdu/2EiY+nTquEUAR/gPTuVax/O2HCE83vvKjtm1N/2dZkhhhjGpPDrvSafzY8bSYExIWH/j72NzMPQ74FRz0OQLnIZ1LmDh50iQak59GENkJ4jwyb+ZVxjJMVo0YyrGBzGAzaW5TKKA==; 3:2TwvrZf2ZstMPX5D+/GQc9QvtSl5roQhLRzVt8TgiF8xQM0FRYJPKgnDf3K9a/7hoQ4NtIUOBJV7IRcE9VSNaRGYVYOWcWldv2vmSx8iR5Dz/0wjEERxvx5X3YkXqmf6; 25:GnkoxvmYiYkFykrZO95CkEJLJ6XXQ+RY1yzRpwmq8BkKu4Qv7myAsGQ3bxFL7T1bmJ4bwj6s4AdE+uBhgbGT8lyKumSCaAIWo9nRk8mmBF2wSckv9mkhHsfVwSUYciIMOm09wk5/07d53nP5XGlC9voHhU6LHJqGCR9j4j7/WMDaq2c1tcJ4kRmoFoLoo2JIhaj31K4Bzxq6fo+PMX2xU/sxuMZ7VJPinp4kl79gQIYH0GYYRZg0yXbvDvg/WnTRGjm58INIK9v7CFQMGwwuxw== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN3PR03MB1479; X-Microsoft-Exchange-Diagnostics: 1; BN3PR03MB1479; 20:muh8YQhHq35BL3Dsl7iJr98pl2aYi8Z0dA8ES5Fyz7aliBeEeOcUYXpUdPxruIG0JitTQTh5fCJf/5kDuRNo0I/3fMA2YLjoszHCqCWJCITpoCRRn3jaxdD9l7++B2TuWBSCSVxGEKvwZMVJ/la+tdZnV67ig38lvdra/8F+YauyLXHOAAzK31BaqiZYKCfNfzt7/zBDpOStHjUQz0Gf482A1H8nWXNpCyJS76aXfEG36wU+LUKX7DJUO4fTKz/BkhYn/wUZrcmOSebcAaL7Gi4Sh6C1iJMDM35kjl6IHNWzVVSZjA7JxRgW+TZOAlBtf9FED6IulHtBzJRIBh1Fh25oNA3Cd3cUHX0cuZpxRKStwf6K9l8ijTr6kQkwFtdPIqVwoofQuAzgLnOIO6QLAmMgeaoqVWeX6TsJS3qTobGoB7oTJF/hRphDpQe+kyaWbUoFRE9KjYJM8geGOlKUWsOva7dsqfWeIlTy6kQ4I8nnFOr5aMj+ImyOCvLUlDR6; 4:uGs+hxoGfFEI+G8doF4A58yYfGM907YDzy7owWrl2Bbmo1M8JfDdcP0B8uW0hGZp92+ru/hWk1EI0kpkyvikhGppgqcfoIn/N9pcRHx+N8UoHqH8Er9t4Bi7sNsnorTOVyu+UwHLh47GzRh1HYguZS2lCcgOd4rgwBD1fxRdT2gepJXkce5AcNXE5bzqLIJxZzPiAmbr9GrAEi2waEkt8Po+ToDSR9RJBqhvKdh5Vsv5HbZRBc6XOeTAtvvBG9yDKtDy1ec9fKfTPHRjeh1cpwqPkEMaiqx+xuxPcgwPQQsd5ASqQ6cNFu5hZ0rIttukmpWj5JDwAMmZlzZjSjW3l3eEJ2zwyK01YkcJ5UXfSYLuidnatWHTi7ZIU594h8wvEJr6QV9zXbMPxGwjpYA6WQ0XbGqIennK4UXBMnz4kWtB5cVnMZ+N+IiRwPy7nmBN X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(101931422205132); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(5005006)(520078)(8121501046)(3002001)(10201501046); SRVR:BN3PR03MB1479; BCL:0; PCL:0; RULEID:; SRVR:BN3PR03MB1479; X-Forefront-PRVS: 07807C55DC X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10019020)(6009001)(199003)(189002)(101416001)(229853001)(189998001)(33646002)(105586002)(87976001)(50226001)(50466002)(106356001)(48376002)(107886002)(122386002)(5004730100002)(97736004)(42186005)(36756003)(5001960100002)(81156007)(19580405001)(47776003)(19580395003)(66066001)(86362001)(2351001)(92566002)(4001430100002)(3846002)(5008740100001)(5003940100001)(110136002)(1096002)(586003)(77096005)(6116002)(40100003)(450100001)(50986999); DIR:OUT; SFP:1102; SCL:1; SRVR:BN3PR03MB1479; H:snotra.am.freescale.net; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: freescale.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BN3PR03MB1479; 23:JRt/EkLWOVBUmNhNwhQLo/uE7Swer4uHzo6A7SE7c?= =?us-ascii?Q?VYdLVKBE4ybuUnLNTZCTPfT9a9uCAiKv6ZXziFPwlAviv7X1jp+HxTxBJoQE?= =?us-ascii?Q?j6x2MDOZyuTOijOB0ZRqSDAf4e4HtCWad2c8UV14VmGludU7NRlfU/7/LXen?= =?us-ascii?Q?rEUqmEATNMmfmLLECKYVKgi9GfmuL2ddO+NIf42dy/KR9IBHqcWhQCS3JAXc?= =?us-ascii?Q?yGBlns1ZO7B2itTm9FbDUYMCeSiEm6PoWZ+VT+LeUsbWEp8Fk1xDhT3hsI+b?= =?us-ascii?Q?8RlSztKlvPMdU4i7BvS6GhJhbfcqBBzcB65wuKXIh7a/KAOI2XSCQwrLsagq?= =?us-ascii?Q?fwt9uyZGqsQ9nx1coimGnziRBBmJd7JjivdvGXo/mQqxMUFIqPFGN4nIlcYr?= =?us-ascii?Q?wy1AhLjvKzn8NmmiknpLxP0XY7nXYKVqlEqieNsj/gDHfUNHqcgju3+E3xc4?= =?us-ascii?Q?z6mYeB/JdexL24pT9tC4TUF58o10KnvmTEbzerRQSqJxoCdZ/0Mjv5ngBT7Y?= =?us-ascii?Q?HmambrJX15rAwLoDUv1pZuZxz87V4iar4UKd9r8Wl7Mx1d+U4128favsTdeh?= =?us-ascii?Q?NLMgWkca66bWbcqo2n0cPpSeQq7UrI0xgYmnLBHLHcbFMxvjdp5JUXvZDZkf?= =?us-ascii?Q?UrbnCahIoHd6QsFhZQ12FAEqBUMnkKlqL9xn2kkybv3/9xJkHiG23qy30+v4?= =?us-ascii?Q?EmAxcgEhRRvgB+QAvBjgdh/wxxk13xi5XAEn6Stmg+T6DpiSTb74l75Xg6Qx?= =?us-ascii?Q?oxv8YNtfycTvh8cyfCleALTNT6ahrtsxfhzqT7IPvX2bF0ZVOiXKM3sa0A3d?= =?us-ascii?Q?p7P6yXelI004PofDG4WdiodAf7B3TFK8wvJb9E+e6N5psK7czmgXt3UZ7+py?= =?us-ascii?Q?EqR4RrNjXiUeAeuXHMzcZemTH+bE6XdiR+Kts0Rp5m5frxbrexLEMw6e25Nc?= =?us-ascii?Q?6Vtyzlso+YORbMogpGGb8Klgl7q8nhQxTxDgjemRF4NQJVpmeZ3k8C45djJG?= =?us-ascii?Q?q0aEmVLD7iFkSuo4icuELXg1hj60j4IRzQGVzJ7OgHN8RsDlSN6eFMDfhsT0?= =?us-ascii?Q?0hamHttWzFq67keZHM3VbQsPeHM?= X-Microsoft-Exchange-Diagnostics: 1; BN3PR03MB1479; 5:MSxWkM7fJRhhw2OktwccWLwdRpaqL2owxPr6F+S3Dhxwsl5wHF7K69wt0HNQIn73lo85nbmr+nivWl+GG5JXB3VchyBTNh9EqeM0HiOOKyZYaHcRsdtwlG8vXuIf46nrUUVyaEuvSEDA0waqIn80gw==; 24:ORrLN2PZrekYXL30dgFzh9pfIZdDHDnzCEZNv3ATwb5tNuZ7MZHUM5PLkLz2TDVON1SgH5dn4YRu+rAhWHYC+humDFHM7CvCpaXLBxJe8M8= X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Dec 2015 22:06:09.1351 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR03MB1479 X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Scott Wood Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: "Linuxppc-dev" e6500 has threads but does not have TLB write conditional. Thus, the hugetlb code needs to take the same lock that the normal TLB miss handlers take, to ensure that the tlbsx and tlbwe are atomic. Signed-off-by: Scott Wood --- arch/powerpc/mm/hugetlbpage-book3e.c | 46 ++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) diff --git a/arch/powerpc/mm/hugetlbpage-book3e.c b/arch/powerpc/mm/hugetlbpage-book3e.c index ba47aaf..180707e 100644 --- a/arch/powerpc/mm/hugetlbpage-book3e.c +++ b/arch/powerpc/mm/hugetlbpage-book3e.c @@ -51,6 +51,48 @@ static inline int mmu_get_tsize(int psize) return mmu_psize_defs[psize].enc; } +#if defined(CONFIG_PPC_FSL_BOOK3E) && defined(CONFIG_PPC64) +#include + +static inline void book3e_tlb_lock(void) +{ + struct paca_struct *paca = get_paca(); + unsigned long tmp; + int token = smp_processor_id() + 1; + + asm volatile("1: lbarx %0, 0, %1;" + "cmpwi %0, 0;" + "bne 2f;" + "stbcx. %2, 0, %1;" + "bne 1b;" + "b 3f;" + "2: lbzx %0, 0, %1;" + "cmpwi %0, 0;" + "bne 2b;" + "b 1b;" + "3:" + : "=&r" (tmp) + : "r" (&paca->tcd_ptr->lock), "r" (token) + : "memory"); +} + +static inline void book3e_tlb_unlock(void) +{ + struct paca_struct *paca = get_paca(); + + isync(); + paca->tcd_ptr->lock = 0; +} +#else +static inline void book3e_tlb_lock(void) +{ +} + +static inline void book3e_tlb_unlock(void) +{ +} +#endif + static inline int book3e_tlb_exists(unsigned long ea, unsigned long pid) { int found = 0; @@ -109,7 +151,10 @@ void book3e_hugetlb_preload(struct vm_area_struct *vma, unsigned long ea, */ local_irq_save(flags); + book3e_tlb_lock(); + if (unlikely(book3e_tlb_exists(ea, mm->context.id))) { + book3e_tlb_unlock(); local_irq_restore(flags); return; } @@ -141,6 +186,7 @@ void book3e_hugetlb_preload(struct vm_area_struct *vma, unsigned long ea, asm volatile ("tlbwe"); + book3e_tlb_unlock(); local_irq_restore(flags); }