[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader |
Date: |
Thu, 02 Jan 2014 13:16:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130923 Thunderbird/17.0.9 |
Il 02/01/2014 11:51, Peter Crosthwaite ha scritto:
>> > No, please use "realize" and avoid init. This way you can use an Error*
>> > to report the error.
>> >
>> > Also, the actual load_image_targphys call probably should be done in a
>> > reset handler, not at realize time.
>> >
> Ok I think that settles it. The actual blobbing needs to happen at
> reset time. Perhaps the correct approach is to do as much as possible
> (file-path / address sanitsation etc) at realize time, then only the
> actual blob load happens at reset. Going on what Paolo said, I think
> for this device ::init is actually a nop.
Yeah, also because init is in fact a legacy interface to realize.
Paolo
- [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/02
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Peter Crosthwaite, 2014/01/02
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/02
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Paolo Bonzini, 2014/01/02
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Peter Crosthwaite, 2014/01/02
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/05
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/05
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Peter Crosthwaite, 2014/01/05
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/05
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Peter Crosthwaite, 2014/01/05
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/05
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/06
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Peter Crosthwaite, 2014/01/06
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Li Guang, 2014/01/06
- Re: [Qemu-devel] [PATCH] hw/misc/blob-loader: add a generic blob loader, Paolo Bonzini, 2014/01/06