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
