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)

[...]


.


--
Yours Sincerely,

Cao Jin



Reply via email to