virtio pmem: coverity fixes

[Qemu-devel] [PATCH 0/3] virtio pmem: coverity fixes

Pankaj Gupta posted 3 patches 4 years, 9 months ago

Download series mbox

Test docker-clang@ubuntu passed

Test asan passed

Test docker-mingw@fedora passed

Test FreeBSD passed

Test checkpatch passed

Test s390x passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190712073554.21918-1-pagupta@redhat.com

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>

hw/virtio/virtio-pmem-pci.c | 4 +---
hw/virtio/virtio-pmem.c     | 4 ++--
2 files changed, 3 insertions(+), 5 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/3] virtio pmem: coverity fixes

Posted by Pankaj Gupta 4 years, 9 months ago

This patch series two fixes for coverity and a 
transactional info removal patch.

Pankaj Gupta (3):
  virtio pmem: fix wrong mem region condition
  virtio pmem: remove memdev null check
  virtio pmem: remove transational device info

 hw/virtio/virtio-pmem-pci.c | 4 +---
 hw/virtio/virtio-pmem.c     | 4 ++--
 2 files changed, 3 insertions(+), 5 deletions(-)

-- 
2.14.5

Re: [Qemu-devel] [PATCH 0/3] virtio pmem: coverity fixes

Posted by Cornelia Huck 4 years, 9 months ago

On Fri, 12 Jul 2019 13:05:51 +0530
Pankaj Gupta <pagupta@redhat.com> wrote:

> This patch series two fixes for coverity and a 
> transactional info removal patch.
> 
> Pankaj Gupta (3):
>   virtio pmem: fix wrong mem region condition
>   virtio pmem: remove memdev null check
>   virtio pmem: remove transational device info
> 
>  hw/virtio/virtio-pmem-pci.c | 4 +---
>  hw/virtio/virtio-pmem.c     | 4 ++--
>  2 files changed, 3 insertions(+), 5 deletions(-)
> 

I think all of this is 4.1 material.

Re: [Qemu-devel] [PATCH 0/3] virtio pmem: coverity fixes

Posted by Pankaj Gupta 4 years, 9 months ago

> 
> On Fri, 12 Jul 2019 13:05:51 +0530
> Pankaj Gupta <pagupta@redhat.com> wrote:
> 
> > This patch series two fixes for coverity and a
> > transactional info removal patch.
> > 
> > Pankaj Gupta (3):
> >   virtio pmem: fix wrong mem region condition
> >   virtio pmem: remove memdev null check
> >   virtio pmem: remove transational device info
> > 
> >  hw/virtio/virtio-pmem-pci.c | 4 +---
> >  hw/virtio/virtio-pmem.c     | 4 ++--
> >  2 files changed, 3 insertions(+), 5 deletions(-)
> > 
> 
> I think all of this is 4.1 material.

Yes, forgot to add in the subject.

Thank you for the review.

Best regards,
Pankaj

> 
>