linux-kernel - Re: [Patch v9 0/4] ledtrig-tty: add additional tty state evaluation

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <2023112745-sector-revenge-f821@gregkh>
Date:   Mon, 27 Nov 2023 13:12:03 +0000
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Florian Eckert <fe@....tdt.de>
Cc:     Eckert.Florian@...glemail.com, pavel@....cz, lee@...nel.org,
        kabel@...nel.org, u.kleine-koenig@...gutronix.de,
        linux-leds@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [Patch v9 0/4] ledtrig-tty: add additional tty state evaluation

On Mon, Nov 27, 2023 at 12:03:07PM +0100, Florian Eckert wrote:
> Changes in v9:
> ==============
> - As requested, I have removed the memory leak fix patch from the previous
>   patch series v8 and have send this patch directly to the stable
>   mailinglist [10].
> - I also have removed the not needed wait for completion code path on
>   sysfs attr store and show, for the new configuration options, as
>   requested by greg k-h [11].
> - Patch v8 3/6 dropped, because the change is not correct as mentiond
>   by greg k-h [12] and this part of the code will be removed anyway
>   in this patch series.
> - Update naming of the function ledtrig_tty_waitforcompletion() to
>   ledtrig_tty_wait_for_completion() which match better with the API call
>   function wait_for_completion_timeout().
> 
> Thanks to all reviewers for the suggestions for improvement :+1: of this
> patch set.

Nice work, thanks for sticking with it:

Reviewed-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>