[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 5/8] migration: Add braces {} for if statement
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH v2 5/8] migration: Add braces {} for if statement |
Date: |
Wed, 14 Oct 2020 20:17:10 +0100 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
* Bihong Yu (yubihong@huawei.com) wrote:
> Signed-off-by: Bihong Yu <yubihong@huawei.com>
> Reviewed-by: Chuan Zheng <zhengchuan@huawei.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> migration/ram.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 0aea78f..09178cc 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -101,14 +101,16 @@ static struct {
>
> static void XBZRLE_cache_lock(void)
> {
> - if (migrate_use_xbzrle())
> + if (migrate_use_xbzrle()) {
> qemu_mutex_lock(&XBZRLE.lock);
> + }
> }
>
> static void XBZRLE_cache_unlock(void)
> {
> - if (migrate_use_xbzrle())
> + if (migrate_use_xbzrle()) {
> qemu_mutex_unlock(&XBZRLE.lock);
> + }
> }
>
> /**
> --
> 1.8.3.1
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
- Re: [PATCH v2 1/8] migration: Do not use C99 // comments, (continued)
[PATCH v2 4/8] migration: Open brace '{' following struct go on the same line, Bihong Yu, 2020/10/12
[PATCH v2 6/8] migration: Do not initialise statics and globals to 0 or NULL, Bihong Yu, 2020/10/12
[PATCH v2 7/8] migration: Open brace '{' following function declarations go on the next line, Bihong Yu, 2020/10/12
[PATCH v2 5/8] migration: Add braces {} for if statement, Bihong Yu, 2020/10/12
[PATCH v2 8/8] migration: Delete redundant spaces, Bihong Yu, 2020/10/12