[PATCH] nbd/server.c: Remove unused field

Nir Soffer posted 1 patch 2 years, 2 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220111194313.581486-1-nsoffer@redhat.com
Maintainers: Eric Blake <eblake@redhat.com>, Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
nbd/server.c | 1 -
1 file changed, 1 deletion(-)
[PATCH] nbd/server.c: Remove unused field
Posted by Nir Soffer 2 years, 2 months ago
NBDRequestData struct has unused QSIMPLEQ_ENTRY filed. It seems that
this field exists since the first git commit and was never used.

Signed-off-by: Nir Soffer <nsoffer@redhat.com>
---
 nbd/server.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/nbd/server.c b/nbd/server.c
index 3927f7789d..ce5b2a1d02 100644
--- a/nbd/server.c
+++ b/nbd/server.c
@@ -70,21 +70,20 @@ static int system_errno_to_nbd_errno(int err)
     default:
         return NBD_EINVAL;
     }
 }
 
 /* Definitions for opaque data types */
 
 typedef struct NBDRequestData NBDRequestData;
 
 struct NBDRequestData {
-    QSIMPLEQ_ENTRY(NBDRequestData) entry;
     NBDClient *client;
     uint8_t *data;
     bool complete;
 };
 
 struct NBDExport {
     BlockExport common;
 
     char *name;
     char *description;
-- 
2.34.1


Re: [PATCH] nbd/server.c: Remove unused field
Posted by Philippe Mathieu-Daudé 2 years, 2 months ago
On 1/11/22 20:43, Nir Soffer wrote:
> NBDRequestData struct has unused QSIMPLEQ_ENTRY filed. It seems that
> this field exists since the first git commit and was never used.
> 
> Signed-off-by: Nir Soffer <nsoffer@redhat.com>
> ---
>  nbd/server.c | 1 -
>  1 file changed, 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Re: [PATCH] nbd/server.c: Remove unused field
Posted by Eric Blake 2 years, 2 months ago
On Tue, Jan 11, 2022 at 09:17:43PM +0100, Philippe Mathieu-Daudé wrote:
> On 1/11/22 20:43, Nir Soffer wrote:
> > NBDRequestData struct has unused QSIMPLEQ_ENTRY filed. It seems that

s/filed/field/

> > this field exists since the first git commit and was never used.

If my git sleuthing is right, that's commit d9a73806, from 2011.

> > 
> > Signed-off-by: Nir Soffer <nsoffer@redhat.com>
> > ---
> >  nbd/server.c | 1 -
> >  1 file changed, 1 deletion(-)
> 
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Thanks, I'll add that to my NBD queue.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org