Received-SPF: pass (google.com: domain of linux-kernel+bounces-63785-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <83e12387-e9b6-4a06-a38e-0f9888cd4e88@oracle.com>
Date: Tue, 13 Feb 2024 10:10:31 -0500
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH V2 0/3] flush workers on suspend
Content-Language: en-US
To: virtualization@lists.linux-foundation.org, linux-kernel@vger.kernel.org
Cc: "Michael S. Tsirkin" <mst@redhat.com>, Jason Wang <jasowang@redhat.com>,
        Si-Wei Liu <si-wei.liu@oracle.com>,
        Eugenio Perez Martin <eperezma@redhat.com>,
        Xie Yongji <xieyongji@bytedance.com>,
        Stefano Garzarella <sgarzare@redhat.com>
References: <1707758174-142161-1-git-send-email-steven.sistare@oracle.com>
From: Steven Sistare <steven.sistare@oracle.com>
Organization: Oracle Corporation
In-Reply-To: <1707758174-142161-1-git-send-email-steven.sistare@oracle.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: 
	=?utf-8?B?N0RBNnBoOG5tU1QyZXY2UCtTV05rOTJxaDRPV1EvSXNZOFBLbEt0ZnRiK01H?=
 =?utf-8?B?dlFKVzRaaDFDUEpFNDFGYzhOWXk5TG5TbzJNVXJtNmpBVkI5UGh4eFZPVS9O?=
 =?utf-8?B?T0taZG1CaHNrb210cWFPa0RGSGQyb2xONTczZ1R1YlZRT09GblZMS0h0RjZ5?=
 =?utf-8?B?NFFZZ3BxeVFYUEJDc0lneTMyN0l1cHk2NUxCS0ZOQ2F3L2tCclFFZ3VuNXY1?=
 =?utf-8?B?ek9DVGt4dXVMUjNWQlFiMWwyRjBzMFJDeS9FYnhOc2lxTUdFaHduMldvbDA4?=
 =?utf-8?B?WGhZRVVWN0J4aHRIUHBSaU1Wd2Y2bVVVcitJclJaUVRXZXdsVU1WVEt1bmY4?=
 =?utf-8?B?ZC9zR2REQTJrSjcxTWJEbXgrT2pJNU5pWGh3djVHQTR0ZHRVL3h3OXlWYVRn?=
 =?utf-8?B?aXZJN3RPczJBYmEzYmRHcUV3djlON214SzNqQ3ZPbjBLV0dwS2VzNThOOHVp?=
 =?utf-8?B?NHVMRkw2NFVwbzRxWTVLaGlXRjdWTUYrMlo1ay9uSjYyVDZIaHQ1dUFwQzlw?=
 =?utf-8?B?SHhCT2JkdkcvKzl2Z2lSVFJOcUhYYUtmaEJ4UmV4T01RanJqRStmQUFrWW40?=
 =?utf-8?B?ZmtyckRCaHpGTE9IMC95dTY3Y0ppdVNwdVQ0ek02cHNYRHRWZG0wQ3ZRdm4w?=
 =?utf-8?B?ekYwdTN3ZEJnNUx0VWJjSFNMZmlJNHNkcEUwalVlRldRazAyVUU1Rk5Kd3lI?=
 =?utf-8?B?WUZJWnFrTm5RVkhBY29wb0pOS1dyTlBaaldqNi9GU3hjV25HUHp4ek9mS0tu?=
 =?utf-8?B?eVl5a1dhZWFSOGdQakc0dWpIVFVOWHVBcTRhZG1SQzNCeVppVit4NEZ3SGZo?=
 =?utf-8?B?RFNZTlBtd2hHY2RmTlVINEY1UlNxQlk2TFliS1R4K2VjbFJQZVorVHVTUW8y?=
 =?utf-8?B?TklMaXZQTk5Pb3BRTjBRU1hrYWx6UEpLWmg3K1FyWDlZMmtKYWFMWTJxejZ5?=
 =?utf-8?B?eDJSREkvREhPNDhlNlE2OG9wUDI3L0I1SlZMZEl1RnU3UFd3ZDNLL0NQbFl2?=
 =?utf-8?B?ZVhVR2Q4R3JrcnJJNEY0bFNDQitCUzI2cGRoZ1hnWlBtTTBQNmVwK2RiQXU3?=
 =?utf-8?B?MTZrV3ErMXZjR2t4L0xYUDJQNUNIckJQTkxqc29xWWZwQ2pLcHVNU2U5a3Yy?=
 =?utf-8?B?dzBVRW45ZGVld1NTMEltcm5KZ2ZrT1lRc3l0SDVWKzJtQk5Vd2U3VTJZUlo2?=
 =?utf-8?B?YlFHSkxtR1d4Y1RPeFF6M3NRaGV0T0sxbEIxTmpiNEEvS1NHeTVLcGhWU25X?=
 =?utf-8?B?cWkzV01rMm4yS1IxY1VsNXl4ZGpKbktXVldJclhlZEtRLzlzeVFENC84RUpW?=
 =?utf-8?B?TTM4LzBZU3F3Z2kvMlIwVmUvV09vcW5FQ2xoZGpZdk04NTNSMElhS0FpM2xy?=
 =?utf-8?B?R0lYRnNBekhZL3pPY3hFU2U5S2ZWeUYzVkRuQUlmc3dqMFlPTHprMytPYlF4?=
 =?utf-8?B?UHlzVU9RQmpNYkVEOTh5K0JZVGVGcE9rSk40bTNGcGRuODZpanJNaWxSOW16?=
 =?utf-8?B?WXA2a05SKzBYMSs4dS9laEcxYWpYVzhnZjIyamxESjVlSHRXL05WRWdtR0RM?=
 =?utf-8?B?cFR2OFlKem9Oa2tqWCtXajFxNkNZUnBHTUdrak03NGVkdU94ZDM3ZkxEeGZJ?=
 =?utf-8?B?MHF4RlJuNzFpNjQyNzFZQ0g2K0dpckpwZDQwWTRNWUZ1YUJxU1QyZDlpbmg4?=
 =?utf-8?B?U3k1K2pjMm9Qamo2S3o1YUFYcktsYmt1VVNaL2RsMEhMZ2VMYWV4d3orZ3I1?=
 =?utf-8?B?THRRWXRuZGxaVnNCckNnWUxhWFJvYUdmYVlFS21NV0xGRFpaOHN4bHE5MTlD?=
 =?utf-8?B?Nkd5RGNSTXRYRGdrT25nMEd4YzFLUzRrdldPRml4Z1puNk9qdzlUWVpsTkM3?=
 =?utf-8?B?V2tYS01IRkI4OG9RWlpaTU54Y2N4R2l2U2RTdGxnTGtIc2RlRUw2a2kwS0FO?=
 =?utf-8?B?WkZhdmNiK2FuekdhZ3FRV2VjOHRSSEJ2TUhVbG1TZW9lTS9DVzNmSlRJdGFv?=
 =?utf-8?B?bDNFVy9wc21XRFJZVi9NdEpPejU4bFlScGhJanUvVjM4K1hQbWRlR3pOYUZv?=
 =?utf-8?B?RXhRa1JYZGJVSTRUNlZYUlFYNlMzWngvclU3TTZTbTZxRzlQYUU2MWFUQVp3?=
 =?utf-8?B?R0lGTTlVWkV1UlFMd2sxaTF4TmZSaUZrMkdtdGh3UEpxOEduVmFhamJtWTJs?=
 =?utf-8?B?Wnc9PQ==?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	zqjpVnrkVH81ppNLEanPWmKiviqaFWW5hjzZSjBX2xRPugQyLEDY1TeR3+pvWx/71GDAntGeuQ2vXn8kQwRp0ogDoal6c7Hh7GtHCuQ7svrjvH3tpxqG8uiW0rMFvVBqPViYPqVi+aefnP9zgp7AkLV8XtcbB98Pw3QviId8QO2e+k3dnwvyIdL6RGFGWCXHqQHrnrPULUqRjlOcDHmsC6Hfh6kWA8VjZOYIxo06VgJ77Kv6EvcC0xGzw4/LxOC2AfAOWaGlKC2p0yI9VlGbZj3Q43//i3g+JY0vJquLrawqgbfsOi/Dop1c0OVgdC0/t3E4UAvUa03R8wUW4o6RkIX5NAGXvx+JA+mBtDXbZkW2deP1FAcUwxvjJcFSrDDU3wW9taCavvvYH0MWJZ8lZI7kEEDLCWwGaPZBXoco8Pb8lD8WP+24EbpEUcsSAFps9TkS/AMK+ipWheQQJnSgVWothf1WRO6cRWvVyKn0PQbzWItT4MWF9UEKBJirLLumg2McEO3MKdGEQ0W46lkytDWCy4fhqY0l3oc2yoDamzw8zLgJTZLInYJTpGav4qVM7gHnAQGnY1FRTnUbpkB3wwyS+myzgKLdUbbG6I/tv6I=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e1545ffe-08b2-4108-e032-08dc2ca5eeef
X-MS-Exchange-CrossTenant-AuthSource: SA2PR10MB4684.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2024 15:10:37.1819
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Yb44RCGo91qm7Eruf0Wh0z92ccrjuDOAK2qctteusbWKinoDNe1viNN0l6BJk04BR2EF1WCrVOdRqtFdFTbXdXKEddW73zqWgXsh/HLUWck=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW6PR10MB7614
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-02-13_08,2024-02-12_03,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 bulkscore=0 suspectscore=0
 phishscore=0 adultscore=0 spamscore=0 mlxlogscore=556 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2311290000
 definitions=main-2402130119
X-Proofpoint-ORIG-GUID: -FHOKLaX_FbR_b3DK7GV1RA5jSPyz2c7
X-Proofpoint-GUID: -FHOKLaX_FbR_b3DK7GV1RA5jSPyz2c7

On 2/12/2024 12:16 PM, Steve Sistare wrote:
> Flush to guarantee no workers are running when suspend returns,
> for vdpa, vpa_sim, and vduse.  (mlx5 already does so, via the path
> mlx5_vdpa_suspend -> unregister_link_notifier -> flush_workqueue.)

Changes in V2:
  - renamed "vduse: suspend" (was vduse: flush workers on suspend)
  - call vhost_dev_flush unconditionally in "vhost-vdpa: flush workers on suspend"

- Steve

> Steve Sistare (3):
>   vhost-vdpa: flush workers on suspend
>   vduse: suspend
>   vdpa_sim: flush workers on suspend
> 
>  drivers/vdpa/vdpa_sim/vdpa_sim.c   | 13 +++++++++++++
>  drivers/vdpa/vdpa_user/vduse_dev.c | 24 ++++++++++++++++++++++++
>  drivers/vhost/vdpa.c               |  3 +++
>  3 files changed, 40 insertions(+)
>