Hi, -----Original Message----- From: Ferruh Yigit <[email protected]> Sent: Thursday, January 28, 2021 21:30 To: Huang, Wei <[email protected]>; [email protected]; Xu, Rosen <[email protected]>; Zhang, Qi Z <[email protected]> Cc: [email protected]; Zhang, Tianfei <[email protected]> Subject: Re: [dpdk-stable] [PATCH v12 3/4] raw/ifpga: add fpga helper function
On 1/26/2021 6:45 AM, Wei Huang wrote: > Below helper functions are used to implement OPAE application. > 1. ifpga_rawdev_lock() and ifpga_rawdev_unlock() protect FPGA access > in multiproces. Why driver needs to expose the lock APIs to the application? Can't application use its own locking mechanism for multiprocess, and driver can use its own locking for its needs. Driver exposing locking internals to applications doesn't look right, but I don't know the full usecase. I agree, these two lock APIs will be used only in rte_pmd_ifpga.c in next patch, application cannot see them. > 2. ifpga_rawdev_get_rsu_stat() and ifpga_rawdev_set_rsu_stat() record > current RSU status and progress. > 3. ifpga_rawdev_get_phy_info() get information of connected retimer. > 4. ifpga_rawdev_partial_reconfigure() do partial reconfiguration. > 5. ifpga_rawdev_cleanup() free software resources allocated for driver. > > Signed-off-by: Wei Huang <[email protected]> > Acked-by: Tianfei Zhang <[email protected]> > Acked-by: Rosen Xu <[email protected]> <...>
