[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/6] device-tree: add re-randomization helper function
From: |
Jason A. Donenfeld |
Subject: |
Re: [PATCH 1/6] device-tree: add re-randomization helper function |
Date: |
Mon, 10 Oct 2022 09:20:54 -0600 |
On Mon, Oct 10, 2022 at 11:54:50AM +0100, Peter Maydell wrote:
> The error is essentially the record-and-replay subsystem saying "the
> replay just asked for a random number at point when the recording
> did not ask for one, and so there's no 'this is what the number was'
> info in the record".
>
> I have had a quick look, and I think the reason for this is that
> load_snapshot() ("reset the VM state to the snapshot state stored in the
> disk image or migration stream") does a system reset. The replay
> process involves a lot of "load state from a snapshot and play
> forwards from there" operations. It doesn't expect that load_snapshot()
> would result in something reading random data, but now that we are
> calling qemu_guest_getrandom() in a reset hook, that happens.
Hmm... so this seems like a bug in the replay code then? Shouldn't that
reset handler get hit during both passes, so the entry should be in
each?
Jason
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Peter Maydell, 2022/10/06
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Jason A. Donenfeld, 2022/10/06
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Peter Maydell, 2022/10/10
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Peter Maydell, 2022/10/10
- Re: [PATCH 1/6] device-tree: add re-randomization helper function,
Jason A. Donenfeld <=
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Peter Maydell, 2022/10/10
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Jason A. Donenfeld, 2022/10/10
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Pavel Dovgalyuk, 2022/10/11
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Jason A. Donenfeld, 2022/10/11
- Re: [PATCH 1/6] device-tree: add re-randomization helper function, Jason A. Donenfeld, 2022/10/11