diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-01-07 01:30:08 -0500 |
|---|---|---|
| committer | Alex Williamson <alex.williamson@redhat.com> | 2017-01-11 14:12:37 -0500 |
| commit | 73da4268fdbae972f617946d1c690f2136964802 (patch) | |
| tree | 8a7c3341dd0456146c5e93d91a3dbd8b33f7b30a /samples/vfio-mdev | |
| parent | 5c677869e0abbffbade2cfd82d46d0eebe823f34 (diff) | |
vfio-mdev: remove some dead code
We set info.count to 1 in mtty_get_irq_info() so static checkers
complain that, "Why do we have impossible conditions?" The answer is
that it seems to be left over dead code that can be safely removed.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Kirti Wankhede <kwankhede@nvidia.com>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Diffstat (limited to 'samples/vfio-mdev')
| -rw-r--r-- | samples/vfio-mdev/mtty.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/samples/vfio-mdev/mtty.c b/samples/vfio-mdev/mtty.c index 382f4797428f..ca495686b9c3 100644 --- a/samples/vfio-mdev/mtty.c +++ b/samples/vfio-mdev/mtty.c | |||
| @@ -1230,9 +1230,6 @@ static long mtty_ioctl(struct mdev_device *mdev, unsigned int cmd, | |||
| 1230 | if (ret) | 1230 | if (ret) |
| 1231 | return ret; | 1231 | return ret; |
| 1232 | 1232 | ||
| 1233 | if (info.count == -1) | ||
| 1234 | return -EINVAL; | ||
| 1235 | |||
| 1236 | if (copy_to_user((void __user *)arg, &info, minsz)) | 1233 | if (copy_to_user((void __user *)arg, &info, minsz)) |
| 1237 | return -EFAULT; | 1234 | return -EFAULT; |
| 1238 | 1235 | ||
