linux-kernel - Re: [Patch v8 0/6] 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: <8acd2694429af4f7205db7d7bb39eab6@dev.tdt.de>
Date:   Fri, 01 Dec 2023 14:08:23 +0100
From:   Florian Eckert <fe@....tdt.de>
To:     Lee Jones <lee@...nel.org>
Cc:     Eckert.Florian@...glemail.com, gregkh@...uxfoundation.org,
        jirislaby@...nel.org, pavel@....cz, kabel@...nel.org,
        u.kleine-koenig@...gutronix.de, m.brock@...mierlo.com,
        linux-kernel@...r.kernel.org, linux-serial@...r.kernel.org,
        linux-leds@...r.kernel.org
Subject: Re: [Patch v8 0/6] ledtrig-tty: add additional tty state evaluation



On 2023-12-01 11:40, Lee Jones wrote:
> On Thu, 09 Nov 2023 09:50:32 +0100, Florian Eckert wrote:
>> Changes in v8:
>> ==============
>> - As requested by greg k-h [6], I have send the patch 2/7 of this 
>> series
>>   about the memory leak also to stable.vger.kernel.org [7]. This has
>>   already received a 'Reviewed-by' from Uwe [8].
>> - As requested by Maarten, I have adopted his suggestion to invert the 
>> LED
>>   blink, so that I do not have to save the 'state' in the tty data
>>   struct [9].
>> 
>> [...]
> 
> Applied, thanks!
> 
> [1/6] tty: add new helper function tty_get_tiocm
>       commit: 5d11a4709f552fa139c2439fead05daeb064a6f4
> [2/6] leds: ledtrig-tty: free allocated ttyname buffer on deactivate
>       (no commit info)
> [3/6] leds: ledtrig-tty: change logging if get icount failed
>       (no commit info)
> [4/6] leds: ledtrig-tty: replace mutex with completion
>       (no commit info)
> [5/6] leds: ledtrig-tty: make rx tx activitate configurable
>       (no commit info)
> [6/6] leds: ledtrig-tty: add additional line state evaluation
>       (no commit info)

I think that was a mistake? Patchset v9 is the correct patchset [1]?

Thanks for applying v9 [1]

Best regards

Florian

Links:

[1] 
https://lore.kernel.org/all/170142724145.3350831.5316050550655479371.b4-ty@kernel.org/