Re: [PATCH regression 4.0 - 4.1] tools perf: Fix build breakage if prefix= is specified

2015-06-19 Thread Lukas Wunner
Hi Jiri, Hi David, On Fri, Jun 19, 2015 at 12:17:06AM +0200, Jiri Olsa wrote: it had effect.. prefix is passed to build framework throught .config-detected file as prefix_SQ variable, which is then referenced in perf Build makefiles You're right, thanks for the clarification. The underlying

Re: [PATCH regression 4.0 - 4.1] tools perf: Fix build breakage if prefix= is specified

2015-06-18 Thread Lukas Wunner
Hi Jirka, On Thu, Jun 18, 2015 at 09:29:32PM +0200, Jiri Olsa wrote: On Thu, Jun 18, 2015 at 01:00:32PM +0200, Lukas Wunner wrote: Invoking Makefile.perf with prefix= breaks the build since Makefile.perf hands that variable down to Makefile.build where it overrides prefix

Re: [PATCH regression 4.0 - 4.1] tools perf: Fix build breakage if prefix= is specified

2015-06-18 Thread Lukas Wunner
Hi David, On Thu, Jun 18, 2015 at 02:26:25PM -0600, David Ahern wrote: It worked for me last week with OL6. I created a standalone perf rpm with 4.1-rc6; it builds just fine with _prefix (rpm variable) set to /usr: [...] %global perf_make \ make -s -C tools/perf V=1 WERROR=0 NO_LIBUNWIND=1

Re: [PATCH 1/6] scripts/kernel-doc: Replacing highlights hash by an array

2015-09-13 Thread Lukas Wunner
Hi, On Sun, Sep 13, 2015 at 02:36:47PM -0600, Jonathan Corbet wrote: > On Mon, 7 Sep 2015 17:01:59 -0300 > Danilo Cesar Lemes de Paula wrote: > > The "highlight" code is very sensible to the order of the hash keys, > > but the order of the keys cannot be predicted.

[PATCH 0/1] async: export current_is_async()

2015-11-19 Thread Lukas Wunner
Hi Tejun, when you introduced current_is_async() with 84b233adcca3, was it a deliberate decision not to export it? All other non-static functions in async.c are exported as well. I'm asking because I would like to use it in i915.ko. Thanks, Lukas Lukas Wunner (1): async: export

[PATCH 1/1] async: export current_is_async()

2015-11-19 Thread Lukas Wunner
Introduced by 84b233adcca3 ("workqueue: implement current_is_async()"). Cc: Tejun Heo <t...@kernel.org> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- kernel/async.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/async.c b/kernel/async.c index 4c3773c..d2edd

Re: [tip:x86/urgent] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-09 Thread Lukas Wunner
On Thu, Jun 09, 2016 at 08:48:03AM +0200, Ingo Molnar wrote: > * Lukas Wunner <lu...@wunner.de> wrote: > > On Wed, Jun 08, 2016 at 11:56:14AM -0700, Yinghai Lu wrote: > > > How do you know that sec is valid ? > > > How about on the system that have one bridge that

Re: [tip:x86/urgent] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-08 Thread Lukas Wunner
On Wed, Jun 08, 2016 at 11:56:14AM -0700, Yinghai Lu wrote: > On Wed, Jun 8, 2016 at 7:23 AM, tip-bot for Lukas Wunner <tip...@zytor.com> > wrote: > > --- a/arch/x86/kernel/early-quirks.c > > +++ b/arch/x86/kernel/early-quirks.c > ... > > Extend bus scan range

Re: [PATCH v2 3/3] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-12 Thread Lukas Wunner
On Sun, Jun 12, 2016 at 01:49:02PM +0200, Ingo Molnar wrote: > * Lukas Wunner <lu...@wunner.de> wrote: > > The solution is to reset the card on boot by writing to a reset bit in > > its mmio space. This must be done as an early quirk and not as a plain > > vanilla PCI q

Re: [tip:x86/urgent] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-09 Thread Lukas Wunner
On Thu, Jun 09, 2016 at 09:37:53AM -0700, Yinghai Lu wrote: > On Thu, Jun 9, 2016 at 4:04 AM, Lukas Wunner <lu...@wunner.de> wrote: > > --- a/arch/x86/kernel/early-quirks.c > > +++ b/arch/x86/kernel/early-quirks.c > > @@ -747,7 +747,8 @@ static int __init check_

Re: [tip:x86/urgent] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-10 Thread Lukas Wunner
On Fri, Jun 10, 2016 at 01:58:45PM +0200, Ingo Molnar wrote: > * Yinghai Lu <ying...@kernel.org> wrote: > > On 6/9/16, Lukas Wunner <lu...@wunner.de> wrote: > > > Well, the PCI core would also scan such a bus twice AFAICS. > > > And the performance penalt

Re: [tip:x86/urgent] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-10 Thread Lukas Wunner
On Fri, Jun 10, 2016 at 02:59:57PM +0200, Ingo Molnar wrote: > * Lukas Wunner <lu...@wunner.de> wrote: > > On Fri, Jun 10, 2016 at 01:58:45PM +0200, Ingo Molnar wrote: > > > * Yinghai Lu <ying...@kernel.org> wrote: > > > > On 6/9/16, Lukas Wunner <lu

[PATCH v2 3/3] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-12 Thread Lukas Wunner
a.redhat.com/show_bug.cgi?id=1149632#c5 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1279130 Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1332732 Tested-by: Konstantin Simanov <k.sima...@stlk.ru># [MacBookPro8,1] Tested-by: Lukas Wunner <lu...@wunner.de>

[PATCH v2 2/3] x86/quirks: Reintroduce scanning of secondary buses

2016-06-12 Thread Lukas Wunner
g> Cc: Peter Zijlstra <pet...@infradead.org> Cc: Thomas Gleixner <t...@linutronix.de> Cc: linux-...@vger.kernel.org Signed-off-by: Lukas Wunner <lu...@wunner.de> --- v2: - Validate secondary bus number (Yinghai Lu) - Move scanning of secondary buses to separate patch to al

[PATCH v2 1/3] x86/quirks: Apply nvidia_bugs quirk only on root bus

2016-06-12 Thread Lukas Wunner
> Cc: Peter Zijlstra <pet...@infradead.org> Cc: Thomas Gleixner <t...@linutronix.de> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- v2: - Move amendment of nvidia_bugs quirk to separate patch to allow for easier reviewing, bisecting and reverting (Bjorn Helgaas) arch

[PATCH v2 0/3] Reset Apple AirPort card

2016-06-12 Thread Lukas Wunner
) - Remove line breaks in read/write_pci_config_16() (Ingo Molnar) Lukas Wunner (3): x86/quirks: Apply nvidia_bugs quirk only on root bus x86/quirks: Reintroduce scanning of secondary buses x86/quirks: Add early quirk to reset Apple AirPort card arch/x86/kernel/early-quirks.c | 98

Re: [tip:x86/urgent] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-12 Thread Lukas Wunner
On Fri, Jun 10, 2016 at 03:08:02PM -0500, Bjorn Helgaas wrote: > On Fri, Jun 10, 2016 at 03:59:04PM +0200, Lukas Wunner wrote: > > On Fri, Jun 10, 2016 at 02:59:57PM +0200, Ingo Molnar wrote: > > > So I'd lean towards lower complexity, but since this is essentially > > >

[PATCH] x86: Add early quirk to reset Apple AirPort card

2016-05-28 Thread Lukas Wunner
k> Cc: x...@kernel.org Cc: sta...@vger.kernel.org Tested-by: Konstantin Simanov <k.sima...@stlk.ru># [MacBookPro8,1] Tested-by: Lukas Wunner <lu...@wunner.de># [MacBookPro9,1] Tested-by: Bryan Paradis <bryan.para...@gmail.com> # [MacBookPro9,2]

Re: [PATCH v2 00/13] Runtime PM for Thunderbolt on Macs

2016-06-15 Thread Lukas Wunner
00, Andreas Noever wrote: > > > > On Fri, May 13, 2016 at 1:15 PM, Lukas Wunner <lu...@wunner.de> wrote: > > > > > This series powers Thunderbolt controllers on Macs down when > > > > > nothing is plugged in, saving 1.7 W on machines with a Light Rid

Re: [PATCH v2 02/10] driver core: Functional dependencies tracking support

2016-06-17 Thread Lukas Wunner
On Fri, Jun 17, 2016 at 02:54:56PM +0200, Rafael J. Wysocki wrote: > On Fri, Jun 17, 2016 at 12:36 PM, Lukas Wunner <lu...@wunner.de> wrote: > > On Fri, Jun 17, 2016 at 08:26:52AM +0200, Marek Szyprowski wrote: > > > From: "Rafael J. Wysocki" <rafael.

Re: [PATCH v2 02/10] driver core: Functional dependencies tracking support

2016-06-17 Thread Lukas Wunner
Hi Marek, On Fri, Jun 17, 2016 at 08:26:52AM +0200, Marek Szyprowski wrote: > From: "Rafael J. Wysocki" > > Currently, there is a problem with handling cases where functional > dependencies between devices are involved. > > What I mean by a "functional dependency"

Re: [PATCH v2 00/16] Common Dell SMBIOS API

2016-02-08 Thread Lukas Wunner
Hi, On Mon, Feb 08, 2016 at 11:20:14AM -0800, Darren Hart wrote: > My only major concern is module load order dependencies. Inter-module > dependencies are frowned upon with good reason, the kernel load ordering is > non-deterministic and it's possible, for example, for dell-laptop to fail to >

Re: linux-next: build failure after merge of the drm-misc tree

2016-02-10 Thread Lukas Wunner
[cc += Rafael J. Wysocki, linux-acpi] Hi Stephen, On Wed, Feb 10, 2016 at 12:24:51PM +1100, Stephen Rothwell wrote: > Hi all, > > After merging the drm-misc tree, today's linux-next build (arm > multi_v7_defconfig) failed like this: > > In file included from

Re: linux-next: build failure after merge of the drm-misc tree

2016-02-10 Thread Lukas Wunner
Hi, On Wed, Feb 10, 2016 at 09:41:38AM +0100, Lukas Wunner wrote: > On Wed, Feb 10, 2016 at 12:24:51PM +1100, Stephen Rothwell wrote: > > Hi all, > > > > After merging the drm-misc tree, today's linux-next build (arm > > multi_v7_defconfig) failed like this: >

Re: Enable CONFIG_VGA_SWITCHEROO cause lspci on MSI GS60-070XCN stuck

2016-02-25 Thread Lukas Wunner
Hi, On Wed, Feb 24, 2016 at 11:32:15PM +0800, Drunkard Zhang wrote: > I may hit a bug, when enabled CONFIG_VGA_SWITCHEROO, run lspci command > on MSI GS60-070XCN will stuck, it eats 100% CPU of one core, and > CPU/memory allocation on this system fails sometimes. Sounds like a deadlock. Does

Re: [PATCH 2/2] nouveau: use new vga_switcheroo power domain.

2016-03-15 Thread Lukas Wunner
Hi Dave, On Thu, Mar 10, 2016 at 08:04:26AM +1000, Dave Airlie wrote: > On 10 March 2016 at 00:40, Lukas Wunner <lu...@wunner.de> wrote: > > On Wed, Mar 09, 2016 at 04:14:05PM +1000, Dave Airlie wrote: > >> From: Dave Airlie <airl...@redhat.com> > >

Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines.

2016-03-15 Thread Lukas Wunner
Hi Mika, On Mon, Mar 14, 2016 at 11:43:35AM +0200, Mika Westerberg wrote: > On Mon, Mar 14, 2016 at 12:19:14PM +1000, Dave Airlie wrote: > > On 11 March 2016 at 23:45, Rafael J. Wysocki wrote: > > > On Friday, March 11, 2016 12:58:15 PM Mika Westerberg wrote: > > >> On Thu,

Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines.

2016-03-09 Thread Lukas Wunner
Hi, On Wed, Mar 09, 2016 at 11:52:33AM -0500, Alex Deucher wrote: > On Wed, Mar 9, 2016 at 9:33 AM, Lukas Wunner <lu...@wunner.de> wrote: > > On Wed, Mar 09, 2016 at 04:14:04PM +1000, Dave Airlie wrote: > >> From: Dave Airlie <airl...@redhat.com> > >> >

Re: Thunderbolt 3 (Skylake / Alpine Ridge) hotplug

2016-03-19 Thread Lukas Wunner
Hi Jack, On Tue, Mar 15, 2016 at 12:29:09PM -0500, Bjorn Helgaas wrote: > On Fri, Feb 26, 2016 at 07:10:03PM +1100, Jack Coulter wrote: > > I'm attempting to use the Thunderbolt 3 (which has a USB Type-C > > connector) port on my laptop, a Dell XPS 15 (9550). The external device > > I'm

Re: [PATCH 2/2] nouveau: use new vga_switcheroo power domain.

2016-03-09 Thread Lukas Wunner
Hi Dave, On Wed, Mar 09, 2016 at 04:14:05PM +1000, Dave Airlie wrote: > From: Dave Airlie > > This fixes GPU auto powerdown on the Lenovo W541, > since we advertise Windows 2013 to the ACPI layer. > > Signed-off-by: Dave Airlie > --- >

Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines.

2016-03-09 Thread Lukas Wunner
Hi Dave, On Wed, Mar 09, 2016 at 04:14:04PM +1000, Dave Airlie wrote: > From: Dave Airlie > > Windows 10 seems to have standardised power control for the > optimus/powerxpress laptops using PR3 power resource hooks. What happened to the Optimus DSM, does this still work? If

[PATCH 1/3] PCI: Add Thunderbolt device IDs

2016-03-20 Thread Lukas Wunner
Gen 1 and 2 chips use the same ID for NHI, bridges and switch. Gen 3 chips and onward use a distinct ID for the NHI. No functional change intended. Cc: Andreas Noever <andreas.noe...@gmail.com> Signed-off-by: Lukas Wunner <lu...@wunner.de> Acked-by: Bjorn Helgaas <bhel

[PATCH 2/3] thunderbolt: Fix typos and magic number

2016-03-20 Thread Lukas Wunner
Fix typo in tb_cfg_print_error() message. Fix bytecount in struct tb_drom_entry_port comment. Replace magic number in tb_switch_alloc(). Rename tb_sw_set_unpplugged() and TB_CAL_IECS. ^ ^ Cc: Andreas Noever <andreas.noe...@gmail.com> Signed-off-by:

[PATCH 3/3] thunderbolt: Support 1st gen Light Ridge controller

2016-03-20 Thread Lukas Wunner
s using this chip. Cc: Andreas Noever <andreas.noe...@gmail.com> Tested-by: Lukas Wunner <lu...@wunner.de> [MacBookPro9,1] Tested-by: William Brown <will...@blackhats.net.au> [MacBookPro8,2] Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/pci/quirks.c |

Re: [PATCH v2 4/6] ACPI / osi: Fix default _OSI(Darwin) support

2016-04-29 Thread Lukas Wunner
Hi Lv Zheng, On Fri, Apr 29, 2016 at 02:07:53AM +, Zheng, Lv wrote: > I just sent UPDATE of PATCH 4/6 and PATCH 6/6 to the mailing list with this > corrected. > I was hoping they could update patchwork content so that the Bugzilla > reporters might use the updated patches for confirmation. >

Re: next-20160517 - lockdep splat in pcie code

2016-05-17 Thread Lukas Wunner
Hi, On Tue, May 17, 2016 at 02:37:42PM -0500, Bjorn Helgaas wrote: > [+cc Lukas, Mika] > On Tue, May 17, 2016 at 02:36:02PM -0400, Valdis Kletnieks wrote: > > Seen during boot on next-20160517. This apparently sneaked into the tree > > sometime after -0502 (probably after -0512 but I can't prove

Re: [PATCH] thunderbolt: Add support for INTEL_FALCON_RIDGE_2C controller

2016-07-13 Thread Lukas Wunner
On Wed, Jul 13, 2016 at 12:17:33AM +0200, Andreas Noever wrote: > Are thunderbolt controllers always installed directly below the root > port? In theory there could be more bridges in between (a candidate > for such a topology would be the mac pro which has 3 controllers). Hm, good point. I

Re: [PATCH] thunderbolt: Add support for INTEL_FALCON_RIDGE_2C controller

2016-07-12 Thread Lukas Wunner
On Tue, Jul 12, 2016 at 11:13:24PM +0200, Lukas Wunner wrote: > On Tue, Jul 12, 2016 at 09:38:51PM +0200, Xavier Gnata wrote: > > Add support to INTEL_FALCON_RIDGE_2C controller and corresponding quirk to > > support suspend/resume. > > Tested against 4.7 master on a

Re: [PATCH] thunderbolt: Add support for INTEL_FALCON_RIDGE_2C controller

2016-07-12 Thread Lukas Wunner
On Tue, Jul 12, 2016 at 09:38:51PM +0200, Xavier Gnata wrote: > Add support to INTEL_FALCON_RIDGE_2C controller and corresponding quirk to > support suspend/resume. > Tested against 4.7 master on a MacBook Air 11" 2015 Nice, thanks for doing this. See below for some comments. > > --- >

Re: [PATCH v5 5/8] thunderbolt: Networking state machine

2016-07-28 Thread Lukas Wunner
On Thu, Jul 28, 2016 at 11:15:18AM +0300, Amir Levy wrote: > +static void nhi_handle_notification_msg(struct tbt_nhi_ctxt *nhi_ctxt, > + const u8 *msg) > +{ > + struct port_net_dev *port; > + u8 port_num; > + > +#define INTER_DOMAIN_LINK_SHIFT 0 >

Re: [PATCH v2 02/10] driver core: Functional dependencies tracking support

2016-07-28 Thread Lukas Wunner
On Thu, Jul 28, 2016 at 02:30:31AM +0200, Rafael J. Wysocki wrote: > On Monday, July 25, 2016 12:48:32 AM Lukas Wunner wrote: > > On Thu, Jul 21, 2016 at 02:25:15AM +0200, Rafael J. Wysocki wrote: > > > On Thursday, July 21, 2016 01:25:53 AM Lukas Wunner wrote: > > &

Re: [PATCH v2 02/10] driver core: Functional dependencies tracking support

2016-07-24 Thread Lukas Wunner
On Thu, Jul 21, 2016 at 02:25:15AM +0200, Rafael J. Wysocki wrote: > On Thursday, July 21, 2016 01:25:53 AM Lukas Wunner wrote: > > On Thu, Jul 21, 2016 at 12:51:31AM +0200, Rafael J. Wysocki wrote: > > > On Wednesday, July 20, 2016 05:23:40 PM Lukas Wunner wrote: > >

Re: [PATCH v4 5/7] thunderbolt: Networking state machine

2016-07-24 Thread Lukas Wunner
On Mon, Jul 18, 2016 at 01:00:38PM +0300, Amir Levy wrote: > + const unique_id_be proto_uuid = APPLE_THUNDERBOLT_IP_PROTOCOL_UUID; > + > + if (memcmp(proto_uuid, hdr->apple_tbt_ip_proto_uuid, > +sizeof(proto_uuid)) != 0) { You may want to use the uuid_be data type provided

[PATCH 5/6] efi: Assign Apple device properties

2016-07-27 Thread Lukas Wunner
ined devices. With Thunderbolt 2 they started to describe three device levels behind host controllers in the namespace. This grossly inflates the ACPI table and still fails if the user daisy-chained more than three devices. Cc: Matt Fleming <m...@codeblueprint.co.uk> Cc: Andreas Noeve

[PATCH 6/6] thunderbolt: Use Device ROM retrieved from EFI

2016-07-27 Thread Lukas Wunner
<andreas.noe...@gmail.com> Tested-by: Lukas Wunner <lu...@wunner.de> [MacBookPro9,1] Tested-by: Pierre Moreau <pierre.mor...@free.fr> [MacBookPro11,3] Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/thunderbolt/Kconfig | 1 + drivers/t

[PATCH 0/6] Apple device properties

2016-07-27 Thread Lukas Wunner
config so that they can be called with efi_call_early(). Would a patch to add those be entertained? Right now we only offer LocateHandle and HandleProtocol, which is somewhat cumbersome and needs more code as the setup_pci() functions show. Thanks, Lukas Lukas Wunner (6): efi: Retrieve Apple device p

[PATCH 4/6] driver core: Don't leak secondary fwnode on device removal

2016-07-27 Thread Lukas Wunner
reas Noever <andreas.noe...@gmail.com> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/base/core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/base/core.c b/drivers/base/core.c index 0a8bdad..70c5be5 100644 --- a/drivers/base/core.c +++ b/drivers/base/core.c @

[PATCH 3/6] efi: Add device path parser

2016-07-27 Thread Lukas Wunner
(and get_device_by_efi_path() itself is exported). The dependency on CONFIG_ACPI is needed for acpi_match_device_ids(). It can be removed if an empty inline stub is added for that function. Cc: Matt Fleming <m...@codeblueprint.co.uk> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/

[PATCH 1/6] efi: Retrieve Apple device properties

2016-07-27 Thread Lukas Wunner
s the property struct is freed but the name and value allocations are not. Cc: rever...@put.as Cc: grub-de...@gnu.org Cc: Matt Fleming <m...@codeblueprint.co.uk> Cc: Andreas Noever <andreas.noe...@gmail.com> Tested-by: Lukas Wunner <lu...@wunner.de> [MacBookPro9,1] Tested-by: Pierr

[PATCH 2/6] ACPI / bus: Make acpi_get_first_physical_node() public

2016-07-27 Thread Lukas Wunner
sical_node() public. Cc: Aleksey Makarov <aleksey.maka...@linaro.org> Cc: Rafael J. Wysocki <rafael.j.wyso...@intel.com> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/acpi/internal.h | 1 - include/linux/acpi.h| 7 +++ 2 files changed, 7 insertions(+), 1 delet

Re: [Intel-gfx] [PATCH 0197/1285] Replace numeric parameter like 0444 with macro

2016-08-02 Thread Lukas Wunner
On Tue, Aug 02, 2016 at 02:37:37PM +0300, Ville Syrjälä wrote: > On Tue, Aug 02, 2016 at 06:48:47PM +0800, Baole Ni wrote: > > I find that the developers often just specified the numeric value > > when calling a macro which is defined with a parameter for access > > permission. > > As we know,

Re: [PATCH 1/2] thunderbolt: Fix resume quirk for Falcon Ridge 4C.

2016-08-03 Thread Lukas Wunner
rs. > > Signed-off-by: Andreas Noever <andreas.noe...@gmail.com> FWIW, this is Reviewed-by: Lukas Wunner <lu...@wunner.de> I also tested it successfully on Light Ridge. I'm sending a follow-up patch separately so that Falcon Ridge chips are no longer declared unsupported. Th

[PATCH] thunderbolt: Don't declare Falcon Ridge unsupported

2016-08-03 Thread Lukas Wunner
Falcon Ridge 4C has been supported by the driver from the beginning, Falcon Ridge 2C support was just added. Don't irritate users with a warning declaring the opposite. Cc: Andreas Noever <andreas.noe...@gmail.com> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/thunderb

Re: [PATCH v2 02/10] driver core: Functional dependencies tracking support

2016-07-20 Thread Lukas Wunner
On Thu, Jul 21, 2016 at 12:51:31AM +0200, Rafael J. Wysocki wrote: > On Wednesday, July 20, 2016 05:23:40 PM Lukas Wunner wrote: > > On Wed, Jul 20, 2016 at 02:52:42PM +0200, Rafael J. Wysocki wrote: > > > On Wednesday, July 20, 2016 08:24:50 AM Lukas Wunner wrote: > >

Re: [PATCH v2 02/10] driver core: Functional dependencies tracking support

2016-07-20 Thread Lukas Wunner
On Wed, Jul 20, 2016 at 02:33:18AM +0200, Rafael J. Wysocki wrote: > On Friday, June 17, 2016 04:07:38 PM Lukas Wunner wrote: > > On Fri, Jun 17, 2016 at 02:54:56PM +0200, Rafael J. Wysocki wrote: > > > On Fri, Jun 17, 2016 at 12:36 PM, Lukas Wunner <lu...@wunner.de> wrote

Re: [PATCH v2 02/10] driver core: Functional dependencies tracking support

2016-07-20 Thread Lukas Wunner
On Wed, Jul 20, 2016 at 02:52:42PM +0200, Rafael J. Wysocki wrote: > On Wednesday, July 20, 2016 08:24:50 AM Lukas Wunner wrote: > > On Wed, Jul 20, 2016 at 02:33:18AM +0200, Rafael J. Wysocki wrote: > > > On Friday, June 17, 2016 04:07:38 PM Lukas Wunner wrote: > > > &

Re: kmem_cache_alloc fail with unable to handle paging request after pci hotplug remove.

2016-07-05 Thread Lukas Wunner
On Tue, Jul 05, 2016 at 11:00:21AM +0800, AceLan Kao wrote: > These are logs from my machine. > > *** Before plug-in the USB key > > u@u-XPS-13-9xxx:~$ sudo lspci - -s 00:1c.0 > 00:1c.0 PCI bridge: Intel Corporation Device 9d10 (rev f1) (prog-if 00 > [Normal decode]) [...] >

Re: kmem_cache_alloc fail with unable to handle paging request after pci hotplug remove.

2016-07-04 Thread Lukas Wunner
On Mon, Jul 04, 2016 at 06:04:42PM +0300, Mathias Nyman wrote: > On 04.07.2016 17:25, Rafael J. Wysocki wrote: > > On Mon, Jul 4, 2016 at 4:26 PM, Mathias Nyman > > wrote: > > > AceLan Kao can get his DELL XPS 13 laptop to hang by plugging/un-plugging > > > a USB

Re: [PATCH 1/6] efi: Retrieve Apple device properties

2016-08-05 Thread Lukas Wunner
On Thu, Aug 04, 2016 at 04:13:45PM +0100, Matt Fleming wrote: > On Thu, 28 Jul, at 02:25:41AM, Lukas Wunner wrote: > > diff --git a/arch/x86/boot/compressed/eboot.c > > b/arch/x86/boot/compressed/eboot.c > > index ff574da..7262ee4 100644 > > --- a/arch/x86/boot/compre

Re: x86/PCI: Scan all functions during probing

2016-08-09 Thread Lukas Wunner
On Tue, Aug 09, 2016 at 01:22:30PM +0200, Thomas Gleixner wrote: > From: Benedikt Spranger > > PCI and PCIBIOS probing only scans devices at function number 0/8/16/... > Subdevices (e.g. multiqueue) have function numbers which are not a > multiple of 8. > > Simple

Re: [PATCH 0/6] Apple device properties

2016-08-09 Thread Lukas Wunner
On Thu, Aug 04, 2016 at 03:57:10PM +0100, Matt Fleming wrote: > On Thu, 28 Jul, at 02:25:41AM, Lukas Wunner wrote: > > Specifically, is the following okay: > > efi_early->call((unsigned long)sys_table->boottime->locate_protocol, ...) > > This probably isn't goi

Re: [PATCH] PM / runtime: Asynchronous "idle" in pm_runtime_allow()

2016-06-29 Thread Lukas Wunner
On Wed, Jun 29, 2016 at 10:02:42AM -0400, Alan Stern wrote: > On Wed, 29 Jun 2016, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > Arjan reports that it takes a relatively long time to enable runtime > > PM for multiple devices at system startup, because

Re: [PATCH v2 3/3] x86/quirks: Add early quirk to reset Apple AirPort card

2016-06-28 Thread Lukas Wunner
Hi Ingo, On Sun, Jun 12, 2016 at 01:13:38PM +0200, Lukas Wunner wrote: > I've just posted v2, taking into account all of Bjorn's suggestions: > https://lkml.org/lkml/2016/6/12/65 > > @Ingo Molnar: I noticed that you've removed the line breaks in > read/write_pci_config_16(

Re: [PATCH 0/6] Apple device properties

2016-08-15 Thread Lukas Wunner
On Mon, Aug 15, 2016 at 12:54:14PM +0100, Matt Fleming wrote: > On Tue, 09 Aug, at 03:38:16PM, Lukas Wunner wrote: > > @@ -208,7 +201,10 @@ struct efi_config { > > __pure const struct efi_config *__efi_early(void); > > > >

Re: [PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-06 Thread Lukas Wunner
On Mon, Feb 06, 2017 at 04:15:02PM -0600, Bjorn Helgaas wrote: > On Mon, Feb 06, 2017 at 10:20:41PM +0100, Lukas Wunner wrote: > > On Mon, Feb 06, 2017 at 11:54:05AM -0600, Bjorn Helgaas wrote: > > > What is the hotplug event that causes generation of this wakeup event? > &

Re: [PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-06 Thread Lukas Wunner
On Mon, Feb 06, 2017 at 10:52:12PM +0100, Rafael J. Wysocki wrote: > On Monday, February 06, 2017 10:20:41 PM Lukas Wunner wrote: > > On Mon, Feb 06, 2017 at 11:54:05AM -0600, Bjorn Helgaas wrote: > > > On Mon, Feb 06, 2017 at 06:54:37AM +0100, Lukas Wunner wrote: > > >

Re: pciehp is broken from 4.10-rc1

2017-02-04 Thread Lukas Wunner
On Sat, Feb 04, 2017 at 08:22:59PM -0800, Yinghai Lu wrote: > On Sat, Feb 4, 2017 at 3:34 PM, Lukas Wunner <lu...@wunner.de> wrote: > > On Sat, Feb 04, 2017 at 01:44:34PM -0800, Yinghai Lu wrote: > >> On Sat, Feb 4, 2017 at 10:56 AM, Lukas Wunner <lu...@wunner.de>

Re: [GIT PULL] PCI fixes for v4.10

2017-02-08 Thread Lukas Wunner
On Wed, Feb 08, 2017 at 01:22:56PM -0600, Bjorn Helgaas wrote: > Bjorn Helgaas (1): > Revert "PCI: pciehp: Add runtime PM support for PCIe hotplug ports" What's the rationale for reverting this? You've received patches to fix the issue on both affected machines, so a revert seems

Re: [PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-08 Thread Lukas Wunner
On Wed, Feb 08, 2017 at 11:57:36AM -0600, Bjorn Helgaas wrote: > On Tue, Feb 07, 2017 at 07:21:01AM +0100, Lukas Wunner wrote: > > On Mon, Feb 06, 2017 at 04:15:02PM -0600, Bjorn Helgaas wrote: > > > On Mon, Feb 06, 2017 at 10:20:41PM +0100, Lukas Wunner wrote: > > > &

Re: pciehp is broken from 4.10-rc1

2017-02-06 Thread Lukas Wunner
On Mon, Feb 06, 2017 at 12:37:06PM +0200, Mika Westerberg wrote: > On Sun, Feb 05, 2017 at 08:34:54AM +0100, Lukas Wunner wrote: > > @Mika, Rafael: Are you aware of Skylake machines with unreliable link > > training, or perhaps errata of Skylake chips related to link training >

Re: [PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-06 Thread Lukas Wunner
On Mon, Feb 06, 2017 at 11:54:05AM -0600, Bjorn Helgaas wrote: > On Mon, Feb 06, 2017 at 06:54:37AM +0100, Lukas Wunner wrote: > > Since commit 68db9bc81436 ("PCI: pciehp: Add runtime PM support for PCIe > > hotplug ports") we runtime suspend a hotplug port to D3hot

Re: [PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-07 Thread Lukas Wunner
On Tue, Feb 07, 2017 at 05:04:45PM +0100, Rafael J. Wysocki wrote: > On Tuesday, February 07, 2017 07:21:01 AM Lukas Wunner wrote: > > On Mon, Feb 06, 2017 at 04:15:02PM -0600, Bjorn Helgaas wrote: > > > On Mon, Feb 06, 2017 at 10:20:41PM +0100, Lukas Wunner wrote: > >

[PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-05 Thread Lukas Wunner
kernel.org> Cc: Rafael J. Wysocki <rafael.j.wyso...@intel.com> Cc: Mika Westerberg <mika.westerb...@linux.intel.com> Cc: Bjorn Helgaas <bhelg...@google.com> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- v1-> v2: Move check for is_hotplug_bridge from pci_finish_runtime_su

Re: [PATCH v5 1/8] PCI: Recognize Thunderbolt devices

2017-02-05 Thread Lukas Wunner
On Sun, Jan 29, 2017 at 01:26:16AM +0100, Lukas Wunner wrote: > On Sat, Jan 28, 2017 at 03:52:08PM -0600, Bjorn Helgaas wrote: > > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: > > > --- a/drivers/pci/probe.c > > > +++ b/drivers/pci/probe.c > &

Re: [git pull] drm fixes for 4.10-rc6 (just missed rc5 tagging :-)

2017-01-25 Thread Lukas Wunner
On Wed, Jan 25, 2017 at 01:54:32PM +0100, Daniel Vetter wrote: > On Wed, Jan 25, 2017 at 06:10:57PM +0900, Michel Dänzer wrote: > > On 25/01/17 05:33 PM, Markus Trippelsdorf wrote: > > > On 2017.01.23 at 09:38 +1000, Dave Airlie wrote: > > >>

Re: [PATCH v5 1/8] PCI: Recognize Thunderbolt devices

2017-01-28 Thread Lukas Wunner
On Sat, Jan 28, 2017 at 03:52:08PM -0600, Bjorn Helgaas wrote: > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: > > We're about to allow runtime PM on Thunderbolt ports in > > pci_bridge_d3_possible() and unblock runtime PM for Thunderbolt host &g

Re: [PATCH v5 1/8] PCI: Recognize Thunderbolt devices

2017-02-12 Thread Lukas Wunner
On Fri, Feb 10, 2017 at 11:42:50AM -0600, Bjorn Helgaas wrote: > On Sun, Jan 29, 2017 at 01:26:16AM +0100, Lukas Wunner wrote: > > On Sat, Jan 28, 2017 at 03:52:08PM -0600, Bjorn Helgaas wrote: > > > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: >

Re: [PATCH v2] PCI: pciehp: Don't enable PME on runtime suspend

2017-02-12 Thread Lukas Wunner
On Thu, Feb 09, 2017 at 05:32:54AM +0100, Lukas Wunner wrote: > On Wed, Feb 08, 2017 at 11:57:36AM -0600, Bjorn Helgaas wrote: > > On Tue, Feb 07, 2017 at 07:21:01AM +0100, Lukas Wunner wrote: > > > On Mon, Feb 06, 2017 at 04:15:02PM -0600, Bjorn Helgaas wrote: > > > &

Re: [GIT PULL] PCI fixes for v4.10

2017-02-12 Thread Lukas Wunner
On Fri, Feb 10, 2017 at 06:39:16PM -0800, Yinghai Lu wrote: > On Thu, Feb 9, 2017 at 12:11 PM, Bjorn Helgaas <helg...@kernel.org> wrote: > > On Thu, Feb 09, 2017 at 09:09:50AM -0600, Bjorn Helgaas wrote: > > > On Thu, Feb 09, 2017 at 05:06:48AM +0100, Lukas Wun

Re: [PATCH v5 7/8] thunderbolt: Power down controller when idle

2017-02-12 Thread Lukas Wunner
On Sat, Jan 28, 2017 at 05:32:37PM -0600, Bjorn Helgaas wrote: > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: > > A Thunderbolt controller appears to the OS as a set of virtual devices: > > One upstream bridge, multiple downstream bridges and one NHI (Native >

Re: [PATCH v5 3/8] PCI: Don't block runtime PM for Thunderbolt host hotplug ports

2017-02-12 Thread Lukas Wunner
On Fri, Feb 10, 2017 at 12:39:12PM -0600, Bjorn Helgaas wrote: > On Sun, Jan 15, 2017 at 09:03:45PM +0100, Lukas Wunner wrote: > > Hotplug ports generally block their parents from suspending to D3hot as > > otherwise their interrupts couldn't be delivered. > > This sounds

Re: [PATCH v2] PCI: pciehp: Optimize PCIe root resume time

2017-01-18 Thread Lukas Wunner
On Thu, Jan 19, 2017 at 02:57:26AM +, Shankar, Vaibhav wrote: > > From: Lukas Wunner [mailto:lu...@wunner.de] > > Sent: Tuesday, January 17, 2017 9:14 PM > > On Wed, Jan 18, 2017 at 01:32:13AM +, Shankar, Vaibhav wrote: > > > > From: Bjorn Helgaas [mailto:he

Re: How to improve PCI bug tracking?

2017-01-18 Thread Lukas Wunner
On Wed, Jan 18, 2017 at 06:29:15PM -0500, Sinan Kaya wrote: > On 1/18/2017 5:53 PM, Bjorn Helgaas wrote: > > I want to improve PCI bug tracking. It feels like our bugs don't get > > the attention they need, and I'm soliciting ideas for how to improve > > this. > > > > Current situation: > > > >

[PATCH v5 8/8] thunderbolt: Runtime suspend NHI when idle

2017-01-15 Thread Lukas Wunner
to the macOS driver. Cc: Andreas Noever <andreas.noe...@gmail.com> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/thunderbolt/nhi.c| 2 ++ drivers/thunderbolt/power.c | 9 + drivers/thunderbolt/switch.c | 9 + drivers/thunderbolt/tb.c | 13 ++

[PATCH v5 1/8] PCI: Recognize Thunderbolt devices

2017-01-15 Thread Lukas Wunner
.l...@intel.com> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- drivers/pci/pci.h | 2 ++ drivers/pci/probe.c | 34 ++ include/linux/pci.h | 1 + 3 files changed, 37 insertions(+) diff --git a/drivers/pci/pci.h b/drivers/pci/pci.h index cb17db242

[PATCH v5 0/8] Runtime PM for Thunderbolt on Macs

2017-01-15 Thread Lukas Wunner
/fetched on GitHub: https://github.com/l1k/linux/commits/thunderbolt_runpm_v5 Thanks, Lukas Chen Yu (1): PM / sleep: Define constant for direct_complete Lukas Wunner (7): PCI: Recognize Thunderbolt devices PCI: Allow runtime PM on Thunderbolt ports PCI: Don't block runtime PM

Re: [PATCH v2] PCI: pciehp: Optimize PCIe root resume time

2017-01-17 Thread Lukas Wunner
On Wed, Jan 18, 2017 at 01:32:13AM +, Shankar, Vaibhav wrote: > > From: Bjorn Helgaas [mailto:helg...@kernel.org] > > Sent: Wednesday, January 11, 2017 10:37 AM > > On Mon, Dec 12, 2016 at 04:32:25PM -0800, Vaibhav Shankar wrote: > > > On Apollolake platforms, PCIe rootport takes a long time

Re: [PATCH][RFC v2] PCI: Workaround to enable poweroff on Mac Pro 11

2016-08-19 Thread Lukas Wunner
On Fri, Aug 19, 2016 at 01:44:52AM +0800, Chen Yu wrote: > People reported that they can not do a poweroff nor a > suspend to memory on their Mac Pro 11. After some investigations > it was found that, once the PCI bridge :00:1c.0 reassigns its > mm windows([mem 0x7fa0-0x7fbf] and >

Re: [PATCH 0/6] Apple device properties

2016-08-22 Thread Lukas Wunner
On Thu, Aug 18, 2016 at 09:34:33PM +0100, Matt Fleming wrote: > On Mon, 15 Aug, at 06:13:58PM, Lukas Wunner wrote: > > But I would like to understand the "cannot jump through pointers at > > runtime" argument because the binary code looks to me like it should > >

[PATCH 1/2] x86/efi: Optimize away setup_gop32/64 if unused

2016-08-22 Thread Lukas Wunner
re, 1406 after CONFIG_X86_64 && CONFIG_EFI_MIXED:2201 before and after Signed-off-by: Lukas Wunner <lu...@wunner.de> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> --- arch/x86/include/asm/efi.h | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/

[PATCH 2/2] x86/efi: Allow invocation of arbitrary boot services

2016-08-22 Thread Lukas Wunner
%ds:0x30(%rax), %edi ; free_pool (32 bit) 0x153 jmp $0x0f5 Size of eboot.o text section: CONFIG_X86_32: 6464 before, 6318 after CONFIG_X86_64 && !CONFIG_EFI_MIXED:7670 before, 7573 after CONFIG_X86_64 && CONFIG_EFI_MIXED:7670 before, 8

Re: [PATCH 1/2] x86/efi: Optimize away setup_gop32/64 if unused

2016-09-06 Thread Lukas Wunner
On Mon, Sep 05, 2016 at 01:37:25PM +0100, Matt Fleming wrote: > On Mon, 22 Aug, at 12:01:21PM, Lukas Wunner wrote: > > Eliminate the 64 bit code path when compiling for x86_32, and vice-versa > > the 32 bit code path when compiling for x86_64 with mixed mode disabled. > > &

[PATCH v2] x86/efi: Optimize away setup_gop32/64 if unused

2016-09-06 Thread Lukas Wunner
!CONFIG_EFI_MIXED:2201 before, 1406 after CONFIG_X86_64 && CONFIG_EFI_MIXED:2201 before and after Signed-off-by: Lukas Wunner <lu...@wunner.de> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> --- arch/x86/include/asm/efi.h | 13 +++-- 1 file changed, 11 insertions

[PATCH v2 3/4] x86/efi: Retrieve and assign Apple device properties

2016-09-07 Thread Lukas Wunner
@put.as Cc: grub-de...@gnu.org Cc: Matt Fleming <m...@codeblueprint.co.uk> Cc: Andreas Noever <andreas.noe...@gmail.com> Tested-by: Lukas Wunner <lu...@wunner.de> [MacBookPro9,1] Tested-by: Pierre Moreau <pierre.mor...@free.fr> [MacBookPro11,3] Signed-off-by: Lukas Wunner

[PATCH v2 4/4] thunderbolt: Use Device ROM retrieved from EFI

2016-09-07 Thread Lukas Wunner
it into the DROM, calculates the CRC and submits the result as a device property. Cc: rever...@put.as Cc: Andreas Noever <andreas.noe...@gmail.com> Tested-by: Lukas Wunner <lu...@wunner.de> [MacBookPro9,1] Tested-by: Pierre Moreau <pierre.mor...@free.fr> [MacBookPro11,3] Signed-off-by

[PATCH v2 1/4] ACPI / bus: Make acpi_get_first_physical_node() public

2016-09-07 Thread Lukas Wunner
sical_node() public. Cc: Aleksey Makarov <aleksey.maka...@linaro.org> Signed-off-by: Lukas Wunner <lu...@wunner.de> Acked-by: Rafael J. Wysocki <rafael.j.wyso...@intel.com> --- drivers/acpi/internal.h | 1 - include/linux/acpi.h| 7 +++ 2 files changed, 7 insertions(+

[PATCH v2 2/4] efi: Add device path parser

2016-09-07 Thread Lukas Wunner
(and get_device_by_efi_path() itself is exported). The dependency on CONFIG_ACPI is needed for acpi_match_device_ids(). It can be removed if an empty inline stub is added for that function. Cc: Matt Fleming <m...@codeblueprint.co.uk> Signed-off-by: Lukas Wunner <lu...@wunner.de> --- driv

[PATCH v2 0/4] Apple device properties

2016-09-07 Thread Lukas Wunner
: https://github.com/l1k/linux/commits/apple_properties_v2 Thanks, Lukas Lukas Wunner (4): ACPI / bus: Make acpi_get_first_physical_node() public efi: Add device path parser x86/efi: Retrieve and assign Apple device properties thunderbolt: Use Device ROM retrieved from EFI Documentation

Re: [PATCH 4/6] driver core: Don't leak secondary fwnode on device removal

2016-08-30 Thread Lukas Wunner
On Wed, Aug 17, 2016 at 02:38:50AM +0200, Rafael J. Wysocki wrote: > On Thursday, July 28, 2016 02:25:41 AM Lukas Wunner wrote: > > If device_add_property_set() is called for a device, a secondary fwnode > > is allocated and assigned to the device but currently not freed once

Re: [RFC/RFT][PATCH v2 2/7] driver core: Functional dependencies tracking support

2016-09-11 Thread Lukas Wunner
On Sun, Sep 11, 2016 at 03:40:58PM +0200, Lukas Wunner wrote: > On Thu, Sep 08, 2016 at 11:27:45PM +0200, Rafael J. Wysocki wrote: > > +/** > > + * device_is_dependent - Check if one device depends on another one > > + * @dev: Device to check dependencies for. > > +

  1   2   3   4   5   >