On 05/11/2015 08:54 AM, Stefan Hajnoczi wrote: > On Mon, May 04, 2015 at 04:42:24PM +0300, Denis V. Lunev wrote: >> @@ -726,7 +727,8 @@ static void raw_refresh_limits(BlockDriverState *bs, >> Error **errp) >> >> raw_probe_alignment(bs, s->fd, errp); >> bs->bl.min_mem_alignment = s->buf_align; >> - bs->bl.opt_mem_alignment = s->buf_align; >> + if (bs->bl.min_mem_alignment > bs->bl.opt_mem_alignment) >> + bs->bl.opt_mem_alignment = bs->bl.min_mem_alignment; > > QEMU coding style uses {} even when the if statement body is only one > line.
scripts/checkpatch.pl helps you check for this and other style issues. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature