Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752761AbbKCICX (ORCPT ); Tue, 3 Nov 2015 03:02:23 -0500 Received: from mail-bl2on0081.outbound.protection.outlook.com ([65.55.169.81]:18944 "EHLO na01-bl2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752079AbbKCICV convert rfc822-to-8bit (ORCPT ); Tue, 3 Nov 2015 03:02:21 -0500 Authentication-Results: spf=softfail (sender IP is 66.35.236.227) smtp.mailfrom=altera.com; canb.auug.org.au; dkim=none (message not signed) header.d=none;canb.auug.org.au; dmarc=none action=none header.from=altera.com; Message-ID: <1446537761.13032.8.camel@leyfoon-vm> Subject: Re: linux-next: build failure after merge of the pci tree From: Ley Foon Tan To: Bjorn Helgaas CC: Stephen Rothwell , "linux-next@vger.kernel.org" , "linux-kernel@vger.kernel.org" Date: Tue, 3 Nov 2015 16:02:41 +0800 In-Reply-To: References: <20151102022307.774bde2c@canb.auug.org.au> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT X-Mailer: Evolution 3.8.4-0ubuntu1 MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11FD011;1:K2qaJi/BIDaD4boWXX4bnJRGXoPUrUbPkrJDll8gWByTnjL259ncD4HnmcbmyI3ZduWPSc1dB0wqJo9Q/3f8xpdqdlDfzsqdRfBuq5sh4f+239w9S7V6bEGabWZW9S1mee4CIXMhxKB7cSrMuN2Tg3A5NF3TaYLXKMv+sFZdMiWGAaICitqCYKH/KVqJsvLi6pH5V8oULmAZKNoY8TlrKe01bqqbahqK0prc1wApeHI0TcOK2uiV6TaOqo6cNsv7sLqCRxFR3tv3ugI03r6Bt1sA+3YPK39h8yeXi6UbT0PvNKDvaVcZNj2xTP9ilB0F4awjuVZuAuo1BDpm/K1oC5U5fe5/c25skij4HWLCgbJdOnlgsusCWki7XpKldjkZ X-Forefront-Antispam-Report: CIP:66.35.236.227;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(2980300002)(189002)(377454003)(199003)(76104003)(51914003)(24454002)(377424004)(87936001)(5820100001)(19580395003)(50226001)(19580405001)(92566002)(76176999)(50986999)(86362001)(33646002)(47776003)(5001960100002)(81156007)(97736004)(23676002)(33716001)(5007970100001)(5001920100001)(110136002)(6806005)(42186005)(5008740100001)(105596002)(103116003)(50466002)(106466001)(189998001)(2950100001)(11100500001)(7099028)(99106002);DIR:OUT;SFP:1101;SCL:1;SRVR:BL2PR03MB372;H:sj-itexedge03.altera.priv.altera.com;FPR:;SPF:SoftFail;PTR:InfoDomainNonexistent;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BL2PR03MB372;2:Hr0gHBjZIxNIuImvC61qbr4xW9kMm2BmV5gdV8UA5ayeuI2qeAt1r2Ja2B7oGxf6tTpiR+bpEY2JcUv0Ca7mdQF7Kwy/LyO+oo/zQE2PUog+SnHYfUoz4bElxSX8at3vYj0xdx5GQpEkdZIaJ1EzI+gi8XIKei9OqOcTBhXuwWc=;3:f503pzi04JGtDigMHZGSxbAlBaQS3nrAERJ6vbTHbNBV40SHpWyGkRWWPomil6VVsdC2lk8jjcEvUNs/L7npjl7no45ckY3R9mTM8kevqX26Y9OAfrYLT9u4dhamosj7RkwfgSc9W+I65Nw6Hbp8TCkBO6BTezp1GzCPzLpFX9Svo4FZgNhb+mFh6budppfd4jl/kwYUHjZ5FDZUYiuL+Yu4Vk7N9i2RTylLPsZsjg0=;25:tGHNv0iHJw52vdN2XWNPjlTmElBukfoYs7EuosxsRjM4VY8v7UkYbV78tpIvtC6g8Sy5l0mg7Mp1vw+GfHVOznwz3nJ8W6DBGmq1vq6+XaxFsjG0tu1rcgKJ1B2MvDrXxQoFhravt46sVjhzdr6RZ+FCds0vLFlWCy9yVuJ7sFkqYkHV0tmX4a/O4yB4IOPsR0gwQ8L1TZkYjd+gdoSjbaXZsmVWwW6jHo6/60ps6ePS4SG08dZ8hDi454bDtUxbd7HhTKUdfaYYLLsNrePWtg==;20:RUKvdgwMg2MrxScbcMiyAwGpuXICw+m8bE6JT9xuEvwytBD6vYaVZFZLy4ucCyepislZcVsTT2ONbnVVMgqpCaDXQjM+pw+5tKDMg5YVlRozZ9V/ARUJir2UWkNDFahq8H9QtAUakfIUy1JTVzjqvvn1V8iCmhtZVvAMujEKLk4= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BL2PR03MB372; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(80048183373757); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(520078)(8121501046)(10201501046)(3002001);SRVR:BL2PR03MB372;BCL:0;PCL:0;RULEID:;SRVR:BL2PR03MB372; X-Microsoft-Exchange-Diagnostics: 1;BL2PR03MB372;4:Rgn1rI4NunVoPiK+wXbWbr3DhNtZwABJWxEz1fZ/QyuOKVzXgqVA3kIv+AugETxSFSJEQw90JaRoc07Xsi4+S9DYjUEWFVpC/Bn+AjYQAKvUH/UznkFSfcoeII+GZ1GS69X0uMt+uv4qfubt0jQ51rX+PMkXUqEs1bG4WVUbC53LCa7HjNHkJ7DZEAsXSMcruX0M3ZZR120O1AJ9JgZakCKK9AL7g2OzPhJxFyTLXUtIVCMBrAjfhlVxh+E23Q3vsxa9oZQxgj+SteoEAoXkY3WkZXOA/WjFgRdO+oBgOZc+7lAOrZuJpEj6vq1EWhrCfOoNU9IQxXgL52a9bAX5crKxs+gUJVLev/m1fDcWPw5/+1OHDK7Y229E2VEWKRcY X-Forefront-PRVS: 0749DC2CE6 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCTDJQUjAzTUIzNzI7MjM6Yk1PektkNEd3MEhlTWxvYmpNRng1cnUycFZQ?= =?utf-8?B?M2J4UXZ5VXAwTDRrSWQ2bytWeWNQbEVjMnZnY2ZBTHFXTVQwU2dYTnRNWG9L?= =?utf-8?B?cWJZM1htbWZvUVNFWFBiVWl6NURHajlPeUJsUEQrMExoZGFiNndTL0w4M1Bl?= =?utf-8?B?cnlIa1pmQlBvWGwwMWlQN2d4VkMyaS9zZmk1dWRPOGNtMkoreDlxOTQrajlt?= =?utf-8?B?UW00c2VGL1UySTQ0VEZOQ3FLeDdabldjWDB1eEVHNnYvOTJDZUkwYjY3RDNW?= =?utf-8?B?ZytyOGlxNXVDUlBkcHZUZGt0MTdKRFpwRVJtTmw3TlJibENGRnA0S1ZhamJB?= =?utf-8?B?NlZwRnZGYXBHNzRZMHFhZ2pSRXdzK3BsK05rTjh1dU0xNzk1OXZYd25pZWIw?= =?utf-8?B?MFo3dVB0ZVRqQ2lGMjdhQmtkOVNOWU83YU1DRkt4ajJrNjFyYnVlbTBlL3Rl?= =?utf-8?B?UzhJRE1sOVcrQ09aT2IrMlBTN1R2K2RuOWxCODVpOVkxY29jcm9OcGdDeHRP?= =?utf-8?B?bUlHSGZSWkhrMFJQWXVGRGQrUGFMeXM0bXdwSkthSXF0Q1BWUlVOZkdubmcz?= =?utf-8?B?RnIwM0FlcjRCOUVUYmpjZmQ2MDNPK0N2RlB1bG9LdGlpbzFEQmJXeUJsYXhi?= =?utf-8?B?Wm5vWDJCbXhHVFljSmNmazVaZStWYzlIMDhXMkhKOVRjTzRlNmp1Y2d5eEJa?= =?utf-8?B?UDZrZGZNSE9SbkdiaWY0Y0pmWUpveXB5bVkrdWZwVjl0VXpHNnV1aklXU3ow?= =?utf-8?B?MS9xeHgrdysycEE2cDQ5YUpIdmYrRG95NkFXOThkdEE5MG52cWZoU2RaVHpq?= =?utf-8?B?WmpEOS9tcmdpVUlMUjZ1YjcxcmIrYSswMVpsTUxUTnJSZDVKaENOMFFhVzYr?= =?utf-8?B?ZEtWYzhKYm5wK213TWozYmlHQmRONDQ5a2JTUlNIUkFrclkxMlBiNFRMbUR5?= =?utf-8?B?WVd3VVJOeDJCbXp0Y1Y4bHJsMG1QWjZrNUpZRjMxVWxzWi9CY0dUWXhIK1pW?= =?utf-8?B?Y2pOZENpeE82bXI1MEQ1bHNsUEFxdXVCTTRXbFRDKy9MR0ZkNVlzUnBQTlow?= =?utf-8?B?QUpYZmxsSktSQ2lWR2pyZHVVcFZDODRIWnJIQXQySDJEdnVkOFBITE8ydlZQ?= =?utf-8?B?M2tCOEJXbTRXQUZveVRqNTRiOVp2dGN5ejl6QWFGNkNrOVhBcms2L2dtV0hX?= =?utf-8?B?VlkyY2FPSENPajQ5RXdRUlkzZ2c4YmI5RWxFcFJxNi8vcjFBNFZWenp1Q0E4?= =?utf-8?B?UkszNGxORitLMlVNNEhZZjJDdVkyeDNLVkV6cWh5OElBem1aNTRuNzlaS0tu?= =?utf-8?B?RHhJcm1RazdjcFBxSW03UE94bFFzdTNrZkZ4eFpuakdhV0R0Vk1kTFJocDJu?= =?utf-8?B?OC9GUHFQdWo2a3FkYWdlbVdiVXJ4d0wrOGlkRmtFS1ZRWmxNYVRISnMvekpS?= =?utf-8?B?VlAyMjRaY3JieWpDOTdjNlJ2MUdlZ2FZTDdYY29mNlJ3NWlKbUs1WFdTQkNZ?= =?utf-8?B?aWJBPT0=?= X-Microsoft-Exchange-Diagnostics: 1;BL2PR03MB372;5:bLBChusx3rS74oZEdLKkbYeGRmx/rgAvi4NVDoCYn8D4LX7+1FgbQZDP4MWyj8npStA7XLEFE6tlLLmlJ/P47Q1du4xfxnhILyw3F0rSJiyYnAfs/VCf/TpsVK0XN8GlUsCLyjr1JDViwgpHSfGBCQ==;24:/fX+xl6eVsTalKpsfOBfBEoxSb5yVOL5NcFOVSmS4EmsHq56ruTAb475Yq7pM6rM+WZt7p6V75s0C2i6zJRLAzdETxpnlH2Uoc9OVyjs2ag=;20:2PpK7sMjlbWcnQTWT2TW4OAtpZ72SjkqTlxEX9bRZQ1+YzalthjMoTPeToFRctnLuZoxwXDUIYAZTocYhips56Jjo9FNnIsEba/oqPwn9CVkfnSBu3zUKkiJhzUgI1sskb+PljK/L4RYhPuT31ov1eWVCuuxmNPGbeZ3mkrbdNw= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: altera.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Nov 2015 08:02:17.8700 (UTC) X-MS-Exchange-CrossTenant-Id: fbd72e03-d4a5-4110-adce-614d51f2077a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=fbd72e03-d4a5-4110-adce-614d51f2077a;Ip=[66.35.236.227];Helo=[sj-itexedge03.altera.priv.altera.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR03MB372 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2408 Lines: 71 On Isn, 2015-11-02 at 12:12 -0600, Bjorn Helgaas wrote: > On Sun, Nov 1, 2015 at 6:00 PM, Ley Foon Tan wrote: > > On Isn, 2015-11-02 at 02:23 +1100, Stephen Rothwell wrote: > >> Hi Bjorn, > >> > >> After merging the pci tree, today's linux-next build (powerpc > >> allyesconfig) failed like this: > >> > >> In file included from include/linux/kvm_host.h:20:0, > >> from arch/powerpc/kernel/asm-offsets.c:54: > >> include/linux/msi.h:174:21: fatal error: asm/msi.h: No such file or directory > >> > >> Caused by commit > >> > >> f19eafec930f ("PCI: altera: Add Altera PCIe MSI driver") > >> > >> I have disabled that driver for today: > >> > >> From: Stephen Rothwell > >> Date: Mon, 2 Nov 2015 02:12:54 +1100 > >> Subject: [PATCH] PCI: altera: disable MSI driver for now > >> > >> Signed-off-by: Stephen Rothwell > >> --- > >> drivers/pci/host/Kconfig | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig > >> index 2af0571356e8..7a31ebbc4379 100644 > >> --- a/drivers/pci/host/Kconfig > >> +++ b/drivers/pci/host/Kconfig > >> @@ -158,6 +158,7 @@ config PCIE_ALTERA > >> config PCIE_ALTERA_MSI > >> bool "Altera PCIe MSI feature" > >> depends on PCI_MSI > >> + depends on BROKEN > >> select PCI_MSI_IRQ_DOMAIN > >> help > >> Say Y here if you want PCIe MSI support for the Altera FPGA. > >> -- > >> 2.6.1 > >> > > Hi Bjorn > > > > This driver depends on the asm/msi.h and not all architectures include > > the asm/msi.h. > > We can fix this by adding "depends on ARM || NIOS2" to Kconfig, same as > > altera pcie driver. What do you think? > > I don't think your driver depends on asm/msi.h because ARM doesn't > have asm/msi.h either. > > I think the best thing is to do this: > > config PCIE_ALTERA_MSI > bool "Altera PCIe MSI feature" > depends on PCIE_ALTERA && PCI_MSI > > That way we don't have to repeat the constraints from PCIE_ALTERA. > This is the same strategy used by PCI_XGENE & PCI_XGENE_MSI. > Okay. Thanks for the fix. Regards Ley Foon -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/