[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/5] xen-mfndump: drop dead assignment to "page" from lookup_pte_func()

To: Jan Beulich <jbeulich@xxxxxxxx>
From: Jason Andryuk <jandryuk@xxxxxxxxx>
Date: Mon, 12 Jun 2023 08:47:41 -0400
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
Delivery-date: Mon, 12 Jun 2023 12:47:58 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Mon, Jun 12, 2023 at 7:45 AM Jan Beulich <jbeulich@xxxxxxxx> wrote:
>
> The variable isn't used past the loop, and its value also isn't
> meaningful across iterations. Reduce its scope to make this more
> obvious.
>
> Coverity ID: 1532310
> Fixes: ae763e422430 ("tools/misc: introduce xen-mfndump")
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Jason Andryuk <jandryuk@xxxxxxxxx>

Thanks,
Jason

Follow-Ups:
- Re: [PATCH 1/5] xen-mfndump: drop dead assignment to "page" from lookup_pte_func()
  - From: Anthony PERARD

References:
- [PATCH 0/5] tools: address Coverity UNUSED issues
  - From: Jan Beulich
- [PATCH 1/5] xen-mfndump: drop dead assignment to "page" from lookup_pte_func()
  - From: Jan Beulich

Prev by Date: Re: [QEMU PATCH 1/1] virtgpu: do not destroy resources when guest suspend
Next by Date: [PATCH v2] xen/arm: rename guest_cpuinfo in domain_cpuinfo
Previous by thread: [PATCH 1/5] xen-mfndump: drop dead assignment to "page" from lookup_pte_func()
Next by thread: Re: [PATCH 1/5] xen-mfndump: drop dead assignment to "page" from lookup_pte_func()
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.