Cao jin <[email protected]> writes: > On 12/19/2015 05:18 AM, Markus Armbruster wrote: >> One short remark in addition to Eduardo's review. >> >> Eduardo Habkost <[email protected]> writes: >> > [...] >>>> >>>> config_fd = open(path, O_RDWR); >>>> if (config_fd < 0) { >>>> + error_setg(errp, "No such device"); >>>> return -ENODEV; >>>> } >> >> Can we come up with nicer error messages? >> > > Ok...will change the error msg to strerror(errno)
There's also error_setg_file_open(), not sure it fits here, but check it out.
