public inbox for linux-s390@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox