On Mon, Nov 02, 2015 at 05:13:22PM +0800, Xiao Guangrong wrote:
[...]
> static MemoryRegion *pc_dimm_get_memory_region(DIMMDevice *dimm)
> {
> - return host_memory_backend_get_memory(dimm->hostmem, &error_abort);
> + Error *local_err = NULL;
> + MemoryRegion *mr;
> +
> + mr = host_memory_backend_get_memory(dimm->hostmem, &local_err);
> +
> + /*
> + * plug a pc-dimm device whose backend memory was not properly
> + * initialized?
> + */
> + assert(!local_err && mr);
I don't know if you are going to remove the errp parameter in the next
version, but if you want to simply abort in case an error is reported by
a function, you can use &error_abort.
--
Eduardo
