All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [bug report] s390/cio: use cssid for pgid generation
       [not found] <alpine.LFD.2.20.1701161356550.1815@schleppi>
@ 2017-01-16 13:18 ` Dan Carpenter
  0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2017-01-16 13:18 UTC (permalink / raw)
  To: linux-s390

On Mon, Jan 16, 2017 at 02:06:11PM +0100, Sebastian Ott wrote:
> Hello Dan,
> 
> On Mon, 16 Jan 2017, Dan Carpenter wrote:
> > The patch c8500e120b81: "s390/cio: use cssid for pgid generation"
> > from Jun 17, 2016, leads to the following static checker warning:
> > 
> > 	drivers/s390/cio/css.c:707 css_generate_pgid()
> > 	warn: impossible condition '(css->cssid < 0) => (0-255 < 0)'
> > 
> 
> Thanks for your report. This issue was already known and should be fixed
> by now. What's puzzling me, is that c8500e120b81 actually is the fixed
> version where cssid is a signed int. So with commit c8500e120b81 you
> should no longer see this warning.
> 

Huh.  You're right.  I didn't read the code carefully...  :/  Sorry for
the noise.

I'm not sure how this error was generated either because I can't
reproduce it today.

regards,
dan carpenter

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-01-16 13:18 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <alpine.LFD.2.20.1701161356550.1815@schleppi>
2017-01-16 13:18 ` [bug report] s390/cio: use cssid for pgid generation Dan Carpenter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.