[Accel-config] Re: [PATCH v2 0/5] accel-config: Add wq ats_disable support

Thomas, Ramesh <ramesh.thomas at intel.com>
Newsgroups dev.linux.lists.accel-config
Message-ID <BYAPR11MB2535F4EEFC853E8941F9CDA2ED179@BYAPR11MB2535.namprd11.prod.outlook.com>
The change looks ok to me.

-Ramesh

On Fri, Jul 09, 2021 at 01:18:21PM +0800, Xiaochen Shen <xiaochen.shen(a)intel.com> wrote:
> Upstream kernel 5.11 introduced ATS disable knob for work queues in
> commit 92de5fa2dc39 ("dmaengine: idxd: add ATS disable knob for work
> queues"). Accel-config needs to add support to match kernel driver
> change.
> 
> Add wq ats_disable support in library, configuration file, json
> interface, and config-wq command.
> Add unit test and documentation for wq option ats_disable.
> 
> Changelog:
> v2:
> - Rebase to the latest in the pending branch (v3.3).
> 
> Xiaochen Shen (5):
>   accel-config: Add ats_disable wq function in lib
>   accel-config: Add ats_disable in config and json
>   accel-config: Add ats_disable attr to config-wq command
>   accel-config: Add ats_disable unit test
>   accel-config: Add documentation for wq option ats-disable
> 
>  Documentation/accfg/accel-config-config-wq.txt |  6 ++++++
>  accfg/config.c                                 |  1 +
>  accfg/config_attr.c                            | 16 ++++++++++++++++
>  accfg/lib/libaccel-config.sym                  |  6 ++++++
>  accfg/lib/libaccfg.c                           |  3 +++
>  accfg/lib/private.h                            |  1 +
>  accfg/libaccel_config.h                        |  3 +++
>  test/dsa.c                                     |  1 +
>  test/dsa.h                                     |  1 +
>  test/libaccfg.c                                |  9 +++++++++
>  util/json.c                                    |  4 ++++
>  11 files changed, 51 insertions(+)
> 
> --
> 1.8.3.1
> _______________________________________________
> Accel-config mailing list -- accel-config(a)lists.01.org
> To unsubscribe send an email to accel-config-leave(a)lists.01.org
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.