Re: [Qemu-devel] [RFC PATCH] vfio: Add sysfsdev property for pci & platform

[P J P]

+-- On Wed, 20 Jan 2016, Alex Williamson wrote --+
| > > +            error_printf("Warning : Device at %s is known to cause
| > > system instability issues during option rom execution. Proceeding
| > > anyway since user specified romfile\n",
| > > +                         vdev->vbasedev.name);
| > 
| > This error message should really be split across lines like
| > the original code was. Likewise for other cases in this patch
| 
| the source to find an error message, it should work.  Breaking printed
| error messages at arbitrary lengths to fit an 80 column window makes
| that more difficult.  Is there sufficient passion for a hard 80 column
| limit on printed strings in QEMU to ignore that useful property?

  I'd say +1 for breaking it at 80'th column. IMO it helps with readability in 
editor or tools like less(1). As for grep, it can still work with a 
substring/pattern from the error message.

--
Prasad J Pandit / Red Hat Product Security Team
47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F