From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933357AbdCUUWX (ORCPT ); Tue, 21 Mar 2017 16:22:23 -0400 Received: from mail-by2nam03on0052.outbound.protection.outlook.com ([104.47.42.52]:11488 "EHLO NAM03-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1757233AbdCUUWU (ORCPT ); Tue, 21 Mar 2017 16:22:20 -0400 Authentication-Results: cavium.com; dkim=none (message not signed) header.d=none;cavium.com; dmarc=none action=none header.from=caviumnetworks.com; Subject: Re: [PATCH v12 4/9] mmc: cavium: Work-around hardware bug on cn6xxx and cnf7xxx To: Arnd Bergmann References: <20170310132507.32025-1-jglauber@cavium.com> <20170310132507.32025-5-jglauber@cavium.com> <6dc8a2a3-ca79-745a-c716-2188319b9378@caviumnetworks.com> <67481a9f-f7b5-4ec0-22cc-f6e019dc131e@caviumnetworks.com> Cc: Ulf Hansson , Jan Glauber , "linux-mmc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "Steven J . Hill" , David Daney From: David Daney Message-ID: <79a1e294-797d-8b1f-0dcd-129df268c089@caviumnetworks.com> Date: Tue, 21 Mar 2017 13:22:05 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [50.233.148.156] X-ClientProxiedBy: BY2PR07CA060.namprd07.prod.outlook.com (10.141.251.35) To SN1PR07MB2431.namprd07.prod.outlook.com (10.169.127.143) X-MS-Office365-Filtering-Correlation-Id: e4f0e93f-5b0a-463b-dc17-08d47097f846 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:SN1PR07MB2431; X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2431;3:z7kEhuwov7r78+nVxOuET+Q8c4oM+1jzv+GXc9j3Q2oi9WWo9EOknayNLlFqQZoDYSMUCNKCXjtkhT/VlFcQbLe0RV4BOsVrNF2f5fOLo8gsIi5yCyCEcwtIr0gSVCZgKeIL1Ntq9S9aF+DK35ia3zzKcauIqystm62LJlrf0s2VQ8kq0rm5ybZEl2lgl33Sgx+t8j//N3Mehxi6XCcrkdwI5pEchREBywK6JPNns2h53D8OvPp5mC6nKK7hXS2f2kVk2kTjBuJkZx8WtJOGTQ==;25:yOM8EE+E4sO7wLYIq8A0awd0xEhudyuVLKhgE3iT1haha8tVApQoTbs4hPsSnN3zzaPlsMqufiiHDRooAEJg0+aP3LWncx51/BTkrbB4KF4QSeU//Mr31ruM6l023LqndK8alngKMcOZ0+qH5VIqnhdKaDyJpE7Nn/PtUrprkiHzspZXQHhLsCRKynMfcE//6amwQsgkb9e2Q7KO12b0BHyEYJETSuw+F0uKEGFEF7v8aaCFx9wjGKvNPJV8P9qDaaIcQeL08n+aQMnp47Rc4Ei3FciOjqUQIjAg0jGWJGkTz9q9oUPCkSL2t5CU2kdDIBrvn6ZfegotKecyOg9QxYxxkl1NoT8Xs0mWBSyQqMJ1zagZZRMG+U2a+q71A8T0MCh/hQZJxYQSuQ+Oyt8r/BSXpRNRO3aii2o5N2uZIEe7snIrNyaR9Y03OlKFfw7yM9WBEFVkO3QKlf/EBrNOxw== X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2431;31:1F01NdU5Vuiy4XqDDNNjijRjQxVMq38EOld1uUcbB3oRupMllDi+RHzDnrs7tNursT+aUsvDHPeIRQ1JpDhMMC8brFI9VuP0WPALxT9hAo4DBt3BHQjB3OBhjjPEnkqTWVuSQT25awWARYjqUINOKZPWXNWnT7PswQRKqkCBUoHyafzXeO4ZBMpd0cmMtITRF7oh3ubwr/Nak6ogvHWW6TTWRaWJl0zKVzmjdTP9bZw=;20:xx8CJNHO+2azWh/Ej76PVmIfm4+2eqz44xSmFBuJOMNjsYEIn4BbBv0G3Wkh73xkfcmr3WOMDKhOB8/Rbp6CZj4ungm1TdtZaepxSygHth0X0D79ytJiBogXTjeB4/0DviJS++B6e/1CK5aoJCHyPbjFzubWS3a+qbUAUhwd6imPVAhP5YX9Sla2B7XO6zE1/Xb8ZFK0vSga34StyiPzHjreLaAUDAKwkVbi1Eox0H5kD43ASihTOWsj6Ca0l4VvOeeSRLUcD5gz1V7WYKKYA3bDwgX4BRVW23g8hsqOwNXbcF/k1sk/O7Kz5OmLj17LDs6WB8APBxDniBnE/q3V1CYmx8oKT/PpjEwHgw3BEQpF4/xy6lqII7XxLV0hZ6VLzvlAz235Bq5JOiquVAVEg5N9OLVxDPGgPEXqDlv5gG7CsAQZPPMrUT3r+zGHMuwXdqZoV2FQV4Iw8fYKi4y2swLAMG8XSxgz90A07PNV5yFK8iXy2ZnX4QLiCj4KUTdmkwNoXBx8vI/hRbt96Vkvz5/LO9wPlAVplgimiDj6q6aIMxC47UwHbXZUXpoWLuC9jA6rnz3b15Lj1tw2Qdgoz4q3zfn0AbNoBV4DKo68YGc= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6041248)(20161123564025)(20161123562025)(20161123560025)(20161123558025)(20161123555025)(6072148);SRVR:SN1PR07MB2431;BCL:0;PCL:0;RULEID:;SRVR:SN1PR07MB2431; X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2431;4:LNf3d4a7ZSWZCQgLqTAURp7mFp9uU1uUMs3QSuuPdRIuSaUK3lL8WlGv6xqZvxnichbShn5JsFFQiAyTa0QEEosy+bWTN4mOO8tpXXUiacQanBgNz77lt+xK8BL7PrJnJqxpIAgu+P3pXM2Z+gnrBRJpf5/ieShY7AZtFcmYH1e0BloVh3BVm8U1rtGJ/Lshd9Fyd0AUWLRW6ifaGz2jfApa/FJQf4pUC60F76OEbNXsr56UyVouH8diYzJgLrbKL6c4tbOj2w/fe7QYG3k+Lb6fjlpUIjR8xyCu2E1efOtnA3NcYGyE9OIODgbM3FZWj0v2jxeZARKzrKwWXfMSPUioTauQSpxGipq4w+tHHENsb4qL9HwDtOjVok5R6mnRwQaKFK4RWWcqdBA6g0DWGsVjcA/Ohvyk9hTwQL18tgD75kYLnrd61GYnHgwiY3Oj6SSjkUqh6iCoyGRpqUfKHMQiCi8zvMDNIXgurx0l2jJC2584BgsjdLHo7hZu0/RdfYzk6liGZgvyeN9TAgi7u1oRR+2ufETkDsHuOT7K5mubHdDYwHzkE5wgLVTopOL4HwUmMdwJHHZxxEPwffqGoLnf7f+TafV3yuSTumbUVAA= X-Forefront-PRVS: 02530BD3AA X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(39830400002)(39450400003)(39410400002)(24454002)(377454003)(305945005)(81166006)(6666003)(6116002)(3846002)(110136004)(53936002)(93886004)(230700001)(7736002)(2950100002)(107886003)(42882006)(38730400002)(229853002)(6916009)(47776003)(25786009)(76176999)(50986999)(53416004)(4326008)(6506006)(65956001)(54356999)(54906002)(31696002)(6512007)(6486002)(23676002)(2906002)(36756003)(50466002)(42186005)(33646002)(64126003)(66066001)(83506001)(31686004)(189998001)(8676002)(53546009)(4001350100001)(5660300001)(32563001);DIR:OUT;SFP:1101;SCL:1;SRVR:SN1PR07MB2431;H:ddl.caveonetworks.com;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtTTjFQUjA3TUIyNDMxOzIzOlFvU2RlWEdTNVJMVE9Ka0V2a0RWVDhrSXpV?= =?utf-8?B?U0I2dDhhcmlVdURpNk9GczBBcEs5OEEzWFV6VlBQZkxWdFZNMTBFV2dtSyt2?= =?utf-8?B?dkhOVlhZcGlhQkxFYUZpWlpZV3VLenZ4eWMzMklySjRpUkgxRjM5VUhsamIy?= =?utf-8?B?WE5BallxcTJyakZLQ2xCWGlrcW9QRTA5RUtiM1lwRmJIQ2lqQzRHa1IyN1py?= =?utf-8?B?aWxaMHRmN0p1VWRmOWRGUXBoR1RyQi9BaG8wVi80RHlLdVI4RXpWUC95UWZx?= =?utf-8?B?MUk1clQ0VnVjOFVBK0ZqY0txVTd6aGFqK1Rpak1EV0xNK2c5NUtXS2M3VmU5?= =?utf-8?B?dHFkTVZFaTR2K2t2NXN4ejBhRTllOGR4Y0QybFJMMFFhZThwRXpUOVlLSFpR?= =?utf-8?B?d0pCczNzRlZjdzNqbC8zbkllcmdmdU1XTzlDL1dPVjNveWNnYTVLd1FIQ2tm?= =?utf-8?B?bUh4Wnc4NmRueXNQN1FUUHhVT2Q4blBVYVVUNkU1Tk5GSXBnZCtTandNSzFh?= =?utf-8?B?RHdqVG9IRk4zeVgxZFFTQnAxeHc0OGc3Z1gxa2t1V0lxYWFnbzFMT2h5cE1x?= =?utf-8?B?MU5peU1aN1YvcjJhQ0pZSnM0bzJZVjNUYk9lSE1MN1Faejhma2NvYWVYRE15?= =?utf-8?B?ZkNZS2dlT1dodXhIejdIUjUvTGh6SkNQMWNVUUNja3A2dldTQyszbERaZzMx?= =?utf-8?B?OS9EVXZvYWc1TGpxaUFoVkNKSFFlT0RUNTArdWlSWjFHRytEL3hTZmdVL0hy?= =?utf-8?B?ZXE2b3p2bld1M2ZkWlVvYnJnWjJUY2NQUEJHSkkrM0xoYU1vcFNiQkJsemNS?= =?utf-8?B?Q3BJaWYwV1hXQVNTZnlxMGs3bDRQSG45K1RhRzRNR0hTSGZ6Q3JvNzlxbVBo?= =?utf-8?B?WnZmOHk2RExhdkw5WG1KdnBEK0o1R2MveUh6Q28yVjFYS0lrTUtWdUI4NFdQ?= =?utf-8?B?ZEF2SmhTYjF5MDdXOStvVUJEQlFBcVFLUEhyUGpuYks5dGZ6ZVA1eU9heVdW?= =?utf-8?B?Z1BzRWNYNUUxU1ppVEJGMGhNS1NNWHBwc2ZuV0Zaek1GeGJBMyt0ZXhaanZr?= =?utf-8?B?cTRHbU55QnRuL3hFZG52YzJMNHQ1RERocmU5QzV3QnVVU25DZDAzZWN1NCt1?= =?utf-8?B?cVBsMXpob0VtTU9GRzVIMWNmWEVmMmtrS2ZHRG1GQTRzY1I5eWNaNC9CZ3BT?= =?utf-8?B?dnBHRTdGZ2pvQVFXY3UzZXFBTWI5QXZ6bjczMnNySzlJMXM0cDJWUk9zVFp6?= =?utf-8?B?Qm5FbXhERVUvNVdVOGdMemlaLzdkR1JPSmlmRDBlYnI0Vzlxc0x6QXFKTmlj?= =?utf-8?B?eXRKYk1mUkNoZ1RXMWtaSUFDZGJsaUhoalhuWm1CMDZnRXFiYWhVdGVocWhY?= =?utf-8?B?SGF5YU5HdjJDMHErbGRXNklzYlpSdzNkazNtbVl5VUFOTFJSR2NtVDVBNmVs?= =?utf-8?B?Vktzb0swaXVQNFBmSTdyZ3JucWZ5ZEV2dlJIZTc4NzB2bkNaVEZqQzhNa2lY?= =?utf-8?B?UXFDclJwU2lTMUQvNmNlc1pUQ1phV2piaDdoK2t1R20vZTFCVWlNb2gvV05R?= =?utf-8?B?Sy9ZRFB6bS9ycWJLUThTOEhXcHVQOGpjY2JSZWpmOFluR01SUmVnMHdpNnp4?= =?utf-8?B?Ni96UTh6MWUxVk5LbmV3TER4K2prZy9qMFBCekhLc25qRDJpNHpEcEV0MC9j?= =?utf-8?Q?w6sIx3pmfAJpxJ8+EZ2IALkzy6mJeFgQHp1gn+T?= X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2431;6:UKXHL3bvQErCwFr7gw5Q3Nnrvx+E/oM74AYvxBRvtjhpsR+Y0xYrq6FOvuxupkb9Wbe87xk5ODVYxJkUKECpGLJNyNj32muv5L8ygk+rowbOAmazmDpIkcWbKeXAbO4e2WvmfaakGK7caaxF4UWjHFxvE2exXiRDsz8e2bb9B4Q2tz789Ja21oagSd5uz74qltwfrL/biFF/AWFLEHOOzS/7o5TvRGvy3xHjq1qnQIem91egDzQY4Jbz8ry34SZM/2Q+dcjxRjXYmtmDzrH/HNPLpIQGYVKswJjZq+q2Lul4HVQ/BFcie06uMt7lqNqKR28AEh35yF+2fRMNKxER/XqTMdp+dU0dy4mCyvs9UBIEY58BqXJn5GSK6LZW8UxQAgNIDumgVPJEfkCCMrJ2cQ==;5:5QT94m72kZTfEGpZCRtaOWbvlLPbFt0GPAieAdriXPvkZRb8kOqsP5pjrohHGNq/eHKtCdbdIfxBiZJf1FP74qJmzPZ9/86U2iS+RW2V5PSCIe8lvFdq8CZBAfS0CFJF+/1s05rmyyWp4NaRfAX8rA==;24:H0uTiKm3PnifIJOFcwweDwTadwUy+LLid3vzWITxixjPhMfBblx0DgqMVYx5p73nQYhEhvSaRzI9T0WTxOUFdSGgtNG62jCIFUh7vQcrpKI= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2431;7:X5DmEZsL7HnpbcluA4tqwFGI9RO9OheuuSLGnXX6lNuYeVlrji1XJ4E83wcnR0mCvd3Qhg7vavEAG+DM8DFzrNiPH8n4eTKRh8M6GhgUmCcdhTCHUIg5tKKj0INEAo9DPnsX65RWLHxYsJctH9Og+ipscIaHM+U+D+sAS77CyI+EPuMKFgBXidYHu2LcLz5ygKhV3AHW5AmiVgdPH8QCwmC+nichW0iQPHXtNQ9d+eIrf/tmxJjkfxaOAVlW94p0lyQRZv6WHwSC9H0hlBT0ynfYm7n88yoAlgoC0Q4kOTzTcdC1zcMN9f3USJsNP6xxpEi3deq/4tlsVryrGXg7DA== X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Mar 2017 20:22:17.0605 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN1PR07MB2431 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/21/2017 12:49 PM, Arnd Bergmann wrote: > On Tue, Mar 21, 2017 at 4:19 PM, David Daney wrote: >> On 03/21/2017 01:58 AM, Arnd Bergmann wrote: >>> >>> On Mon, Mar 20, 2017 at 9:45 PM, David Daney >>> wrote: >>>> >>>> On 03/17/2017 07:13 AM, Ulf Hansson wrote: >>>>> >>>>> My point is really that we should avoid exporting SoC specific APIs >>>>> which shall be called from drivers. This is old fashion. >>>> >>>> >>>> >>>> Some people find it objectionable to see 1-off architecture specific >>>> in-line >>>> asm in a driver file, but I agree that putting it as close to the user as >>>> possible makes sense. >>> >>> >>> The proper solution might be to create an architecture independent >>> interface >>> for it, what it is that the function does. Can you explain what the >>> purpose >>> of locking/unlocking the cache line for MMC is? Is this something that >>> could be done more generally in the dma_map_ops implementation? >> >> >> It is a 1-off erratum workaround that is only needed on fewer than five >> models/revisions of a mips64 based SoC family. As such, creating a general >> purpose, architecture independent, framework is clearly not the proper >> approach. > > If this is just for maintaining coherency of the DMA operation inbetween, > then there is already a generic API for that, which the driver calls. > Adding the workaround into octeon_dma_map_sg() would be a way > to abstract the platform erratum from the driver. > Either I am bad at explaining things, or you are not reading what I wrote. These are two facts about the bug: 1) The bug has nothing to do with coherency management, so hacking something into dma_map* is the wrong thing to do. 2) The bug effects exactly one device, so hacking something into common code that is used by other devices is the wrong thing to do. Suggesting that we use an alternate set of facts, although an interesting exercise, doesn't get us closer to answering the question of which source code file should contain the code. This is one opinion about the bug: 1) The bug is in the device, not the "platform", so putting the workaround code in the driver for the device may be the cleanest approach. David Daney