[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v9 16/17] libxl: convert libxl_device_vkb_add to an async operation

Roger Pau Monne writes ("Re: [PATCH v9 16/17] libxl: convert 
libxl_device_vkb_add to an async operation"):
> Ian Jackson wrote:
> > This is rather unfriendly - doesn't it amount to aborting on an
> > invalid domain configuration.  And I'm not sure I understand the
> > logic, which is new in your patch.
> This was part of the previous code, but this was done inside of a switch
> instead of an if. Since it's not possible to specify vkb devices in the
> config file (at least xl.cfg(5) doesn't mention anything about vkbs),
> I'm quite sure d_config->vkbs == NULL and d_config->num_vkbs == 0 always
> or something has gone extremely wrong, just added the abort to be sure.

People who write other code which calls libxl might try to pass vkbds,
but then the code would crash.  And if they didn't test it with HVM
domains, they wouldn't notice that their code had a crash bug.

I think it should cause the creation to fail with ERROR_INVAL.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.