[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/5] coccinelle: use DIV_ROUND_UP
|
From: |
Eric Blake |
|
Subject: |
Re: [Qemu-devel] [PATCH 2/5] coccinelle: use DIV_ROUND_UP |
|
Date: |
Wed, 7 Jun 2017 16:50:23 -0500 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 06/07/2017 02:46 AM, Marc-André Lureau wrote:
> The coccinelle/round.cocci script doesn't catch hard coded values.
>
> I used the following script over qemu code base:
>
> (
> - ((e1) + 3) / (4)
> + DIV_ROUND_UP(e1,4)
As in 1/5, can't you also write a search for ((e1) + (e2) - 1) / e2, to
cover non-constant divisions?
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/5] Code cleanups with Coccinelle, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH 1/5] coccinelle: replace code with ROUND_UP macro, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH 2/5] coccinelle: use DIV_ROUND_UP, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH 3/5] arch: introduce ELF_NOTE_SIZE macro, Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH 4/5] Replace g_malloc()+memcpy() with g_memdup(), Marc-André Lureau, 2017/06/07
- [Qemu-devel] [PATCH 5/5] coccinelle: prefer glib g_new/g_renew macros, Marc-André Lureau, 2017/06/07