lkml.org 
[lkml]   [2018]   [Jul]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3/4] ntb: idt: Discard temperature sensor IRQ handler
    Date
    IDT PCIe-switch temperature sensor interface is very broken. First
    of all only a few combinations of TMPCTL threshold enable bits
    really cause the interrupts unmasked. Even if an individual bit
    indicates the event unmasked, corresponding IRQ just isn't generated.
    Most of the threshold enable bits combinations are in fact useless and
    non of them can help to create a fully functional alarm interface.
    So to speak, we can't create a well defined hwmon alarms based on
    the IDT PCI-switch threshold IRQs.

    Secondly a single threshold IRQ (not a combination of thresholds) can
    be successfully enabled without the issue described above. But in this
    case we experienced an enormous number of interrupts generated by
    the chip if the temperature got near the enabled threshold value. Filter
    adjustment didn't help much. It also doesn't provide a hysteresis settings.
    Due to the temperature sample fluctuations near the threshold the
    interrupts spate makes the system nearly unusable until the temperature
    value finally settled so being pushed either to be fully higher or lower
    the threshold.

    All of these issues makes the temperature sensor alarm interface useless
    and even at some point dangerous to be used in the driver. In this case
    it is safer to completely discard it and disable the temperature alarm
    interrupts.

    Signed-off-by: Serge Semin <fancer.lancer@gmail.com>
    ---
    drivers/ntb/hw/idt/ntb_hw_idt.c | 41 +----------------------------------------
    drivers/ntb/hw/idt/ntb_hw_idt.h | 5 ++---
    2 files changed, 3 insertions(+), 43 deletions(-)

    diff --git a/drivers/ntb/hw/idt/ntb_hw_idt.c b/drivers/ntb/hw/idt/ntb_hw_idt.c
    index af767a13556a..3d48267ae315 100644
    --- a/drivers/ntb/hw/idt/ntb_hw_idt.c
    +++ b/drivers/ntb/hw/idt/ntb_hw_idt.c
    @@ -2076,38 +2076,6 @@ static struct attribute *idt_temp_attrs[] = {
    ATTRIBUTE_GROUPS(idt_temp);

    /*
    - * idt_temp_isr() - temperature sensor alarm events ISR
    - * @ndev: IDT NTB hardware driver descriptor
    - * @ntint_sts: NT-function interrupt status
    - *
    - * It handles events of temperature crossing alarm thresholds. Since reading
    - * of TMPALARM register clears it up, the function doesn't analyze the
    - * read value, instead the current temperature value just warningly printed to
    - * log.
    - * The method is called from PCIe ISR bottom-half routine.
    - */
    -static void idt_temp_isr(struct idt_ntb_dev *ndev, u32 ntint_sts)
    -{
    - unsigned long mdeg;
    -
    - /* Read the current temperature value */
    - idt_read_temp(ndev, IDT_TEMP_CUR, &mdeg);
    -
    - /* Read the temperature alarm to clean the alarm status out */
    - /*(void)idt_sw_read(ndev, IDT_SW_TMPALARM);*/
    -
    - /* Clean the corresponding interrupt bit */
    - idt_nt_write(ndev, IDT_NT_NTINTSTS, IDT_NTINTSTS_TMPSENSOR);
    -
    - dev_dbg(&ndev->ntb.pdev->dev,
    - "Temp sensor IRQ detected %#08x", ntint_sts);
    -
    - /* Print temperature value to log */
    - dev_warn(&ndev->ntb.pdev->dev, "Temperature %hhd.%hhuC",
    - idt_get_deg(mdeg), idt_get_deg_frac(mdeg));
    -}
    -
    -/*
    * idt_init_temp() - initialize temperature sensor interface
    * @ndev: IDT NTB hardware driver descriptor
    *
    @@ -2189,7 +2157,7 @@ static int idt_init_isr(struct idt_ntb_dev *ndev)
    goto err_free_vectors;
    }

    - /* Unmask Message/Doorbell/SE/Temperature interrupts */
    + /* Unmask Message/Doorbell/SE interrupts */
    ntint_mask = idt_nt_read(ndev, IDT_NT_NTINTMSK) & ~IDT_NTINTMSK_ALL;
    idt_nt_write(ndev, IDT_NT_NTINTMSK, ntint_mask);

    @@ -2204,7 +2172,6 @@ err_free_vectors:
    return ret;
    }

    -
    /*
    * idt_deinit_ist() - deinitialize PCIe interrupt handler
    * @ndev: IDT NTB hardware driver descriptor
    @@ -2265,12 +2232,6 @@ static irqreturn_t idt_thread_isr(int irq, void *devid)
    handled = true;
    }

    - /* Handle temperature sensor interrupt */
    - if (ntint_sts & IDT_NTINTSTS_TMPSENSOR) {
    - idt_temp_isr(ndev, ntint_sts);
    - handled = true;
    - }
    -
    dev_dbg(&ndev->ntb.pdev->dev, "IDT IRQs 0x%08x handled", ntint_sts);

    return handled ? IRQ_HANDLED : IRQ_NONE;
    diff --git a/drivers/ntb/hw/idt/ntb_hw_idt.h b/drivers/ntb/hw/idt/ntb_hw_idt.h
    index 032f81cb4d44..3517cd2e2baa 100644
    --- a/drivers/ntb/hw/idt/ntb_hw_idt.h
    +++ b/drivers/ntb/hw/idt/ntb_hw_idt.h
    @@ -688,15 +688,14 @@
    * @IDT_NTINTMSK_DBELL: Doorbell interrupt mask bit
    * @IDT_NTINTMSK_SEVENT: Switch Event interrupt mask bit
    * @IDT_NTINTMSK_TMPSENSOR: Temperature sensor interrupt mask bit
    - * @IDT_NTINTMSK_ALL: All the useful interrupts mask
    + * @IDT_NTINTMSK_ALL: NTB-related interrupts mask
    */
    #define IDT_NTINTMSK_MSG 0x00000001U
    #define IDT_NTINTMSK_DBELL 0x00000002U
    #define IDT_NTINTMSK_SEVENT 0x00000008U
    #define IDT_NTINTMSK_TMPSENSOR 0x00000080U
    #define IDT_NTINTMSK_ALL \
    - (IDT_NTINTMSK_MSG | IDT_NTINTMSK_DBELL | \
    - IDT_NTINTMSK_SEVENT | IDT_NTINTMSK_TMPSENSOR)
    + (IDT_NTINTMSK_MSG | IDT_NTINTMSK_DBELL | IDT_NTINTMSK_SEVENT)

    /*
    * NTGSIGNAL register fields related constants
    --
    2.12.0
    \
     
     \ /
      Last update: 2018-07-15 22:07    [W:7.902 / U:0.056 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site