UCT/IB: Add auto to the list of DEVX objects

[tvegas1]

What?

Add auto to the list of DEVX objects. This configuration is used on Grace primarily (#11010).

Why?

Fixes parser issue:

parser.c:1375 UCX  ERROR Invalid value for MLX5_DEVX_OBJECTS: 'auto'. Expected: comma-separated list of: [rcqp|rcsrq|dct|dcsrq|dci|cq]
uct_md.c:272  UCX  ERROR Failed to read MD config

[roiedanino]

Maybe we can extend the config type to accept auto? Or create a dedicated config type for that?

[tvegas1]

AFAIU it is already added in #11010, just that parser did not allow it.

[roiedanino]

Yes, what I meant was that CONFIG_TYPE_BITMAP or CONFIG_TYPE_BITMAP_AUTO will allow auto value under the hood, so there will be no need to list it as a devx object; the config type will imply it.

[guy-ealey-morag]

It looks like "auto" is just one of the bitmap values so why treat it differently in the parser?

static const char *uct_ib_devx_objs[] = {
    [UCT_IB_DEVX_OBJ_RCQP]  = "rcqp",
    [UCT_IB_DEVX_OBJ_RCSRQ] = "rcsrq",
    [UCT_IB_DEVX_OBJ_DCT]   = "dct",
    [UCT_IB_DEVX_OBJ_DCSRQ] = "dcsrq",
    [UCT_IB_DEVX_OBJ_DCI]   = "dci",
    [UCT_IB_DEVX_OBJ_CQ]    = "cq",
    [UCT_IB_DEVX_OBJ_AUTO]  = "auto",
    NULL
};

The only thing I can suggest is to have a parser that checks directly with the values of uct_ib_devx_objs instead of the currently separate values string "rcqp,rcsrq,dct,dcsrq,dci,cq,auto"

[tvegas1]

wait, need to rework that pr.

[tvegas1]

modified list in this PR is actually the default flag list. it does not make sense to add auto here, original issue is actually a setup issue where a ucx.conf with auto is used with older library.