Message ID | 1424122891-9390-5-git-send-email-thuth@linux.vnet.ibm.com |
---|---|
State | New |
Headers | show |
On 2015-02-16 at 16:41, Thomas Huth wrote: > qemu_try_blockalign0() and nbd_export_close_all() are not > used anymore and thus can be removed. > > Signed-off-by: Thomas Huth <thuth@linux.vnet.ibm.com> > Cc: Kevin Wolf <kwolf@redhat.com> > Cc: Stefan Hajnoczi <stefanha@redhat.com> > Cc: Paolo Bonzini <pbonzini@redhat.com> > --- > block.c | 11 ----------- > include/block/block.h | 1 - > include/block/nbd.h | 1 - > nbd.c | 9 --------- > 4 files changed, 0 insertions(+), 22 deletions(-) NACK, I'm using nbd_export_close_all() in my "block: Rework bdrv_close_all()" series. I'm not so sure about qemu_try_blockalign0(); it has never been used, but I introduced it because we have qemu_blockalign(), qemu_try_blockalign(), and qemu_blockalign0() (the latter of which I introduced along with qemu_try_blockalign0(), and this function is used). So I'd be fine with removing qemu_try_blockalign0() again, but I don't really see the point in doing so. It is not a function that is per-se deprecated or something, quite the opposite, actually. If people can make use of that function, they should most certainly do so. Max > diff --git a/block.c b/block.c > index 210fd5f..2128f82 100644 > --- a/block.c > +++ b/block.c > @@ -5335,17 +5335,6 @@ void *qemu_try_blockalign(BlockDriverState *bs, size_t size) > return qemu_try_memalign(align, size); > } > > -void *qemu_try_blockalign0(BlockDriverState *bs, size_t size) > -{ > - void *mem = qemu_try_blockalign(bs, size); > - > - if (mem) { > - memset(mem, 0, size); > - } > - > - return mem; > -} > - > /* > * Check if all memory in this vector is sector aligned. > */ > diff --git a/include/block/block.h b/include/block/block.h > index 321295e..7ee9ab3 100644 > --- a/include/block/block.h > +++ b/include/block/block.h > @@ -433,7 +433,6 @@ void bdrv_set_guest_block_size(BlockDriverState *bs, int align); > void *qemu_blockalign(BlockDriverState *bs, size_t size); > void *qemu_blockalign0(BlockDriverState *bs, size_t size); > void *qemu_try_blockalign(BlockDriverState *bs, size_t size); > -void *qemu_try_blockalign0(BlockDriverState *bs, size_t size); > bool bdrv_qiov_is_aligned(BlockDriverState *bs, QEMUIOVector *qiov); > > struct HBitmapIter; > diff --git a/include/block/nbd.h b/include/block/nbd.h > index b759595..fccef2d 100644 > --- a/include/block/nbd.h > +++ b/include/block/nbd.h > @@ -95,7 +95,6 @@ BlockBackend *nbd_export_get_blockdev(NBDExport *exp); > > NBDExport *nbd_export_find(const char *name); > void nbd_export_set_name(NBDExport *exp, const char *name); > -void nbd_export_close_all(void); > > NBDClient *nbd_client_new(NBDExport *exp, int csock, > void (*close)(NBDClient *)); > diff --git a/nbd.c b/nbd.c > index e56afbc..634bf84 100644 > --- a/nbd.c > +++ b/nbd.c > @@ -1067,15 +1067,6 @@ BlockBackend *nbd_export_get_blockdev(NBDExport *exp) > return exp->blk; > } > > -void nbd_export_close_all(void) > -{ > - NBDExport *exp, *next; > - > - QTAILQ_FOREACH_SAFE(exp, &exports, next, next) { > - nbd_export_close(exp); > - } > -} > - > static ssize_t nbd_co_send_reply(NBDRequest *req, struct nbd_reply *reply, > int len) > {
Hi Max, On Mon, 16 Feb 2015 17:26:56 -0500 Max Reitz <mreitz@redhat.com> wrote: > On 2015-02-16 at 16:41, Thomas Huth wrote: > > qemu_try_blockalign0() and nbd_export_close_all() are not > > used anymore and thus can be removed. > > > > Signed-off-by: Thomas Huth <thuth@linux.vnet.ibm.com> > > Cc: Kevin Wolf <kwolf@redhat.com> > > Cc: Stefan Hajnoczi <stefanha@redhat.com> > > Cc: Paolo Bonzini <pbonzini@redhat.com> > > --- > > block.c | 11 ----------- > > include/block/block.h | 1 - > > include/block/nbd.h | 1 - > > nbd.c | 9 --------- > > 4 files changed, 0 insertions(+), 22 deletions(-) > > NACK, I'm using nbd_export_close_all() in my "block: Rework > bdrv_close_all()" series. Alright, if it's going to be used again, then it must not be removed, of course. > I'm not so sure about qemu_try_blockalign0(); it has never been used, > but I introduced it because we have qemu_blockalign(), > qemu_try_blockalign(), and qemu_blockalign0() (the latter of which I > introduced along with qemu_try_blockalign0(), and this function is used). > > So I'd be fine with removing qemu_try_blockalign0() again, but I don't > really see the point in doing so. It is not a function that is per-se > deprecated or something, quite the opposite, actually. If people can > make use of that function, they should most certainly do so. I'm not a big fan of providing unused (and likely untested) functions just because they could be used somewhen in a distant future (unless they are part of a proper library) ... I'd rather add such functions again when they are really being used. Anyway, this is just a small, trivial function, and if you prefer to keep it, then please ignore this patch simply completely. There is certainly no urgent need for removing unused functions, I just wanted to make people aware that there are some. Thomas
diff --git a/block.c b/block.c index 210fd5f..2128f82 100644 --- a/block.c +++ b/block.c @@ -5335,17 +5335,6 @@ void *qemu_try_blockalign(BlockDriverState *bs, size_t size) return qemu_try_memalign(align, size); } -void *qemu_try_blockalign0(BlockDriverState *bs, size_t size) -{ - void *mem = qemu_try_blockalign(bs, size); - - if (mem) { - memset(mem, 0, size); - } - - return mem; -} - /* * Check if all memory in this vector is sector aligned. */ diff --git a/include/block/block.h b/include/block/block.h index 321295e..7ee9ab3 100644 --- a/include/block/block.h +++ b/include/block/block.h @@ -433,7 +433,6 @@ void bdrv_set_guest_block_size(BlockDriverState *bs, int align); void *qemu_blockalign(BlockDriverState *bs, size_t size); void *qemu_blockalign0(BlockDriverState *bs, size_t size); void *qemu_try_blockalign(BlockDriverState *bs, size_t size); -void *qemu_try_blockalign0(BlockDriverState *bs, size_t size); bool bdrv_qiov_is_aligned(BlockDriverState *bs, QEMUIOVector *qiov); struct HBitmapIter; diff --git a/include/block/nbd.h b/include/block/nbd.h index b759595..fccef2d 100644 --- a/include/block/nbd.h +++ b/include/block/nbd.h @@ -95,7 +95,6 @@ BlockBackend *nbd_export_get_blockdev(NBDExport *exp); NBDExport *nbd_export_find(const char *name); void nbd_export_set_name(NBDExport *exp, const char *name); -void nbd_export_close_all(void); NBDClient *nbd_client_new(NBDExport *exp, int csock, void (*close)(NBDClient *)); diff --git a/nbd.c b/nbd.c index e56afbc..634bf84 100644 --- a/nbd.c +++ b/nbd.c @@ -1067,15 +1067,6 @@ BlockBackend *nbd_export_get_blockdev(NBDExport *exp) return exp->blk; } -void nbd_export_close_all(void) -{ - NBDExport *exp, *next; - - QTAILQ_FOREACH_SAFE(exp, &exports, next, next) { - nbd_export_close(exp); - } -} - static ssize_t nbd_co_send_reply(NBDRequest *req, struct nbd_reply *reply, int len) {
qemu_try_blockalign0() and nbd_export_close_all() are not used anymore and thus can be removed. Signed-off-by: Thomas Huth <thuth@linux.vnet.ibm.com> Cc: Kevin Wolf <kwolf@redhat.com> Cc: Stefan Hajnoczi <stefanha@redhat.com> Cc: Paolo Bonzini <pbonzini@redhat.com> --- block.c | 11 ----------- include/block/block.h | 1 - include/block/nbd.h | 1 - nbd.c | 9 --------- 4 files changed, 0 insertions(+), 22 deletions(-)