[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH 11/29] s390x/sclp: use QEMU_IS_ALIGNED macro
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-trivial] [PATCH 11/29] s390x/sclp: use QEMU_IS_ALIGNED macro |
Date: |
Tue, 18 Jul 2017 12:03:19 +0200 |
On Tue, 18 Jul 2017 11:58:40 +0200
Cornelia Huck <address@hidden> wrote:
> On Tue, 18 Jul 2017 03:09:47 -0300
> Philippe Mathieu-Daudé <address@hidden> wrote:
>
> > Applied using the Coccinelle semantic patch
> > scripts/coccinelle/use_osdep.cocci
> >
> > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > ---
> > hw/s390x/sclp.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/hw/s390x/sclp.c b/hw/s390x/sclp.c
> > index 9253dbbc64..0a207c7345 100644
> > --- a/hw/s390x/sclp.c
> > +++ b/hw/s390x/sclp.c
> > @@ -237,7 +237,7 @@ static void assign_storage(SCLPDevice *sclp, SCCB *sccb)
> > }
> > assign_addr = (assign_info->rn - 1) * mhd->rzm;
> >
> > - if ((assign_addr % MEM_SECTION_SIZE == 0) &&
> > + if (QEMU_IS_ALIGNED(assign_addr, MEM_SECTION_SIZE) &&
> > (assign_addr >= mhd->padded_ram_size)) {
> > /* Re-use existing memory region if found */
> > mr = memory_region_find(sysmem, assign_addr, 1).mr;
> > @@ -297,7 +297,7 @@ static void unassign_storage(SCLPDevice *sclp, SCCB
> > *sccb)
> > unassign_addr = (assign_info->rn - 1) * mhd->rzm;
> >
> > /* if the addr is a multiple of 256 MB */
> > - if ((unassign_addr % MEM_SECTION_SIZE == 0) &&
> > + if (QEMU_IS_ALIGNED(unassign_addr, MEM_SECTION_SIZE) &&
> > (unassign_addr >= mhd->padded_ram_size)) {
> > mhd->standby_state_map[(unassign_addr -
> > mhd->padded_ram_size) / MEM_SECTION_SIZE] = 0;
>
> Acked-by: Cornelia Huck <address@hidden>
Oh, and I assume this will go through trivial, right?
- Re: [Qemu-trivial] [PATCH 07/29] ds1338: use QEMU_IS_ALIGNED macro, (continued)
- [Qemu-trivial] [PATCH 08/29] ui: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 09/29] net: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 10/29] net/rocker: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 11/29] s390x/sclp: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 12/29] sm501: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 13/29] tcg: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 14/29] cris: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 16/29] lm32: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 18/29] nios2: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18
- [Qemu-trivial] [PATCH 19/29] disas: use QEMU_IS_ALIGNED macro, Philippe Mathieu-Daudé, 2017/07/18