Re: drivers/staging/gpib/common/gpib_os.c:871 ibioctl() warn: inconsistent returns '&board->big_gpib_mutex'.

[Dan Carpenter]

On Mon, Dec 09, 2024 at 09:50:15AM +0300, Dan Carpenter wrote:
> 9dde4559e93955 Dave Penkler 2024-09-18  757  	switch (cmd) {
> 9dde4559e93955 Dave Penkler 2024-09-18  758  	case IBEVENT:
> 9dde4559e93955 Dave Penkler 2024-09-18  759  		retval = event_ioctl(board, arg);
> 9dde4559e93955 Dave Penkler 2024-09-18  760  		goto done;
> 9dde4559e93955 Dave Penkler 2024-09-18  761  	case IBCLOSEDEV:
> 9dde4559e93955 Dave Penkler 2024-09-18  762  		retval = close_dev_ioctl(filep, board, arg);
> 9dde4559e93955 Dave Penkler 2024-09-18  763  		goto done;
> 9dde4559e93955 Dave Penkler 2024-09-18  764  	case IBOPENDEV:
> 9dde4559e93955 Dave Penkler 2024-09-18  765  		retval = open_dev_ioctl(filep, board, arg);
> 9dde4559e93955 Dave Penkler 2024-09-18  766  		goto done;
> 9dde4559e93955 Dave Penkler 2024-09-18  767  	case IBSPOLL_BYTES:
> 9dde4559e93955 Dave Penkler 2024-09-18  768  		retval = status_bytes_ioctl(board, arg);
> 9dde4559e93955 Dave Penkler 2024-09-18  769  		goto done;
> 9dde4559e93955 Dave Penkler 2024-09-18  770  	case IBWAIT:
> 9dde4559e93955 Dave Penkler 2024-09-18  771  		retval = wait_ioctl(file_priv, board, arg);
> 9dde4559e93955 Dave Penkler 2024-09-18  772  		if (retval == -ERESTARTSYS)
> 9dde4559e93955 Dave Penkler 2024-09-18  773  			return retval;
> 
> mutex_unlock(&board->big_gpib_mutex) before returning.
> 

Sorry, this is a false positive.  The kbuild-bot can't use cross function
analysis because it doesn't scale.  But if you had cross function analysis
enabled then it wouldn't have printed this false positive.

regards,
dan carpenter