From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com ([192.55.52.93]:34582 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751798AbcLLNuF (ORCPT ); Mon, 12 Dec 2016 08:50:05 -0500 Subject: Re: [RFT PATCH 1/1] xhci: free xhci virtual devices with leaf nodes first To: Guenter Roeck References: <582DC88C.5040308@linux.intel.com> <1479903867-561-1-git-send-email-mathias.nyman@linux.intel.com> <20161128202455.GC10709@roeck-us.net> <583EBAE4.6000905@linux.intel.com> <20161209212845.GA16317@roeck-us.net> Cc: linux-usb@vger.kernel.org, stable@vger.kernel.org From: Mathias Nyman Message-ID: <584EAB42.3000609@linux.intel.com> Date: Mon, 12 Dec 2016 15:50:58 +0200 MIME-Version: 1.0 In-Reply-To: <20161209212845.GA16317@roeck-us.net> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: stable-owner@vger.kernel.org List-ID: On 09.12.2016 23:28, Guenter Roeck wrote: > On Wed, Nov 30, 2016 at 01:41:24PM +0200, Mathias Nyman wrote: >> On 28.11.2016 22:24, Guenter Roeck wrote: >>> On Wed, Nov 23, 2016 at 02:24:27PM +0200, Mathias Nyman wrote: >>>> the tt_info provided by a HS hub might be in use to by a child device >>>> Make sure we free the devices in the correct order. >>>> >>>> This is needed in special cases such as when xhci controller is >>>> reset when resuming from hibernate, and all virt_devices are freed. >>>> >>>> Also free the virt_devices starting from max slot_id as children >>>> more commonly have higher slot_id than parent. >>>> >>>> CC: >>>> Signed-off-by: Mathias Nyman >>>> >>>> --- >>>> >>>> Guenter Roeck, does this work for you? >>>> >>> Yes, it does. >>> >>> Tested-by: Guenter Roeck >>> >>> Thanks, >>> Guenter >>> >> >> Thanks, I'll send it forward with proper Reported-by and Tested-by tags >> after 4.10-rc1 >> > Do you have this patch sitting in some branch, by any chance ? > I would like to pick it up if possible. > Now in git://git.kernel.org/pub/scm/linux/kernel/git/mnyman/xhci.git for-usb-linus Branch is not very well tested yet -Mathias