Cao jin <caoj.f...@cn.fujitsu.com> writes: > On 12/19/2015 05:18 AM, Markus Armbruster wrote: >> One short remark in addition to Eduardo's review. >> >> Eduardo Habkost <ehabk...@redhat.com> 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.