Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Heinrich, On Wed, 16 Mar 2022 at 14:47, Heinrich Schuchardt wrote: > > > > Am 16. März 2022 20:23:37 MEZ schrieb Simon Glass : > >Hi Masami, > > > >On Wed, 16 Mar 2022 at 00:09, Masami Hiramatsu > > wrote: > >> > >> Hi Simon, > >> > >> 2022年3月16日(水) 12:13 Simon Glass : > >> > > >> > Hi Masami,

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Am 16. März 2022 20:23:37 MEZ schrieb Simon Glass : >Hi Masami, > >On Wed, 16 Mar 2022 at 00:09, Masami Hiramatsu > wrote: >> >> Hi Simon, >> >> 2022年3月16日(水) 12:13 Simon Glass : >> > >> > Hi Masami, >> > >> > On Tue, 15 Mar 2022 at 02:36, Masami Hiramatsu >> > wrote: >> > > >> > > Hi Simon, >>

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Masami, On Wed, 16 Mar 2022 at 00:09, Masami Hiramatsu wrote: > > Hi Simon, > > 2022年3月16日(水) 12:13 Simon Glass : > > > > Hi Masami, > > > > On Tue, 15 Mar 2022 at 02:36, Masami Hiramatsu > > wrote: > > > > > > Hi Simon, > > > > > > 2022年3月15日(火) 14:04 Simon Glass : > > > > > > > > Hi Masami,

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon, 2022年3月16日(水) 12:13 Simon Glass : > > Hi Masami, > > On Tue, 15 Mar 2022 at 02:36, Masami Hiramatsu > wrote: > > > > Hi Simon, > > > > 2022年3月15日(火) 14:04 Simon Glass : > > > > > > Hi Masami, > > > > > > On Mon, 14 Mar 2022 at 18:40, Masami Hiramatsu > > > wrote: > > > > > > > > Hi Sim

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Tue, Mar 15, 2022 at 09:13:15PM -0600, Simon Glass wrote: > Hi Masami, > > On Tue, 15 Mar 2022 at 02:36, Masami Hiramatsu > wrote: > > > > Hi Simon, > > > > 2022年3月15日(火) 14:04 Simon Glass : > > > > > > Hi Masami, > > > > > > On Mon, 14 Mar 2022 at 18:40, Masami Hiramatsu > > > wrote: > > > >

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Masami, On Tue, 15 Mar 2022 at 02:36, Masami Hiramatsu wrote: > > Hi Simon, > > 2022年3月15日(火) 14:04 Simon Glass : > > > > Hi Masami, > > > > On Mon, 14 Mar 2022 at 18:40, Masami Hiramatsu > > wrote: > > > > > > Hi Simon, > > > > > > 2022年3月15日(火) 3:24 Simon Glass : > > > > > > > > > > OK, wel

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon, 2022年3月15日(火) 14:04 Simon Glass : > > Hi Masami, > > On Mon, 14 Mar 2022 at 18:40, Masami Hiramatsu > wrote: > > > > Hi Simon, > > > > 2022年3月15日(火) 3:24 Simon Glass : > > > > > > > > OK, well 'reset by a user' presumably starts the board up and then > > > > > runs some code to do the u

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Masami, On Mon, 14 Mar 2022 at 18:40, Masami Hiramatsu wrote: > > Hi Simon, > > 2022年3月15日(火) 3:24 Simon Glass : > > > > > > OK, well 'reset by a user' presumably starts the board up and then > > > > runs some code to do the update in U-Boot? Is that right? If so, we > > > > just need to trigg

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Tue, Mar 15, 2022 at 09:40:34AM +0900, Masami Hiramatsu wrote: > Hi Simon, > > 2022年3月15日(火) 3:24 Simon Glass : > > > > > > OK, well 'reset by a user' presumably starts the board up and then > > > > runs some code to do the update in U-Boot? Is that right? If so, we > > > > just need to trigger

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon, 2022年3月15日(火) 3:24 Simon Glass : > > > > OK, well 'reset by a user' presumably starts the board up and then > > > runs some code to do the update in U-Boot? Is that right? If so, we > > > just need to trigger that update from the test. We don't need to test > > > the actual reset, at lea

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Masami, On Mon, 14 Mar 2022 at 01:35, Masami Hiramatsu wrote: > > Hi Simon, > > 2022年3月14日(月) 15:45 Simon Glass : > > > > Hi Takahiro, > > > > On Sun, 13 Mar 2022 at 20:43, AKASHI Takahiro > > wrote: > > > > > > On Sun, Mar 13, 2022 at 08:15:02PM -0600, Simon Glass wrote: > > > > Hi Takahiro,

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Mon, Mar 14, 2022 at 04:35:45PM +0900, Masami Hiramatsu wrote: > Hi Simon, > > 2022年3月14日(月) 15:45 Simon Glass : > > > > Hi Takahiro, > > > > On Sun, 13 Mar 2022 at 20:43, AKASHI Takahiro > > wrote: > > > > > > On Sun, Mar 13, 2022 at 08:15:02PM -0600, Simon Glass wrote: > > > > Hi Takahiro, >

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon, 2022年3月14日(月) 15:45 Simon Glass : > > Hi Takahiro, > > On Sun, 13 Mar 2022 at 20:43, AKASHI Takahiro > wrote: > > > > On Sun, Mar 13, 2022 at 08:15:02PM -0600, Simon Glass wrote: > > > Hi Takahiro, > > > > > > On Sun, 13 Mar 2022 at 19:08, AKASHI Takahiro > > > wrote: > > > > > > > > H

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Takahiro, On Sun, 13 Mar 2022 at 20:43, AKASHI Takahiro wrote: > > On Sun, Mar 13, 2022 at 08:15:02PM -0600, Simon Glass wrote: > > Hi Takahiro, > > > > On Sun, 13 Mar 2022 at 19:08, AKASHI Takahiro > > wrote: > > > > > > Hi Simon, > > > > > > On Fri, Mar 11, 2022 at 07:24:39PM -0700, Simon G

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Sun, Mar 13, 2022 at 08:15:02PM -0600, Simon Glass wrote: > Hi Takahiro, > > On Sun, 13 Mar 2022 at 19:08, AKASHI Takahiro > wrote: > > > > Hi Simon, > > > > On Fri, Mar 11, 2022 at 07:24:39PM -0700, Simon Glass wrote: > > > Hi Takahiro, > > > > > > On Fri, 18 Feb 2022 at 18:16, AKASHI Takahir

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Takahiro, On Sun, 13 Mar 2022 at 19:08, AKASHI Takahiro wrote: > > Hi Simon, > > On Fri, Mar 11, 2022 at 07:24:39PM -0700, Simon Glass wrote: > > Hi Takahiro, > > > > On Fri, 18 Feb 2022 at 18:16, AKASHI Takahiro > > wrote: > > > > > > On Fri, Feb 18, 2022 at 02:48:54PM +0100, Heinrich Schuch

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon, On Fri, Mar 11, 2022 at 07:24:39PM -0700, Simon Glass wrote: > Hi Takahiro, > > On Fri, 18 Feb 2022 at 18:16, AKASHI Takahiro > wrote: > > > > On Fri, Feb 18, 2022 at 02:48:54PM +0100, Heinrich Schuchardt wrote: > > > On 2/18/22 03:16, Masami Hiramatsu wrote: > > > > Hi Simon, > > > >

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Masami, On Sun, 13 Mar 2022 at 08:00, Masami Hiramatsu wrote: > > Hi Simon and Takahiro, > > I also would like to know how the UEFI is tested in the U-Boot. > It seems that we have lib/efi_selftest/ but it seems not unified to > 'ut' command. I think it would be good to add it there. There is

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon and Takahiro, I also would like to know how the UEFI is tested in the U-Boot. It seems that we have lib/efi_selftest/ but it seems not unified to 'ut' command. Should I expand efi_selftest for testing capsule update with reset? Thank you, 2022年3月12日(土) 11:24 Simon Glass : > > Hi Takahir

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Takahiro, On Fri, 18 Feb 2022 at 18:16, AKASHI Takahiro wrote: > > On Fri, Feb 18, 2022 at 02:48:54PM +0100, Heinrich Schuchardt wrote: > > On 2/18/22 03:16, Masami Hiramatsu wrote: > > > Hi Simon, > > > > > > Thank you for your reply. > > > > > > 2022年2月18日(金) 2:56 Simon Glass : > > > > > > >

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Heinrich, 2022年2月18日(金) 22:59 Heinrich Schuchardt : > > On 2/17/22 18:55, Simon Glass wrote: > > Hi Masami, > > > > On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu > > wrote: > >> > >> Hi Simon, > >> > >> Let me confirm your point. > >> So are you concerning the 'real' reset for the capsule upd

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Fri, Feb 18, 2022 at 02:48:54PM +0100, Heinrich Schuchardt wrote: > On 2/18/22 03:16, Masami Hiramatsu wrote: > > Hi Simon, > > > > Thank you for your reply. > > > > 2022年2月18日(金) 2:56 Simon Glass : > > > > > > > > Hi Masami, > > > > > > On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu > > >

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Thu, Feb 17, 2022 at 10:55:58AM -0700, Simon Glass wrote: > Hi Masami, > > On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu > wrote: > > > > Hi Simon, > > > > Let me confirm your point. > > So are you concerning the 'real' reset for the capsule update test > > case itself or this patch? > > > >

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Fri, Feb 18, 2022 at 02:59:05PM +0100, Heinrich Schuchardt wrote: > On 2/17/22 18:55, Simon Glass wrote: > > Hi Masami, > > > > On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu > > wrote: > > > > > > Hi Simon, > > > > > > Let me confirm your point. > > > So are you concerning the 'real' reset

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On 2/17/22 18:55, Simon Glass wrote: Hi Masami, On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu wrote: Hi Simon, Let me confirm your point. So are you concerning the 'real' reset for the capsule update test case itself or this patch? I'm actually learning how the test is working, so please h

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On 2/18/22 03:16, Masami Hiramatsu wrote: Hi Simon, Thank you for your reply. 2022年2月18日(金) 2:56 Simon Glass : Hi Masami, On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu wrote: Hi Simon, Let me confirm your point. So are you concerning the 'real' reset for the capsule update test case it

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon, Thank you for your reply. 2022年2月18日(金) 2:56 Simon Glass : > > Hi Masami, > > On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu > wrote: > > > > Hi Simon, > > > > Let me confirm your point. > > So are you concerning the 'real' reset for the capsule update test > > case itself or this pat

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Masami, On Wed, 16 Feb 2022 at 18:11, Masami Hiramatsu wrote: > > Hi Simon, > > Let me confirm your point. > So are you concerning the 'real' reset for the capsule update test > case itself or this patch? > > I'm actually learning how the test is working, so please help me to > understand how

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Simon, Let me confirm your point. So are you concerning the 'real' reset for the capsule update test case itself or this patch? I'm actually learning how the test is working, so please help me to understand how I can solve it. There are 3 environments to run the test, sandbox, Qemu, and a rea

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Heinrich, On Wed, 16 Feb 2022 at 10:50, Heinrich Schuchardt wrote: > > On 2/16/22 16:46, Tom Rini wrote: > > On Wed, Feb 16, 2022 at 04:32:40PM +0100, Heinrich Schuchardt wrote: > >> On 2/16/22 16:26, Simon Glass wrote: > >>> Hi Masami, > >>> > >>> On Tue, 15 Feb 2022 at 02:05, Masami Hiramats

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Wed, Feb 16, 2022 at 06:45:32PM +0100, Heinrich Schuchardt wrote: > On 2/16/22 16:46, Tom Rini wrote: > > On Wed, Feb 16, 2022 at 04:32:40PM +0100, Heinrich Schuchardt wrote: > > > On 2/16/22 16:26, Simon Glass wrote: > > > > Hi Masami, > > > > > > > > On Tue, 15 Feb 2022 at 02:05, Masami Hiram

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Heinrich, On Wed, 16 Feb 2022 at 08:37, Heinrich Schuchardt wrote: > > On 2/16/22 16:26, Simon Glass wrote: > > Hi Masami, > > > > On Tue, 15 Feb 2022 at 02:05, Masami Hiramatsu > > wrote: > >> > >> Since now the capsule_on_disk will restart the u-boot sandbox right > >> after the capsule upd

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On 2/16/22 16:46, Tom Rini wrote: On Wed, Feb 16, 2022 at 04:32:40PM +0100, Heinrich Schuchardt wrote: On 2/16/22 16:26, Simon Glass wrote: Hi Masami, On Tue, 15 Feb 2022 at 02:05, Masami Hiramatsu wrote: Since now the capsule_on_disk will restart the u-boot sandbox right after the capsule

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Wed, Feb 16, 2022 at 04:32:40PM +0100, Heinrich Schuchardt wrote: > On 2/16/22 16:26, Simon Glass wrote: > > Hi Masami, > > > > On Tue, 15 Feb 2022 at 02:05, Masami Hiramatsu > > wrote: > > > > > > Since now the capsule_on_disk will restart the u-boot sandbox right > > > after the capsule upd

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On 2/16/22 16:26, Simon Glass wrote: Hi Masami, On Tue, 15 Feb 2022 at 02:05, Masami Hiramatsu wrote: Since now the capsule_on_disk will restart the u-boot sandbox right after the capsule update, if CONFIG_EFI_CAPSULE_ON_DISK_EARLY=y, the boot with a new capsule file will repeat reboot sequen

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Masami, On Tue, 15 Feb 2022 at 02:05, Masami Hiramatsu wrote: > > Since now the capsule_on_disk will restart the u-boot sandbox right > after the capsule update, if CONFIG_EFI_CAPSULE_ON_DISK_EARLY=y, the > boot with a new capsule file will repeat reboot sequence. On the > other hand, if CONFI

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Takahiro, 2022年2月16日(水) 10:35 AKASHI Takahiro : > > On Tue, Feb 15, 2022 at 06:05:50PM +0900, Masami Hiramatsu wrote: > > Since now the capsule_on_disk will restart the u-boot sandbox right > > after the capsule update, if CONFIG_EFI_CAPSULE_ON_DISK_EARLY=y, the > > boot with a new capsule file

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On Tue, Feb 15, 2022 at 06:05:50PM +0900, Masami Hiramatsu wrote: > Since now the capsule_on_disk will restart the u-boot sandbox right > after the capsule update, if CONFIG_EFI_CAPSULE_ON_DISK_EARLY=y, the > boot with a new capsule file will repeat reboot sequence. On the > other hand, if CONFIG_E

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi 2022年2月15日(火) 23:15 Heinrich Schuchardt : > > On 2/15/22 10:05, Masami Hiramatsu wrote: > > Since now the capsule_on_disk will restart the u-boot sandbox right > > after the capsule update, if CONFIG_EFI_CAPSULE_ON_DISK_EARLY=y, the > > boot with a new capsule file will repeat reboot sequence.

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On 2/15/22 10:05, Masami Hiramatsu wrote: Since now the capsule_on_disk will restart the u-boot sandbox right after the capsule update, if CONFIG_EFI_CAPSULE_ON_DISK_EARLY=y, the boot with a new capsule file will repeat reboot sequence. On the other hand, if CONFIG_EFI_CAPSULE_ON_DISK_EARLY=n, th

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On 2/15/22 14:51, Heinrich Schuchardt wrote: On 2/15/22 10:09, Masami Hiramatsu wrote: Hi Heinrich, I and Takahiro found that the capsule update test case and test framework didn't expect that a command can reboot the sandbox and the sandbox can reboot while booting, which happens when my "Re

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

On 2/15/22 10:09, Masami Hiramatsu wrote: Hi Heinrich, I and Takahiro found that the capsule update test case and test framework didn't expect that a command can reboot the sandbox and the sandbox can reboot while booting, which happens when my "Reset system after CapsuleUpdate on disk" patch

Re: [PATCH] test/py: efi_capsule: Handle expected reset after capsule on disk

Hi Heinrich, I and Takahiro found that the capsule update test case and test framework didn't expect that a command can reboot the sandbox and the sandbox can reboot while booting, which happens when my "Reset system after CapsuleUpdate on disk" patch applied. This patch fixes those issues. Thank