lkml.org 
[lkml]   [2020]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.9 716/757] soundwire: cadence: fix race condition between suspend and Slave device alerts
    Date
    From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

    [ Upstream commit d2068da5c85697b5880483dd7beaba98e0b62e02 ]

    In system suspend stress cases, the SOF CI reports timeouts. The root
    cause is that an alert is generated while the system suspends. The
    interrupt handling generates transactions on the bus that will never
    be handled because the interrupts are disabled in parallel.

    As a result, the transaction never completes and times out on resume.
    This error doesn't seem too problematic since it happens in a work
    queue, and the system recovers without issues.

    Nevertheless, this race condition should not happen. When doing a
    system suspend, or when disabling interrupts, we should make sure the
    current transaction can complete, and prevent new work from being
    queued.

    BugLink: https://github.com/thesofproject/linux/issues/2344
    Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
    Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
    Reviewed-by: Rander Wang <rander.wang@linux.intel.com>
    Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
    Acked-by: Jaroslav Kysela <perex@perex.cz>
    Link: https://lore.kernel.org/r/20200817222340.18042-1-yung-chuan.liao@linux.intel.com
    Signed-off-by: Vinod Koul <vkoul@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/soundwire/cadence_master.c | 24 +++++++++++++++++++++++-
    drivers/soundwire/cadence_master.h | 1 +
    2 files changed, 24 insertions(+), 1 deletion(-)

    diff --git a/drivers/soundwire/cadence_master.c b/drivers/soundwire/cadence_master.c
    index 24eafe0aa1c3e..1330ffc475960 100644
    --- a/drivers/soundwire/cadence_master.c
    +++ b/drivers/soundwire/cadence_master.c
    @@ -791,7 +791,16 @@ irqreturn_t sdw_cdns_irq(int irq, void *dev_id)
    CDNS_MCP_INT_SLAVE_MASK, 0);

    int_status &= ~CDNS_MCP_INT_SLAVE_MASK;
    - schedule_work(&cdns->work);
    +
    + /*
    + * Deal with possible race condition between interrupt
    + * handling and disabling interrupts on suspend.
    + *
    + * If the master is in the process of disabling
    + * interrupts, don't schedule a workqueue
    + */
    + if (cdns->interrupt_enabled)
    + schedule_work(&cdns->work);
    }

    cdns_writel(cdns, CDNS_MCP_INTSTAT, int_status);
    @@ -924,6 +933,19 @@ int sdw_cdns_enable_interrupt(struct sdw_cdns *cdns, bool state)
    slave_state = cdns_readl(cdns, CDNS_MCP_SLAVE_INTSTAT1);
    cdns_writel(cdns, CDNS_MCP_SLAVE_INTSTAT1, slave_state);
    }
    + cdns->interrupt_enabled = state;
    +
    + /*
    + * Complete any on-going status updates before updating masks,
    + * and cancel queued status updates.
    + *
    + * There could be a race with a new interrupt thrown before
    + * the 3 mask updates below are complete, so in the interrupt
    + * we use the 'interrupt_enabled' status to prevent new work
    + * from being queued.
    + */
    + if (!state)
    + cancel_work_sync(&cdns->work);

    cdns_writel(cdns, CDNS_MCP_SLAVE_INTMASK0, slave_intmask0);
    cdns_writel(cdns, CDNS_MCP_SLAVE_INTMASK1, slave_intmask1);
    diff --git a/drivers/soundwire/cadence_master.h b/drivers/soundwire/cadence_master.h
    index 7638858397df9..15b0834030866 100644
    --- a/drivers/soundwire/cadence_master.h
    +++ b/drivers/soundwire/cadence_master.h
    @@ -129,6 +129,7 @@ struct sdw_cdns {

    bool link_up;
    unsigned int msg_count;
    + bool interrupt_enabled;

    struct work_struct work;

    --
    2.25.1


    \
     
     \ /
      Last update: 2020-10-27 17:40    [W:2.566 / U:0.492 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site