From patchwork Tue Nov 15 12:58:55 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Marek Szyprowski X-Patchwork-Id: 82325 Delivered-To: patch@linaro.org Received: by 10.140.97.165 with SMTP id m34csp1513951qge; Tue, 15 Nov 2016 05:00:59 -0800 (PST) X-Received: by 10.99.158.18 with SMTP id s18mr27368126pgd.163.1479214859707; Tue, 15 Nov 2016 05:00:59 -0800 (PST) Return-Path: Received: from bombadil.infradead.org (bombadil.infradead.org. [2001:1868:205::9]) by mx.google.com with ESMTPS id t3si26435484pfg.143.2016.11.15.05.00.59 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 15 Nov 2016 05:00:59 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-arm-kernel-bounces+patch=linaro.org@lists.infradead.org designates 2001:1868:205::9 as permitted sender) client-ip=2001:1868:205::9; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-arm-kernel-bounces+patch=linaro.org@lists.infradead.org designates 2001:1868:205::9 as permitted sender) smtp.mailfrom=linux-arm-kernel-bounces+patch=linaro.org@lists.infradead.org Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.85_2 #1 (Red Hat Linux)) id 1c6dKu-0006lF-2P; Tue, 15 Nov 2016 12:59:40 +0000 Received: from mailout4.w1.samsung.com ([210.118.77.14]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1c6dKp-0006iw-B7 for linux-arm-kernel@lists.infradead.org; Tue, 15 Nov 2016 12:59:37 +0000 Received: from eucas1p2.samsung.com (unknown [182.198.249.207]) by mailout4.w1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0OGO0032EPEN3W60@mailout4.w1.samsung.com> for linux-arm-kernel@lists.infradead.org; Tue, 15 Nov 2016 12:59:11 +0000 (GMT) Received: from eusmges1.samsung.com (unknown [203.254.199.239]) by eucas1p1.samsung.com (KnoxPortal) with ESMTP id 20161115125910eucas1p1841f8b98d2b63a77e8513c32295f4b6a~HOfqIaNYK2569825698eucas1p1-; Tue, 15 Nov 2016 12:59:10 +0000 (GMT) Received: from eucas1p2.samsung.com ( [182.198.249.207]) by eusmges1.samsung.com (EUCPMTA) with SMTP id 82.C0.23383.F960B285; Tue, 15 Nov 2016 12:59:11 +0000 (GMT) Received: from eusmgms2.samsung.com (unknown [182.198.249.180]) by eucas1p2.samsung.com (KnoxPortal) with ESMTP id 20161115125910eucas1p2ab762d2c7336130d7d2a81150f53f1aa~HOfpdecqY3095130951eucas1p2r; Tue, 15 Nov 2016 12:59:10 +0000 (GMT) X-AuditID: cbfec7ef-f79e76d000005b57-d8-582b069fb966 Received: from eusync2.samsung.com ( [203.254.199.212]) by eusmgms2.samsung.com (EUCPMTA) with SMTP id 4F.D6.10494.2860B285; Tue, 15 Nov 2016 12:58:43 +0000 (GMT) Received: from AMDC2765.digital.local ([106.116.147.25]) by eusync2.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id <0OGO002J0PEIRGA0@eusync2.samsung.com>; Tue, 15 Nov 2016 12:59:09 +0000 (GMT) From: Marek Szyprowski To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v2] ARM: dma-mapping: preallocate DMA-debug hash tables in core_initcall Date: Tue, 15 Nov 2016 13:58:55 +0100 Message-id: <1479214735-11324-1-git-send-email-m.szyprowski@samsung.com> X-Mailer: git-send-email 1.9.1 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrKIsWRmVeSWpSXmKPExsWy7djP87rz2bQjDBZfULTYOGM9q8Wmx9dY LdYeuctusfnDSzaLGZNfsjmwerQ097B5bF5S79G3ZRVjAHMUl01Kak5mWWqRvl0CV8aSW6tY C+bKVsyYvp25gbFRqouRk0NCwETiZPdDdghbTOLCvfVsXYxcHEICyxglDl7+wwLhfGaUWH1+ NmMXIwdYx4n/MSANYEUT210gahqYJKZMmAE2iU3AUKLrbRcbiC0ioCExpesxO0gRs8BeRokX p7axgCSEBaIkpizfDdbAIqAqsWDGNlYQm1fAQ+J4905WiJPkJE4emwxl72CTeHzKBOIIWYlN B5ghwi4SL9peQJUIS7w6vgXqGxmJy5O7WSDsfkaJplZtCHsGo8S5t7wQtrXE4eMXwXqZBfgk Jm2bzgwxnleio00IwvSQmHVOCaLaUaJ/2W5miNdjJa7u3sA2gVF6ASPDKkaR1NLi3PTUYkO9 4sTc4tK8dL3k/NxNjMDIO/3v+PsdjE+bQw4xCnAwKvHwCpzQjBBiTSwrrsw9xCjBwawkwmvJ qh0hxJuSWFmVWpQfX1Sak1p8iFGag0VJnHfvgivhQgLpiSWp2ampBalFMFkmDk6pBsbqI252 gm/X/kviefYnV1le20za8fXHZ5Zq7XIGVtxff98UOvi1y3Rzwz/7tX9/+P9UXRPX8qJyZtiD re2r5oVEZD09LDzpU+gmoRVfrvdOD/8xR5L504MZC99vd2yJalQ93xAk9U5Lrbb76Xox3YZn mebVmQr6G3UV2PiCtiXc/uglfH/tZD8lluKMREMt5qLiRABDZJcPuAIAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrNLMWRmVeSWpSXmKPExsVy+t/xK7rNbNoRBp/yLDbOWM9qsenxNVaL tUfuslts/vCSzWLG5JdsDqweLc09bB6bl9R79G1ZxRjAHOVmk5GamJJapJCal5yfkpmXbqsU GuKma6GkkJeYm2qrFKHrGxKkpFCWmFMK5BkZoAEH5wD3YCV9uwS3jCW3VrEWzJWtmDF9O3MD Y6NUFyMHh4SAicSJ/zFdjJxAppjEhXvr2UBsIYEljBIz10h3MXIB2U1MEg/XHQFLsAkYSnS9 7QKzRQQ0JKZ0PWYHKWIW2M8o8fvAD1aQhLBAlMSU5bvZQWwWAVWJBTO2gcV5BTwkjnfvZIXY Jidx8thk1gmM3AsYGVYxiqSWFuem5xYb6RUn5haX5qXrJefnbmIEBty2Yz+37GDsehd8iFGA g1GJh3fHUc0IIdbEsuLK3EOMEhzMSiK8lqzaEUK8KYmVValF+fFFpTmpxYcYTYGWT2SWEk3O B0ZDXkm8oYmhuaWhkbGFhbmRkZI479QPV8KFBNITS1KzU1MLUotg+pg4OKUaGJu47Q/WZ/5X uj7xiJSkuoVNRdVUrxth6Qfsy15tfOVqJCuZ5M5huvi4rpz/B56JIZttgpI2x0n3eNVWPJ7r /+7llvnvvkZeu8DPuy5e6d6xZPfmiBscFx3X/WGQE9I909uS1fFJgF9N9/em5JPBuXtXrlJm YrVjTU9uzzFvrz7ybKeykjezEktxRqKhFnNRcSIAwz0gLk4CAAA= X-MTR: 20000000000000000@CPGS X-CMS-MailID: 20161115125910eucas1p2ab762d2c7336130d7d2a81150f53f1aa X-Msg-Generator: CA X-Sender-IP: 182.198.249.180 X-Local-Sender: =?UTF-8?B?TWFyZWsgU3p5cHJvd3NraRtTUlBPTC1LZXJuZWwgKFRQKRs=?= =?UTF-8?B?7IK87ISx7KCE7J6QG1NlbmlvciBTb2Z0d2FyZSBFbmdpbmVlcg==?= X-Global-Sender: =?UTF-8?B?TWFyZWsgU3p5cHJvd3NraRtTUlBPTC1LZXJuZWwgKFRQKRtT?= =?UTF-8?B?YW1zdW5nIEVsZWN0cm9uaWNzG1NlbmlvciBTb2Z0d2FyZSBFbmdpbmVlcg==?= X-Sender-Code: =?UTF-8?B?QzEwG0VIURtDMTBDRDAyQ0QwMjczOTI=?= CMS-TYPE: 201P X-HopCount: 7 X-CMS-RootMailID: 20161115125910eucas1p2ab762d2c7336130d7d2a81150f53f1aa X-RootMTR: 20161115125910eucas1p2ab762d2c7336130d7d2a81150f53f1aa References: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20161115_045935_617020_F3FA73A5 X-CRM114-Status: GOOD ( 16.05 ) X-Spam-Score: -8.3 (--------) X-Spam-Report: SpamAssassin version 3.4.0 on bombadil.infradead.org summary: Content analysis details: (-8.3 points) pts rule name description ---- ---------------------- -------------------------------------------------- -5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/, high trust [210.118.77.14 listed in list.dnswl.org] -0.0 RCVD_IN_MSPIKE_H3 RBL: Good reputation (+3) [210.118.77.14 listed in wl.mailspike.net] -0.0 SPF_HELO_PASS SPF: HELO matches SPF record -1.4 RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] -0.0 RCVD_IN_MSPIKE_WL Mailspike good senders X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Russell King , Bartlomiej Zolnierkiewicz , Seung-Woo Kim , Marek Szyprowski MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patch=linaro.org@lists.infradead.org fs_initcall is definitely too late to initialize DMA-debug hash tables, because some drivers might get probed and use DMA mapping framework already in core_initcall. Late initialization of DMA-debug results in false warning about accessing memory, that was not allocated, like this one: ------------[ cut here ]------------ WARNING: CPU: 5 PID: 1 at lib/dma-debug.c:1104 check_unmap+0xa1c/0xe50 exynos-sysmmu 10a60000.sysmmu: DMA-API: device driver tries to free DMA memory it has not allocated [device address=0x000000006ebd0000] [size=16384 bytes] Modules linked in: CPU: 5 PID: 1 Comm: swapper/0 Not tainted 4.9.0-rc5-00028-g39dde3d-dirty #44 Hardware name: SAMSUNG EXYNOS (Flattened Device Tree) [] (unwind_backtrace) from [] (show_stack+0x20/0x24) [] (show_stack) from [] (dump_stack+0x84/0xa0) [] (dump_stack) from [] (__warn+0x14c/0x180) [] (__warn) from [] (warn_slowpath_fmt+0x48/0x50) [] (warn_slowpath_fmt) from [] (check_unmap+0xa1c/0xe50) [] (check_unmap) from [] (debug_dma_unmap_page+0x98/0xc8) [] (debug_dma_unmap_page) from [] (exynos_iommu_domain_free+0x158/0x380) [] (exynos_iommu_domain_free) from [] (iommu_domain_free+0x34/0x60) [] (iommu_domain_free) from [] (release_iommu_mapping+0x30/0xb8) [] (release_iommu_mapping) from [] (arm_iommu_release_mapping+0x4c/0x50) [] (arm_iommu_release_mapping) from [] (s5p_mfc_probe+0x640/0x80c) [] (s5p_mfc_probe) from [] (platform_drv_probe+0x70/0x148) [] (platform_drv_probe) from [] (driver_probe_device+0x12c/0x6b0) [] (driver_probe_device) from [] (__driver_attach+0x128/0x17c) [] (__driver_attach) from [] (bus_for_each_dev+0x88/0xc8) [] (bus_for_each_dev) from [] (driver_attach+0x34/0x58) [] (driver_attach) from [] (bus_add_driver+0x18c/0x32c) [] (bus_add_driver) from [] (driver_register+0x98/0x148) [] (driver_register) from [] (__platform_driver_register+0x58/0x74) [] (__platform_driver_register) from [] (s5p_mfc_driver_init+0x1c/0x20) [] (s5p_mfc_driver_init) from [] (do_one_initcall+0x64/0x258) [] (do_one_initcall) from [] (kernel_init_freeable+0x3d0/0x4d0) [] (kernel_init_freeable) from [] (kernel_init+0x18/0x134) [] (kernel_init) from [] (ret_from_fork+0x14/0x3c) ---[ end trace dc54c54bd3581296 ]--- This patch moves initialization of DMA-debug to core_initcall. This is safe from the initialization perspective. dma_debug_do_init() internally calls debugfs functions and debugfs also gets initialised at core_initcall(), and that is earlier than arch code in the link order, so it will get initialized just before the DMA-debug. Reported-by: Seung-Woo Kim Signed-off-by: Marek Szyprowski --- Changelog: v2: - changed to core_initcall on suggestion of Russell King --- arch/arm/mm/dma-mapping.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 1.9.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c index ab4f745..d1abbcf 100644 --- a/arch/arm/mm/dma-mapping.c +++ b/arch/arm/mm/dma-mapping.c @@ -1167,7 +1167,7 @@ static int __init dma_debug_do_init(void) dma_debug_init(PREALLOC_DMA_DEBUG_ENTRIES); return 0; } -fs_initcall(dma_debug_do_init); +core_initcall(dma_debug_do_init); #ifdef CONFIG_ARM_DMA_USE_IOMMU