linux-kernel - Re: [PATCH] usb: dwc3: Properly handle processing of pending events

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] [day] [month] [year] [list]

Message-ID: <50b82871-6308-21e1-77db-0c8793df196d@kernel.org>
Date:   Wed, 2 Aug 2023 17:34:22 +0300
From:   Roger Quadros <rogerq@...nel.org>
To:     Elson Roy Serrao <quic_eserrao@...cinc.com>,
        Thinh.Nguyen@...opsys.com, gregkh@...uxfoundation.org,
        felipe.balbi@...ux.intel.com, surong.pang@...soc.com
Cc:     linux-kernel@...r.kernel.org, linux-usb@...r.kernel.org,
        quic_wcheng@...cinc.com, quic_jackp@...cinc.com,
        stable@...r.kernel.org
Subject: Re: [PATCH] usb: dwc3: Properly handle processing of pending events



On 01/08/2023 22:26, Elson Roy Serrao wrote:
> If dwc3 is runtime suspended we defer processing the event buffer
> until resume, by setting the pending_events flag. Set this flag before
> triggering resume to avoid race with the runtime resume callback.
> 
> While handling the pending events, in addition to checking the event
> buffer we also need to process it. Handle this by explicitly calling
> dwc3_thread_interrupt(). Also balance the runtime pm get() operation
> that triggered this processing.
> 
> Cc: stable@...r.kernel.org
> Fixes: fc8bb91bc83e ("usb: dwc3: implement runtime PM")
> Signed-off-by: Elson Roy Serrao <quic_eserrao@...cinc.com>

Reviewed-by: Roger Quadros <rogerq@...nel.org>