diff options
| author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-12-16 19:11:28 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-12-16 19:11:28 -0500 |
| commit | 138a6d7ea2deb3080116a4f99019e181b62e5ce5 (patch) | |
| tree | 424622059405352935fbf1dde015a1e130b56198 /drivers/misc/mic/host | |
| parent | 4fc6069e7df873bda9baabe0fa9f0be23363159a (diff) | |
| parent | 319e2e3f63c348a9b66db4667efa73178e18b17d (diff) | |
Merge 3.13-rc4 into char-misc-next
We want these fixes in here.
Diffstat (limited to 'drivers/misc/mic/host')
| -rw-r--r-- | drivers/misc/mic/host/mic_boot.c | 2 | ||||
| -rw-r--r-- | drivers/misc/mic/host/mic_virtio.c | 30 | ||||
| -rw-r--r-- | drivers/misc/mic/host/mic_x100.c | 4 |
3 files changed, 18 insertions, 18 deletions
diff --git a/drivers/misc/mic/host/mic_boot.c b/drivers/misc/mic/host/mic_boot.c index 7558d9186438..b75c6b5cc20f 100644 --- a/drivers/misc/mic/host/mic_boot.c +++ b/drivers/misc/mic/host/mic_boot.c | |||
| @@ -62,7 +62,7 @@ void mic_bootparam_init(struct mic_device *mdev) | |||
| 62 | { | 62 | { |
| 63 | struct mic_bootparam *bootparam = mdev->dp; | 63 | struct mic_bootparam *bootparam = mdev->dp; |
| 64 | 64 | ||
| 65 | bootparam->magic = MIC_MAGIC; | 65 | bootparam->magic = cpu_to_le32(MIC_MAGIC); |
| 66 | bootparam->c2h_shutdown_db = mdev->shutdown_db; | 66 | bootparam->c2h_shutdown_db = mdev->shutdown_db; |
| 67 | bootparam->h2c_shutdown_db = -1; | 67 | bootparam->h2c_shutdown_db = -1; |
| 68 | bootparam->h2c_config_db = -1; | 68 | bootparam->h2c_config_db = -1; |
diff --git a/drivers/misc/mic/host/mic_virtio.c b/drivers/misc/mic/host/mic_virtio.c index 8d3c21cd1d9c..752ff873f891 100644 --- a/drivers/misc/mic/host/mic_virtio.c +++ b/drivers/misc/mic/host/mic_virtio.c | |||
| @@ -41,7 +41,7 @@ static int mic_virtio_copy_to_user(struct mic_vdev *mvdev, | |||
| 41 | * We are copying from IO below an should ideally use something | 41 | * We are copying from IO below an should ideally use something |
| 42 | * like copy_to_user_fromio(..) if it existed. | 42 | * like copy_to_user_fromio(..) if it existed. |
| 43 | */ | 43 | */ |
| 44 | if (copy_to_user(ubuf, dbuf, len)) { | 44 | if (copy_to_user(ubuf, (void __force *)dbuf, len)) { |
| 45 | err = -EFAULT; | 45 | err = -EFAULT; |
| 46 | dev_err(mic_dev(mvdev), "%s %d err %d\n", | 46 | dev_err(mic_dev(mvdev), "%s %d err %d\n", |
| 47 | __func__, __LINE__, err); | 47 | __func__, __LINE__, err); |
| @@ -66,7 +66,7 @@ static int mic_virtio_copy_from_user(struct mic_vdev *mvdev, | |||
| 66 | * We are copying to IO below and should ideally use something | 66 | * We are copying to IO below and should ideally use something |
| 67 | * like copy_from_user_toio(..) if it existed. | 67 | * like copy_from_user_toio(..) if it existed. |
| 68 | */ | 68 | */ |
| 69 | if (copy_from_user(dbuf, ubuf, len)) { | 69 | if (copy_from_user((void __force *)dbuf, ubuf, len)) { |
| 70 | err = -EFAULT; | 70 | err = -EFAULT; |
| 71 | dev_err(mic_dev(mvdev), "%s %d err %d\n", | 71 | dev_err(mic_dev(mvdev), "%s %d err %d\n", |
| 72 | __func__, __LINE__, err); | 72 | __func__, __LINE__, err); |
| @@ -293,7 +293,7 @@ static void mic_virtio_init_post(struct mic_vdev *mvdev) | |||
| 293 | continue; | 293 | continue; |
| 294 | } | 294 | } |
| 295 | mvdev->mvr[i].vrh.vring.used = | 295 | mvdev->mvr[i].vrh.vring.used = |
| 296 | mvdev->mdev->aper.va + | 296 | (void __force *)mvdev->mdev->aper.va + |
| 297 | le64_to_cpu(vqconfig[i].used_address); | 297 | le64_to_cpu(vqconfig[i].used_address); |
| 298 | } | 298 | } |
| 299 | 299 | ||
| @@ -378,7 +378,7 @@ int mic_virtio_config_change(struct mic_vdev *mvdev, | |||
| 378 | void __user *argp) | 378 | void __user *argp) |
| 379 | { | 379 | { |
| 380 | DECLARE_WAIT_QUEUE_HEAD_ONSTACK(wake); | 380 | DECLARE_WAIT_QUEUE_HEAD_ONSTACK(wake); |
| 381 | int ret = 0, retry = 100, i; | 381 | int ret = 0, retry, i; |
| 382 | struct mic_bootparam *bootparam = mvdev->mdev->dp; | 382 | struct mic_bootparam *bootparam = mvdev->mdev->dp; |
| 383 | s8 db = bootparam->h2c_config_db; | 383 | s8 db = bootparam->h2c_config_db; |
| 384 | 384 | ||
| @@ -401,7 +401,7 @@ int mic_virtio_config_change(struct mic_vdev *mvdev, | |||
| 401 | mvdev->dc->config_change = MIC_VIRTIO_PARAM_CONFIG_CHANGED; | 401 | mvdev->dc->config_change = MIC_VIRTIO_PARAM_CONFIG_CHANGED; |
| 402 | mvdev->mdev->ops->send_intr(mvdev->mdev, db); | 402 | mvdev->mdev->ops->send_intr(mvdev->mdev, db); |
| 403 | 403 | ||
| 404 | for (i = retry; i--;) { | 404 | for (retry = 100; retry--;) { |
| 405 | ret = wait_event_timeout(wake, | 405 | ret = wait_event_timeout(wake, |
| 406 | mvdev->dc->guest_ack, msecs_to_jiffies(100)); | 406 | mvdev->dc->guest_ack, msecs_to_jiffies(100)); |
| 407 | if (ret) | 407 | if (ret) |
| @@ -467,7 +467,7 @@ static int mic_copy_dp_entry(struct mic_vdev *mvdev, | |||
| 467 | } | 467 | } |
| 468 | 468 | ||
| 469 | /* Find the first free device page entry */ | 469 | /* Find the first free device page entry */ |
| 470 | for (i = mic_aligned_size(struct mic_bootparam); | 470 | for (i = sizeof(struct mic_bootparam); |
| 471 | i < MIC_DP_SIZE - mic_total_desc_size(dd_config); | 471 | i < MIC_DP_SIZE - mic_total_desc_size(dd_config); |
| 472 | i += mic_total_desc_size(devp)) { | 472 | i += mic_total_desc_size(devp)) { |
| 473 | devp = mdev->dp + i; | 473 | devp = mdev->dp + i; |
| @@ -525,6 +525,7 @@ int mic_virtio_add_device(struct mic_vdev *mvdev, | |||
| 525 | char irqname[10]; | 525 | char irqname[10]; |
| 526 | struct mic_bootparam *bootparam = mdev->dp; | 526 | struct mic_bootparam *bootparam = mdev->dp; |
| 527 | u16 num; | 527 | u16 num; |
| 528 | dma_addr_t vr_addr; | ||
| 528 | 529 | ||
| 529 | mutex_lock(&mdev->mic_mutex); | 530 | mutex_lock(&mdev->mic_mutex); |
| 530 | 531 | ||
| @@ -559,17 +560,16 @@ int mic_virtio_add_device(struct mic_vdev *mvdev, | |||
| 559 | } | 560 | } |
| 560 | vr->len = vr_size; | 561 | vr->len = vr_size; |
| 561 | vr->info = vr->va + vring_size(num, MIC_VIRTIO_RING_ALIGN); | 562 | vr->info = vr->va + vring_size(num, MIC_VIRTIO_RING_ALIGN); |
| 562 | vr->info->magic = MIC_MAGIC + mvdev->virtio_id + i; | 563 | vr->info->magic = cpu_to_le32(MIC_MAGIC + mvdev->virtio_id + i); |
| 563 | vqconfig[i].address = mic_map_single(mdev, | 564 | vr_addr = mic_map_single(mdev, vr->va, vr_size); |
| 564 | vr->va, vr_size); | 565 | if (mic_map_error(vr_addr)) { |
| 565 | if (mic_map_error(vqconfig[i].address)) { | ||
| 566 | free_pages((unsigned long)vr->va, get_order(vr_size)); | 566 | free_pages((unsigned long)vr->va, get_order(vr_size)); |
| 567 | ret = -ENOMEM; | 567 | ret = -ENOMEM; |
| 568 | dev_err(mic_dev(mvdev), "%s %d err %d\n", | 568 | dev_err(mic_dev(mvdev), "%s %d err %d\n", |
| 569 | __func__, __LINE__, ret); | 569 | __func__, __LINE__, ret); |
| 570 | goto err; | 570 | goto err; |
| 571 | } | 571 | } |
| 572 | vqconfig[i].address = cpu_to_le64(vqconfig[i].address); | 572 | vqconfig[i].address = cpu_to_le64(vr_addr); |
| 573 | 573 | ||
| 574 | vring_init(&vr->vr, num, vr->va, MIC_VIRTIO_RING_ALIGN); | 574 | vring_init(&vr->vr, num, vr->va, MIC_VIRTIO_RING_ALIGN); |
| 575 | ret = vringh_init_kern(&mvr->vrh, | 575 | ret = vringh_init_kern(&mvr->vrh, |
| @@ -639,7 +639,7 @@ void mic_virtio_del_device(struct mic_vdev *mvdev) | |||
| 639 | struct mic_vdev *tmp_mvdev; | 639 | struct mic_vdev *tmp_mvdev; |
| 640 | struct mic_device *mdev = mvdev->mdev; | 640 | struct mic_device *mdev = mvdev->mdev; |
| 641 | DECLARE_WAIT_QUEUE_HEAD_ONSTACK(wake); | 641 | DECLARE_WAIT_QUEUE_HEAD_ONSTACK(wake); |
| 642 | int i, ret, retry = 100; | 642 | int i, ret, retry; |
| 643 | struct mic_vqconfig *vqconfig; | 643 | struct mic_vqconfig *vqconfig; |
| 644 | struct mic_bootparam *bootparam = mdev->dp; | 644 | struct mic_bootparam *bootparam = mdev->dp; |
| 645 | s8 db; | 645 | s8 db; |
| @@ -652,16 +652,16 @@ void mic_virtio_del_device(struct mic_vdev *mvdev) | |||
| 652 | "Requesting hot remove id %d\n", mvdev->virtio_id); | 652 | "Requesting hot remove id %d\n", mvdev->virtio_id); |
| 653 | mvdev->dc->config_change = MIC_VIRTIO_PARAM_DEV_REMOVE; | 653 | mvdev->dc->config_change = MIC_VIRTIO_PARAM_DEV_REMOVE; |
| 654 | mdev->ops->send_intr(mdev, db); | 654 | mdev->ops->send_intr(mdev, db); |
| 655 | for (i = retry; i--;) { | 655 | for (retry = 100; retry--;) { |
| 656 | ret = wait_event_timeout(wake, | 656 | ret = wait_event_timeout(wake, |
| 657 | mvdev->dc->guest_ack, msecs_to_jiffies(100)); | 657 | mvdev->dc->guest_ack, msecs_to_jiffies(100)); |
| 658 | if (ret) | 658 | if (ret) |
| 659 | break; | 659 | break; |
| 660 | } | 660 | } |
| 661 | dev_dbg(mdev->sdev->parent, | 661 | dev_dbg(mdev->sdev->parent, |
| 662 | "Device id %d config_change %d guest_ack %d\n", | 662 | "Device id %d config_change %d guest_ack %d retry %d\n", |
| 663 | mvdev->virtio_id, mvdev->dc->config_change, | 663 | mvdev->virtio_id, mvdev->dc->config_change, |
| 664 | mvdev->dc->guest_ack); | 664 | mvdev->dc->guest_ack, retry); |
| 665 | mvdev->dc->config_change = 0; | 665 | mvdev->dc->config_change = 0; |
| 666 | mvdev->dc->guest_ack = 0; | 666 | mvdev->dc->guest_ack = 0; |
| 667 | skip_hot_remove: | 667 | skip_hot_remove: |
diff --git a/drivers/misc/mic/host/mic_x100.c b/drivers/misc/mic/host/mic_x100.c index 6fccb13493a5..5562fdd3ef4e 100644 --- a/drivers/misc/mic/host/mic_x100.c +++ b/drivers/misc/mic/host/mic_x100.c | |||
| @@ -400,8 +400,8 @@ mic_x100_load_ramdisk(struct mic_device *mdev) | |||
| 400 | * so copy over the ramdisk @ 128M. | 400 | * so copy over the ramdisk @ 128M. |
| 401 | */ | 401 | */ |
| 402 | memcpy_toio(mdev->aper.va + (mdev->bootaddr << 1), fw->data, fw->size); | 402 | memcpy_toio(mdev->aper.va + (mdev->bootaddr << 1), fw->data, fw->size); |
| 403 | iowrite32(cpu_to_le32(mdev->bootaddr << 1), &bp->hdr.ramdisk_image); | 403 | iowrite32(mdev->bootaddr << 1, &bp->hdr.ramdisk_image); |
| 404 | iowrite32(cpu_to_le32(fw->size), &bp->hdr.ramdisk_size); | 404 | iowrite32(fw->size, &bp->hdr.ramdisk_size); |
| 405 | release_firmware(fw); | 405 | release_firmware(fw); |
| 406 | error: | 406 | error: |
| 407 | return rc; | 407 | return rc; |
