Return-path: Received: from youngberry.canonical.com ([91.189.89.112]:38748 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756371AbeAIBQm (ORCPT ); Mon, 8 Jan 2018 20:16:42 -0500 Received: from mail-wm0-f72.google.com ([74.125.82.72]) by youngberry.canonical.com with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.76) (envelope-from ) id 1eYiWu-0006iA-RJ for linux-wireless@vger.kernel.org; Tue, 09 Jan 2018 01:16:40 +0000 Received: by mail-wm0-f72.google.com with SMTP id w74so4492778wmf.0 for ; Mon, 08 Jan 2018 17:16:40 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: <20171110025415.7854-1-drake@endlessm.com> <87y3na7eei.fsf@qca.qualcomm.com> <87bmk5hx9x.fsf@kamboji.qca.qualcomm.com> <87bmi45yim.fsf@kamboji.qca.qualcomm.com> From: AceLan Kao Date: Tue, 9 Jan 2018 09:16:39 +0800 Message-ID: (sfid-20180109_022011_602284_2EBA2F36) Subject: Re: [v2] ath9k: add MSI support To: Daniel Drake Cc: Kalle Valo , Russell Hu , "linux-wireless@vger.kernel.org" , Ryan Hsu , Robert Chang , Aeolus Yang , ath9k-devel , "linux@endlessm.com" , "rafael.j.wysocki@intel.com" , "andy@infradead.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi Kalle, I'm happy to see Russel's patch can be merged, and I'm willing to rebase and merge those quirks into one commit and submit it again. Although the solution is not perfect, but it's nice to have. Thanks. Best regards, AceLan Kao. 2018-01-09 7:07 GMT+08:00 Daniel Drake : > On Mon, Jan 8, 2018 at 6:24 AM, Kalle Valo wrote: >> (Adding AceLan) >> >> Daniel Drake writes: >> >>> On Wed, Nov 15, 2017 at 7:38 AM, Daniel Drake wrote: >>>> On Tue, Nov 14, 2017 at 8:15 PM, Kalle Valo wrote: >>>>>> Can't be fixed in firmware, but it would be good to have confirmation >>>>>> of the hardware behavivour, and maybe some other solution is possible? >>>>>> Are you following this up within Qualcomm? >>>>> >>>>> No time to do that right now, sorry. >>>> >>>> I got several autoresponders from people on this thread from Qualcomm >>>> Taiwan. Would it be useful for us to drop off a sample of the affected >>>> product at your Taipei or Hsinchu office so that you can investigate >>>> further? >>> >>> Ping - how can we collaborate on this? >> >> Are you asking me? While looking at my todo list for this year I doubt I >> can find time to help with the MSI implementation or bugfixing. > > So far you are the only Qualcomm person to reply to the many mails I > have written on this topic, so I appreciate the response. I have sunk > many hours into this unfortunate situation so I'd really appreciate if > you could point me to someone at Qualcomm who can provide a response. > I am willing to continue doing the hard work, but I do need some > Qualcomm help in getting past brick walls. > >> But my plan is that first I would apply Russel's patch which makes it >> possible to enable MSI with a module parameter: >> >> https://patchwork.kernel.org/patch/9999249/ > > This isn't enough to fix many of the systems that are affected by this > issue. You add the parameter, enable it, and MSI support totally fails > to deliver any interrupts. Pasting again from earlier: > > I have tested your patch on Acer Aspire ES1-432. It does not work - I > still can't connect to wifi. > /proc/interrupts shows that no MSI interrupts are delivered, the counters are 0. > > lspci -vv shows: > Capabilities: [50] MSI: Enable+ Count=1/4 Maskable+ 64bit+ > Address: 00000000fee0f00c Data: 4142 > Masking: 0000000e Pending: 00000000 > > So MSI is enabled and the vector number is 0x42 (decimal 66). > However my kernel log is now totally spammed with: > do_IRQ: 0.64 No irq handler for vector > > My assumption here is that the ath9k hardware implementation of MSI is > buggy, and it is therefore corrupting the MSI vector number by zeroing > out the lower 2 bits (e.g. 66 -> 64). > > It would be very useful if Qualcomm could confirm if this behaviour is > really true. > > For more info please see: > https://marc.info/?l=linux-pci&m=150238260826803&w=2 > https://marc.info/?t=150631283200001&r=1&w=2 > https://marc.info/?l=linux-pci&m=150831581725596&w=2 > > Thanks, > Daniel