On Thu, Jan 18, 2018 at 03:51:17PM -0600, Eric Blake wrote:
> On 01/18/2018 02:44 PM, Michael S. Tsirkin wrote:
> > From: Yuval Shaia <yuval.sh...@oracle.com>
> >
> > This function should be declared in generic header file so we can
> > utilize it.
> >
> > Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> > Signed-off-by: Yuval Shaia <yuval.sh...@oracle.com>
> > Signed-off-by: Marcel Apfelbaum <mar...@redhat.com>
> > Reviewed-by: Michael S. Tsirkin <m...@redhat.com>
> > Signed-off-by: Michael S. Tsirkin <m...@redhat.com>
> > ---
> > include/qemu/host-utils.h | 10 ++++++++++
> > hw/pci/shpc.c | 13 ++-----------
> > 2 files changed, 12 insertions(+), 11 deletions(-)
> >
> > diff --git a/include/qemu/host-utils.h b/include/qemu/host-utils.h
> > index 5ac621c..38da849 100644
> > --- a/include/qemu/host-utils.h
> > +++ b/include/qemu/host-utils.h
> > @@ -400,6 +400,16 @@ static inline uint64_t pow2ceil(uint64_t value)
> > return 0x8000000000000000ull >> (n - 1);
> > }
> >
> > +static inline uint32_t pow2roundup32(uint32_t x)
> > +{
> > + x |= (x >> 1);
> > + x |= (x >> 2);
> > + x |= (x >> 4);
> > + x |= (x >> 8);
> > + x |= (x >> 16);
> > + return x + 1;
> > +}
>
> How does this differ from pow2ceil(), other than the difference in type?
Hi Michael,
I tested pvrdma device with pow2ceil and it works fine however i have no
idea how to test shpc.
So from pvrdma perspective this commit can be dropped. I can replace it
with one that change shpc_bar_size to use pow2ceil but again - compiled,
not tested.
Yuval
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3266
> Virtualization: qemu.org | libvirt.org
>
