[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 06/27] block: Make bdrv_reopen_{commit, abort
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v5 06/27] block: Make bdrv_reopen_{commit, abort} private functions |
Date: |
Fri, 27 May 2016 15:42:35 +0800 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Tue, 05/24 18:09, Max Reitz wrote:
> On 17.05.2016 09:35, Fam Zheng wrote:
> > As parts of the transactional reopen, they are not necessary outside
> > block.c. Make them static.
> >
> > Signed-off-by: Fam Zheng <address@hidden>
> > ---
> > block.c | 6 ++++--
> > include/block/block.h | 2 --
> > 2 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/block.c b/block.c
> > index 1b42303..ad3663c 100644
> > --- a/block.c
> > +++ b/block.c
> > @@ -1943,6 +1943,8 @@ BlockReopenQueue *bdrv_reopen_queue(BlockReopenQueue
> > *bs_queue,
> > * to all devices.
> > *
> > */
> > +static void bdrv_reopen_commit(BDRVReopenState *reopen_state);
> > +static void bdrv_reopen_abort(BDRVReopenState *reopen_state);
>
> I'd rather put these declarations above the comment describing
> bdrv_reopen_multiple(); or just at the top of the file, we have two such
> declarations there already.
Right, moving them to above the comment.
Fam
- Re: [Qemu-devel] [PATCH v5 02/27] qapi: Add lock-mode in blockdev-add options, (continued)
- [Qemu-devel] [PATCH v5 04/27] block: Introduce image file locking, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 07/27] block: Handle image locking during reopen, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 05/27] block: Add bdrv_image_locked, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 06/27] block: Make bdrv_reopen_{commit, abort} private functions, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 08/27] osdep: Add qemu_lock_fd and qemu_unlock_fd, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 12/27] gluster: Implement .bdrv_lockf, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 09/27] osdep: Introduce qemu_dup, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 11/27] raw-posix: Implement .bdrv_lockf, Fam Zheng, 2016/05/17
- [Qemu-devel] [PATCH v5 10/27] raw-posix: Use qemu_dup, Fam Zheng, 2016/05/17