Re: [PATCH v4] usb: core: Solve race condition in anchor cleanup functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH v4] usb: core: Solve race condition in anchor cleanup functions
From: Eli Billauer <eli.billauer@xxxxxxxxx>
Date: Thu, 30 Jul 2020 18:14:15 +0300
Cc: gregkh@xxxxxxxxxxxxxxxxxxx, linux-usb@xxxxxxxxxxxxxxx, hdegoede@xxxxxxxxxx, oneukum@xxxxxxx
In-reply-to: <20200730144614.GB1604718@rowland.harvard.edu>
References: <20200730141836.16550-1-eli.billauer@gmail.com> <20200730144614.GB1604718@rowland.harvard.edu>
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.12) Gecko/20100907 Fedora/3.0.7-1.fc12 Thunderbird/3.0.7

On 30/07/20 17:46, Alan Stern wrote:

This loop ought to have a cpu_relax() somewhere, probably here.  Same
for the other two functions.  Otherwise this looks okay to me.

This function isn't so well documented. Does it go with an include file?asm/processor.h, for example? Or just add the call?


Thanks,
   Eli

Follow-Ups:
- Re: [PATCH v4] usb: core: Solve race condition in anchor cleanup functions
  - From: Alan Stern

References:
- [PATCH v4] usb: core: Solve race condition in anchor cleanup functions
  - From: eli . billauer
- Re: [PATCH v4] usb: core: Solve race condition in anchor cleanup functions
  - From: Alan Stern

Prev by Date: Re: [PATCH RFC 1/4] USB: UDC: Don't wipe deallocated memory
Next by Date: Re: [PATCH v4] usb: core: Solve race condition in anchor cleanup functions
Previous by thread: Re: [PATCH v4] usb: core: Solve race condition in anchor cleanup functions
Next by thread: Re: [PATCH v4] usb: core: Solve race condition in anchor cleanup functions
Index(es):
- Date
- Thread

[Index of Archives] [Linux Media] [Linux Input] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI] [Old Linux USB Devel Archive]

Powered by Linux