[PATCH v1] raw/ifpga: fix monitor thread issues

Huang, Wei wei.huang at intel.com
Tue Mar 8 02:39:12 CET 2022


Hi Thomas,

> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Tuesday, March 8, 2022 06:30
> To: Zhang, Tianfei <tianfei.zhang at intel.com>; Huang, Wei
> <wei.huang at intel.com>
> Cc: dev at dpdk.org; Xu, Rosen <rosen.xu at intel.com>; Zhang, Qi Z
> <qi.z.zhang at intel.com>; nipun.gupta at nxp.com; hemant.agrawal at nxp.com;
> stable at dpdk.org; Yigit, Ferruh <ferruh.yigit at intel.com>
> Subject: Re: [PATCH v1] raw/ifpga: fix monitor thread issues
> 
> 01/03/2022 09:47, Wei Huang:
> > Monitor thread handles graceful shutdown according to the value of
> > specific sensors in device, two issues are found below.
> > 1. Thread is not created when card is probed.
> > 2. Thread is canceled without checking presence of other cards.
> > To fix them, thread is created in pci device probe function, a
> > reference count is checked before canceling the thread.
> >
> > Fixes: 9c006c45 ("raw/ifpga: scan PCIe BDF device tree")
> > Cc: stable at dpdk.org
> >
> > Signed-off-by: Wei Huang <wei.huang at intel.com>
> > Acked-by: Tianfei Zhang <tianfei.zhang at intel.com>
> 
> In general, it is better to have 2 separate patches for 2 different issues.
> 

These two issue are related, if I split them into two patches, the thread cannot work correctly with any one of the patches applied.

> Applied
> 
> 



More information about the stable mailing list