linux-kernel - Re: [PATCH v3 0/3] software node: Introduce software_node_find_by

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 19 Aug 2019 13:39:33 +0300
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Heikki Krogerus <heikki.krogerus@...ux.intel.com>
Cc:     "Rafael J. Wysocki" <rafael@...nel.org>,
        Andy Shevchenko <andy@...radead.org>,
        Hans de Goede <hdegoede@...hat.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Darren Hart <dvhart@...radead.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 0/3] software node: Introduce software_node_find_by_name()

On Mon, Aug 19, 2019 at 1:08 PM Heikki Krogerus
<heikki.krogerus@...ux.intel.com> wrote:
>
> Hi,
>
> There was still one bug spotted by Andy in v2. The role switch node
> was not removed in case of an error (patch 2/3). It is now fixed.

It would be nice to have immutable branch for these changes. There is
at least some other activity regard to intel_cht_int33fe driver.

> The cover letter from v2:
>
> This is the second version of this series where I'm introducing that
> helper.
>
> Hans and Andy! Because of the changes I made to patch 2/3, I'm not
> carrying your reviewed-by tags in it. I would appreciate if you
> could take another look at that patch.
>
> I added a note to the kernel-doc comment in patch 1/3 that the caller
> of the helper function needs to release the ref count after use as
> proposed by Andy.
>
> In patch 2/3, since we have to now modify the role switch descriptor,
> I'm filling the structure in stack memory and removing the constant
> static version. The content of the descriptor is copied during switch
> registration in any case, so we don't need to store it in the driver.
>
> I also noticed a bug in 2/3. I never properly destroyed the software
> node when the mux was removed. That leak is now also fixed.
>
> thanks,
>
> Heikki Krogerus (3):
>   software node: Add software_node_find_by_name()
>   usb: roles: intel_xhci: Supplying software node for the role mux
>   platform/x86: intel_cht_int33fe: Use new API to gain access to the
>     role switch
>
>  drivers/base/swnode.c                         | 37 ++++++++++++
>  drivers/platform/x86/intel_cht_int33fe.c      | 57 ++++---------------
>  .../usb/roles/intel-xhci-usb-role-switch.c    | 27 ++++++---
>  include/linux/property.h                      |  4 ++
>  4 files changed, 71 insertions(+), 54 deletions(-)
>
> --
> 2.23.0.rc1
>


-- 
With Best Regards,
Andy Shevchenko