Re: [PATCH] rose: correct integer overflow check

2016-02-18 Thread Ralf Baechle
On Thu, Feb 18, 2016 at 04:03:16PM -0500, Insu Yun wrote: > > Because of the types on the right hand side of the comparison > the expressions are all promoted to unsigned. > > Did you look at the compiler's assembler output?  I did when > reviewing your patch. > > > I checked t

Re: [PATCH] rose: correct integer overflow check

2016-02-18 Thread David Miller
From: Insu Yun Date: Wed, 17 Feb 2016 15:25:13 -0500 > Since rose_ndevs is signed integer type, > it can be overflowed when it is negative. > > Signed-off-by: Insu Yun That's not how the expression is evaluated. Because of the types on the right hand side of the comparison the expressions are