[PULL,13/29] vga: use DIV_ROUND_UP

Message ID	20170831103432.6007-14-marcandre.lureau@redhat.com
State	New
Headers	show Return-Path: <qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org> DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 65D76285A8 From: =?utf-8?q?Marc-Andr=C3=A9_Lureau?= <marcandre.lureau@redhat.com> To: qemu-devel@nongnu.org Date: Thu, 31 Aug 2017 12:34:16 +0200 Message-Id: <20170831103432.6007-14-marcandre.lureau@redhat.com> In-Reply-To: <20170831103432.6007-1-marcandre.lureau@redhat.com> References: <20170831103432.6007-1-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PULL 13/29] vga: use DIV_ROUND_UP Precedence: list Cc: qemu-trivial@nongnu.org, peter.maydell@linaro.org, =?utf-8?q?Marc-Andr?= =?utf-8?b?w6kgTHVyZWF1?= <marcandre.lureau@redhat.com> Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" <qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>
Series	[PULL,01/29] i386: use ROUND_UP macro \| expand [PULL,01/29] i386: use ROUND_UP macro [PULL,02/29] vnc: use QEMU_ALIGN_DOWN [PULL,03/29] vhdx: use QEMU_ALIGN_DOWN [PULL,04/29] vhost: use QEMU_ALIGN_DOWN [PULL,05/29] i8254: use QEMU_ALIGN_DOWN [PULL,06/29] pcspk: use QEMU_ALIGN_DOWN [PULL,07/29] dmg: use DIV_ROUND_UP [PULL,08/29] qcow2: use DIV_ROUND_UP [PULL,09/29] vpc: use DIV_ROUND_UP [PULL,10/29] vvfat: use DIV_ROUND_UP [PULL,11/29] vnc: use DIV_ROUND_UP [PULL,12/29] ui: use DIV_ROUND_UP [PULL,13/29] vga: use DIV_ROUND_UP [PULL,14/29] virtio-gpu: use DIV_ROUND_UP [PULL,15/29] monitor: use DIV_ROUND_UP [PULL,16/29] console: use DIV_ROUND_UP [PULL,17/29] virtio-serial: use DIV_ROUND_UP [PULL,18/29] piix: use DIV_ROUND_UP [PULL,19/29] q35: use DIV_ROUND_UP [PULL,20/29] usb-hub: use DIV_ROUND_UP [PULL,21/29] msix: use DIV_ROUND_UP [PULL,22/29] ppc: use DIV_ROUND_UP [PULL,23/29] i386/dump: use DIV_ROUND_UP [PULL,24/29] kvm: use DIV_ROUND_UP [PULL,25/29] decnumber: use DIV_ROUND_UP [PULL,26/29] i386: introduce ELF_NOTE_SIZE macro [PULL,27/29] i386: replace g_malloc()+memcpy() with g_memdup() [PULL,28/29] test-iov: replace g_malloc()+memcpy() with g_memdup() [PULL,29/29] eepro100: replace g_malloc()+memcpy() with g_memdup()

Message ID

20170831103432.6007-14-marcandre.lureau@redhat.com

State

New

Headers

DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 65D76285A8
From: =?utf-8?q?Marc-Andr=C3=A9_Lureau?= <marcandre.lureau@redhat.com>
To: qemu-devel@nongnu.org
Date: Thu, 31 Aug 2017 12:34:16 +0200
Message-Id: <20170831103432.6007-14-marcandre.lureau@redhat.com>
In-Reply-To: <20170831103432.6007-1-marcandre.lureau@redhat.com>
References: <20170831103432.6007-1-marcandre.lureau@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: [Qemu-devel] [PULL 13/29] vga: use DIV_ROUND_UP
Precedence: list
Cc: qemu-trivial@nongnu.org, peter.maydell@linaro.org, =?utf-8?q?Marc-Andr?=
	=?utf-8?b?w6kgTHVyZWF1?= <marcandre.lureau@redhat.com>
Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org
Sender: "Qemu-devel"
	<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>

Series

[PULL,01/29] i386: use ROUND_UP macro | expand

Commit Message

Marc-André Lureau Aug. 31, 2017, 10:34 a.m. UTC

I used the clang-tidy qemu-round check to generate the fix:
https://github.com/elmarco/clang-tools-extra

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Richard Henderson <rth@twiddle.net>
---
 hw/display/vga.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/display/vga.c b/hw/display/vga.c
index 63421f9ee8..3433102ef3 100644
--- a/hw/display/vga.c
+++ b/hw/display/vga.c
@@ -1621,7 +1621,7 @@  static void vga_draw_graphic(VGACommonState *s, int full_update)
            s->line_compare, sr(s, VGA_SEQ_CLOCK_MODE));
 #endif
     addr1 = (s->start_addr * 4);
-    bwidth = (width * bits + 7) / 8;
+    bwidth = DIV_ROUND_UP(width * bits, 8);
     y_start = -1;
     d = surface_data(surface);
     linesize = surface_stride(surface);

[PULL,13/29] vga: use DIV_ROUND_UP

Commit Message

Patch