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

Re: [PATCH for-4.18 v2] automation: fix race condition in adl-suspend test


  • To: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
  • From: Henry Wang <Henry.Wang@xxxxxxx>
  • Date: Tue, 31 Oct 2023 09:58:57 +0000
  • Accept-language: zh-CN, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=nCOvR4gGS4luczqOBj/y35twqeROydkSg4dXX8vuHaE=; b=es34FeXUEiuro0DSK7NEOyDvU2Y8NgEx6Sk2gHTE5hAsStnOKnPqVKraLKWGYSb+sDdN54+nj26Mh6wThdHYqP69ZS1HX2z2dK4c2NbHRIC41j1suqb9Z/N78Q2hC+eUuo0/zd0x9cbLCD3TjwDTlzcPyg+PqX/sf73+mRtWoJDQvmuju2UER4Ub6OKoiNk57UyB7wLsniXqFtaz8HobLR91Gaahxbk27U7LoIGH2yCg3Ofy856dNlp4RcUIriwzROcMQxHlWfadQG6qxtSUeW7fGI+oN90GZR0hTXpY21/EkOpIXKujrjuGN0JwcqLBr4ZAd+OwKz7jIGWIeKyCuw==
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=nCOvR4gGS4luczqOBj/y35twqeROydkSg4dXX8vuHaE=; b=oPMM99RdZQ+oAPZ0ZCSLF0/Rdudj+P/ELH6KZgi6FnLVCtvrUZhf5wYdVHcd0Efce85IPTt8An5Jpy5Thzo2yBbWtxuMjAO1LYAQTCEAk8jJlq3Fvar8MFgD+vV8EwD5uX9GLYnrDGCCJF+oMQLmNYSC956IUQnX4mchdGFzzYfqp1DNHGIGQnL2uBdMjvQ5uPmDrTmO544auberkfTBB0udL24lR7qAmo/CJ+DESoJpRSf2p8EKTS91K7N/dTlUZuUAxpk/ZZ0FKn9NwwZOZDJPtXyqQWu84ALBP81qG/0xHAVqAkY/01jigFjj4+hHqcfuGzt8a7FLGoi5zfir8A==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=CBR+HCopr5xmf0QCXyG6n77REUwnyaKe1YtAP+OBCRA+nC0dButkRNT4HRo4jQTn51QDBQYXK9vh8J33uv/wKop6pyvgrhg+O96aRq3Ji+J/6KXxG82UTBH5ezJUqWPKIAjGdTlV/NsByD6Jl16CJHNABAF2AGwj3dHZEf/dU+3FXZxfsDhqyjM1K/5Rq3chUO2/U2ifqNfz+u5prg/VO4V1lQcnoEAG6fZuGOLrcQL/rOpKhA3XIbzsyOrW2Oqd4wdiKtsqWy7Az+EhWsRxafNywV+XY9abgbtbEx+MOK5N7QThkVDz1a2FcpOwL+7wADySuS+Lx9kW8JOP/cplpw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iqB7Ueh0YEuoHXAzMmYlG+gErDfbBZg7pyvcsiP0YX9ifVZDW6AGHumf3fJbC5ANHiEyKhtLvT1g++hyuFV1/an1KCR9soC7kJumhZRIoOIrJSpJpZ+p8Z6nIh58XHoP+tY9Ci55/ksSa/EIFfhjcaJHdRpMJgDoxWIFl1YSDHEiO9rR+RtTse8frRhHdLoLNHjY43A9owt1tgE0zzYBNwTL+0fZJ3pGrsSlIdcdK1xr0dLSiueCyFOl9WQPuXezySBTLtVoARknRDRaPwjh0F/NlKLyRRV9ogqEzEyr0QTxhyQsTVC9ocXWf1MtJZ96oXmU8DzMNoqykvOegLcQCQ==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Doug Goldstein <cardoe@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • Delivery-date: Tue, 31 Oct 2023 09:59:16 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHaC6Bt3/qzioi050e1eTk9g/gvwrBjqm2A
  • Thread-topic: [PATCH for-4.18 v2] automation: fix race condition in adl-suspend test

Hi Marek,

> On Oct 31, 2023, at 10:16, Marek Marczykowski-Górecki 
> <marmarek@xxxxxxxxxxxxxxxxxxxxxx> wrote:
> 
> If system suspends too quickly, the message for the test controller to
> wake up the system may be not sent to the console before suspending.
> This will cause the test to timeout.
> 
> Fix this by calling sync on the console and waiting a bit after printing
> the message. The test controller then resumes the system 30s after the
> message, so as long as the delay + suspending takes less time it is
> okay.
> 
> Signed-off-by: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>

I think now that we branched, this patch should be committed to both staging 
and staging-4.18.
For staging 4.18:

Release-acked-by: Henry Wang <Henry.Wang@xxxxxxx>

I will remove the commit moratorium for staging once OSSTest does a successful 
sync between
staging and master. Thanks.

Kind regards,
Henry


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.