On 1/25/19 4:37 PM, Stefan Hajnoczi wrote: > On Fri, Jan 25, 2019 at 10:36:17AM +0000, Peter Maydell wrote: >> On Fri, 25 Jan 2019 at 10:28, Stefan Hajnoczi wrote: >>> >>> On Thu, Jan 24, 2019 at 12:11:55PM +0100, Philippe Mathieu-Daudé wrote: >>>> The device realize() is also executed before the guest is started, is >>>> this call really necessary? >>> >>> My rationale was that machine init only happens before the guest is >>> started while ->realize() is called by hotplug too. >> >> Yes, but before realize the flash memory can't actually be >> visible to the guest, so there's no need to invalidate anything. >> >>> That said, can pflash devices be hotplugged? >> >> I guess you could have one inside a hotplugged device in theory; >> they can't be directly hotpluggged. (I bet that in practice >> there are leaks though if you did do that, and we don't have >> any devices that do so today.) Can we assume pflash aren't hotpluggable until someone find an use case and is willing to work on it? I can't find any practical example. >>> If you guys prefer not flushing from pflash ->realize() then I'll drop >>> it. >> >> I think it makes more sense not to do a flush in the realize method. > > Okay, I'll resend this patch. Thanks! Phil.