On Thu, Mar 26, 2020 at 11:21:47AM +0000, Peter Maydell wrote: > On Thu, 26 Mar 2020 at 00:39, David Gibson wrote: > > > > On Tue, Mar 24, 2020 at 05:39:12PM +1100, Alexey Kardashevskiy wrote: > > > Coverity detected an issue (CID 1421903) with potential call of clz64(0) > > > which returns 64 which make it do "<<" with a negative number. > > > > > > This checks the mask and avoids undefined behaviour. > > > > > > In practice pgsizes and memory_region_iommu_get_min_page_size() always > > > have some common page sizes and even if they did not, the resulting page > > > size would be 0x8000.0000.0000.0000 (gcc 9.2) and > > > ioctl(VFIO_IOMMU_SPAPR_TCE_CREATE) would fail anyway. > > > > > > Signed-off-by: Alexey Kardashevskiy > > > > Applied to ppc-for-5.1. > > As a coverity-issue-fix it would be nice to have this in > 5.0 unless you think it's particularly risky. In fact, I realized that shortly after and moved it. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson