Re: [RFC v2 1/7] vhost: Delete trailing dot in errpr_setg argument

[Eugenio Perez Martin]

On Tue, Feb 9, 2021 at 5:25 PM Eric Blake <eblake@redhat.com> wrote:
>
> On 2/9/21 9:37 AM, Eugenio Pérez wrote:
> > As error_setg points
>
> Incomplete sentence?
>
> Missing Signed-off-by.
>

Sorry, I should have paid more attention.

Maybe it is better to send this though qemu-trivial, so it does not
mess with this series?

Thanks!

> > ---
> >  hw/virtio/vhost.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
> > index 6e17d631f7..1c5b442081 100644
> > --- a/hw/virtio/vhost.c
> > +++ b/hw/virtio/vhost.c
> > @@ -1358,7 +1358,7 @@ int vhost_dev_init(struct vhost_dev *hdev, void 
> > *opaque,
> >      if (hdev->migration_blocker == NULL) {
> >          if (!(hdev->features & (0x1ULL << VHOST_F_LOG_ALL))) {
> >              error_setg(&hdev->migration_blocker,
> > -                       "Migration disabled: vhost lacks VHOST_F_LOG_ALL 
> > feature.");
> > +                       "Migration disabled: vhost lacks VHOST_F_LOG_ALL 
> > feature");
> >          } else if (vhost_dev_log_is_shared(hdev) && 
> > !qemu_memfd_alloc_check()) {
> >              error_setg(&hdev->migration_blocker,
> >                         "Migration disabled: failed to allocate shared 
> > memory");
> >
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3226
> Virtualization:  qemu.org | libvirt.org
>