On Tue, 2019-08-27 at 17:14 +0100, Chris Wilson wrote: > e.g. benchmarks/gem_wsim.c:2936:4: warning: absolute value function ‘abs’ given an argument of type ‘long int’ but has parameter of type ‘int’ which may cause truncation of value [-Wabsolute-value] > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Reviewed-by: Simon Ser <simon.ser@xxxxxxxxx> > --- > benchmarks/gem_syslatency.c | 2 +- > benchmarks/gem_wsim.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/benchmarks/gem_syslatency.c b/benchmarks/gem_syslatency.c > index 7671dc43e..d7cf0ed49 100644 > --- a/benchmarks/gem_syslatency.c > +++ b/benchmarks/gem_syslatency.c > @@ -343,7 +343,7 @@ static unsigned long calibrate_nop(unsigned int target_us, > sz = loops * sz / elapsed(&t_start, &t_end) * 1e3 * target_us; > sz = ALIGN(sz, sizeof(uint32_t)); > } while (elapsed(&t_0, &t_end) < 5 || > - abs(sz - prev) > (sz * tolerance_pct / 100)); > + labs(sz - prev) > (sz * tolerance_pct / 100)); > > close(fd); > > diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c > index a76fdbfe2..b8e22b3f6 100644 > --- a/benchmarks/gem_wsim.c > +++ b/benchmarks/gem_wsim.c > @@ -2933,7 +2933,7 @@ static unsigned long calibrate_nop(unsigned int tolerance_pct) > size = loops * size / elapsed(&t_start, &t_end) / 1e6 * usecs; > size = ALIGN(size, sizeof(uint32_t)); > } while (elapsed(&t_0, &t_end) < 5 || > - abs(size - last_size) > (size * tolerance_pct / 100)); > + labs(size - last_size) > (size * tolerance_pct / 100)); > > return size / sizeof(uint32_t); > } _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
