[Accel-config] Re: [PATCH v1 0/7] Refactor error handling at enable/disable wq/dev
Dave Jiang <dave.jiang at intel.com>
| Newsgroups | dev.linux.lists.accel-config |
|---|---|
| Message-ID | <[email protected]> |
On 3/2/2021 10:44 AM, ramesh.thomas(a)intel.com wrote: > From: Ramesh Thomas <ramesh.thomas(a)intel.com> > > Functions enabliing/disabling workqueues and devices had several issues > with status return and printing of error messages. Fix the return value > issues and refactor code to make error handling simpler. > > Ramesh Thomas (7): > accel-config: Fix wrong return values in accfg_device_get_cmd_status > accel-config: Refactor accfg_device_get_cmd_status_str > accel-config: Add 0 = "Successful completion" cmd status array > accel-config: Refactor handling printing of device cmd status > accel-config: Print device command status only on errors > accel-config: Remove redundant check of wq state > accel-config: Fix and refactor status return and printing > > accfg/enable.c | 125 ++++++++++++++----------------------------- > accfg/lib/libaccfg.c | 30 ++++------- > 2 files changed, 50 insertions(+), 105 deletions(-) Reviewed-by: Dave Jiang <dave.jiang(a)intel.com> for the series.