lkml.org 
[lkml]   [2021]   [Sep]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 040/306] s390/qdio: cancel the ESTABLISH ccw after timeout
    Date
    From: Julian Wiedmann <jwi@linux.ibm.com>

    commit 1c1dc8bda3a05c60877a6649775894db5343bdea upstream.

    When the ESTABLISH ccw does not complete within the specified timeout,
    try our best to cancel the ccw program that is still active on the
    device. Otherwise the IO subsystem might be accessing it even after
    the driver eg. called qdio_free().

    Fixes: 779e6e1c724d ("[S390] qdio: new qdio driver.")
    Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
    Reviewed-by: Benjamin Block <bblock@linux.ibm.com>
    Cc: <stable@vger.kernel.org> # 2.6.27
    Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/s390/cio/qdio_main.c | 51 +++++++++++++++++++++++++------------------
    1 file changed, 30 insertions(+), 21 deletions(-)

    --- a/drivers/s390/cio/qdio_main.c
    +++ b/drivers/s390/cio/qdio_main.c
    @@ -1025,6 +1025,33 @@ static void qdio_shutdown_queues(struct
    }
    }

    +static int qdio_cancel_ccw(struct qdio_irq *irq, int how)
    +{
    + struct ccw_device *cdev = irq->cdev;
    + int rc;
    +
    + spin_lock_irq(get_ccwdev_lock(cdev));
    + qdio_set_state(irq, QDIO_IRQ_STATE_CLEANUP);
    + if (how & QDIO_FLAG_CLEANUP_USING_CLEAR)
    + rc = ccw_device_clear(cdev, QDIO_DOING_CLEANUP);
    + else
    + /* default behaviour is halt */
    + rc = ccw_device_halt(cdev, QDIO_DOING_CLEANUP);
    + spin_unlock_irq(get_ccwdev_lock(cdev));
    + if (rc) {
    + DBF_ERROR("%4x SHUTD ERR", irq->schid.sch_no);
    + DBF_ERROR("rc:%4d", rc);
    + return rc;
    + }
    +
    + wait_event_interruptible_timeout(cdev->private->wait_q,
    + irq->state == QDIO_IRQ_STATE_INACTIVE ||
    + irq->state == QDIO_IRQ_STATE_ERR,
    + 10 * HZ);
    +
    + return 0;
    +}
    +
    /**
    * qdio_shutdown - shut down a qdio subchannel
    * @cdev: associated ccw device
    @@ -1063,27 +1090,7 @@ int qdio_shutdown(struct ccw_device *cde
    qdio_shutdown_queues(irq_ptr);
    qdio_shutdown_debug_entries(irq_ptr);

    - /* cleanup subchannel */
    - spin_lock_irq(get_ccwdev_lock(cdev));
    - qdio_set_state(irq_ptr, QDIO_IRQ_STATE_CLEANUP);
    - if (how & QDIO_FLAG_CLEANUP_USING_CLEAR)
    - rc = ccw_device_clear(cdev, QDIO_DOING_CLEANUP);
    - else
    - /* default behaviour is halt */
    - rc = ccw_device_halt(cdev, QDIO_DOING_CLEANUP);
    - spin_unlock_irq(get_ccwdev_lock(cdev));
    - if (rc) {
    - DBF_ERROR("%4x SHUTD ERR", irq_ptr->schid.sch_no);
    - DBF_ERROR("rc:%4d", rc);
    - goto no_cleanup;
    - }
    -
    - wait_event_interruptible_timeout(cdev->private->wait_q,
    - irq_ptr->state == QDIO_IRQ_STATE_INACTIVE ||
    - irq_ptr->state == QDIO_IRQ_STATE_ERR,
    - 10 * HZ);
    -
    -no_cleanup:
    + rc = qdio_cancel_ccw(irq_ptr, how);
    qdio_shutdown_thinint(irq_ptr);
    qdio_shutdown_irq(irq_ptr);

    @@ -1316,10 +1323,12 @@ int qdio_establish(struct ccw_device *cd
    return 0;

    err_ccw_timeout:
    + qdio_cancel_ccw(irq_ptr, QDIO_FLAG_CLEANUP_USING_CLEAR);
    err_ccw_start:
    qdio_shutdown_thinint(irq_ptr);
    err_thinint:
    qdio_shutdown_irq(irq_ptr);
    + qdio_set_state(irq_ptr, QDIO_IRQ_STATE_INACTIVE);
    mutex_unlock(&irq_ptr->setup_mutex);
    return rc;
    }

    \
     
     \ /
      Last update: 2021-09-16 18:12    [W:4.402 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site