RE: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: RE: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
From: "Yu, Fenghua" <fenghua.yu@xxxxxxxxx>
Date: Wed, 24 Aug 2022 21:11:59 +0000
Accept-language: en-US
Dlp-product: dlpe-windows
Dlp-reaction: no-action
Dlp-version: 11.6.500.17
In-reply-to: <38e416b47bb30fa161e52f24ecbcf95015480fed.camel@redhat.com>
References: <20220823162435.2099389-1-jsnitsel@redhat.com> <20220823163709.2102468-1-jsnitsel@redhat.com> <905d3feb-f75b-e91c-f3de-b69718aa5c69@intel.com> <20220824005435.jyexxvjxj3z7tc2f@cantor> <223e5a43-95a5-da54-0ff7-c2e088a072e3@intel.com> <38e416b47bb30fa161e52f24ecbcf95015480fed.camel@redhat.com>

Hi, Jerry,

> I see another potential issue. If a software reset is attempted idxd_device_reinit()
> will be called which walks the wqs, and if a wq has the state
> IDXD_WQ_ENABLED it calls idxd_wq_enable(), but the first thing
> idxd_wq_enable() does is see that the state is IDXD_WQ_ENABLED and returns 0.
> Without the wq enable command being sent, it will not be re-enabled, yes?

Could you please describe how to reproduce the issues and test case?

Thanks.

-Fenghua

Follow-Ups:
- Re: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
  - From: Jerry Snitselaar

References:
- [PATCH] idxd: avoid deadlock in process_misc_interrupts()
  - From: Jerry Snitselaar
- [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
  - From: Jerry Snitselaar
- Re: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
  - From: Dave Jiang
- Re: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
  - From: Jerry Snitselaar
- Re: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
  - From: Dave Jiang
- Re: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
  - From: Jerry Snitselaar

Prev by Date: Re: [PATCH] dmaengine: idxd: Set workqueue state to disabled before trying to re-enable
Next by Date: Re: [PATCH] dmaengine: idxd: Set workqueue state to disabled before trying to re-enable
Previous by thread: Re: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
Next by thread: Re: [PATCH v2] dmaengine: idxd: avoid deadlock in process_misc_interrupts()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel] [Linux ARM (vger)] [Linux ARM MSM] [Linux Omap] [Linux Arm] [Linux Tegra] [Fedora ARM] [Linux for Samsung SOC] [eCos] [Linux PCI] [Linux Fastboot] [Gcc Help] [Git] [DCCP] [IETF Announce] [Security] [Linux MIPS] [Yosemite Campsites]