Re: [PATCH] x86: simplefb: avoid overflow

2013-09-06 Thread Tom Gundersen
On Fri, Sep 6, 2013 at 1:24 PM, David Herrmann wrote: > Hi > > On Fri, Sep 6, 2013 at 12:59 PM, Geert Uytterhoeven > wrote: >> On Fri, Sep 6, 2013 at 11:55 AM, David Herrmann >> wrote: >>> On Fri, Sep 6, 2013 at 11:32 AM, Tom Gundersen wrote: lfb_size can easily be say 4M, which would mak

Re: [PATCH] x86: simplefb: avoid overflow

2013-09-06 Thread David Herrmann
Hi On Fri, Sep 6, 2013 at 12:59 PM, Geert Uytterhoeven wrote: > On Fri, Sep 6, 2013 at 11:55 AM, David Herrmann wrote: >> On Fri, Sep 6, 2013 at 11:32 AM, Tom Gundersen wrote: >>> lfb_size can easily be say 4M, which would make the bitshit overflow and >>> the test fail. >>> >>> Signed-off-by:

Re: [PATCH] x86: simplefb: avoid overflow

2013-09-06 Thread Geert Uytterhoeven
On Fri, Sep 6, 2013 at 1:08 PM, H. Peter Anvin wrote: > On 09/06/2013 03:59 AM, Geert Uytterhoeven wrote: >> On Fri, Sep 6, 2013 at 11:55 AM, David Herrmann >> wrote: >>> On Fri, Sep 6, 2013 at 11:32 AM, Tom Gundersen wrote: lfb_size can easily be say 4M, which would make the bitshit overfl

Re: [PATCH] x86: simplefb: avoid overflow

2013-09-06 Thread H. Peter Anvin
On 09/06/2013 03:59 AM, Geert Uytterhoeven wrote: On Fri, Sep 6, 2013 at 11:55 AM, David Herrmann wrote: On Fri, Sep 6, 2013 at 11:32 AM, Tom Gundersen wrote: lfb_size can easily be say 4M, which would make the bitshit overflow and the test fail. Signed-off-by: Tom Gundersen Cc: David Herrm

Re: [PATCH] x86: simplefb: avoid overflow

2013-09-06 Thread Geert Uytterhoeven
On Fri, Sep 6, 2013 at 11:55 AM, David Herrmann wrote: > On Fri, Sep 6, 2013 at 11:32 AM, Tom Gundersen wrote: >> lfb_size can easily be say 4M, which would make the bitshit overflow and >> the test fail. >> >> Signed-off-by: Tom Gundersen >> Cc: David Herrmann >> Cc: H. Peter Anvin >> --- >>

Re: [PATCH] x86: simplefb: avoid overflow

2013-09-06 Thread David Herrmann
Hi On Fri, Sep 6, 2013 at 11:32 AM, Tom Gundersen wrote: > lfb_size can easily be say 4M, which would make the bitshit overflow and > the test fail. > > Signed-off-by: Tom Gundersen > Cc: David Herrmann > Cc: H. Peter Anvin > --- > arch/x86/kernel/sysfb_simplefb.c | 2 +- > 1 file changed, 1

[PATCH] x86: simplefb: avoid overflow

2013-09-06 Thread Tom Gundersen
lfb_size can easily be say 4M, which would make the bitshit overflow and the test fail. Signed-off-by: Tom Gundersen Cc: David Herrmann Cc: H. Peter Anvin --- arch/x86/kernel/sysfb_simplefb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/sysfb_simplefb.c b