在 2023/2/9 16:48, David Marchand 写道:
On Thu, Feb 9, 2023 at 4:24 AM fengchengwen <fengcheng...@huawei.com> wrote:
Acked-by: Chengwen Feng <fengcheng...@huawei.com>
And, how abort add error log in telemetry valid_name, so so that problems can
be identified as early as possible.
On 2023/2/9 11:03, Huisong Li wrote:
The telemetry lib has added a allowed characters set for dictionary names,
See commit
2537fb0c5f34 ("telemetry: limit characters allowed in dictionary names")
The space is not in this set, which cause the heap ID in /eal/heap_info
cannot be displayed. Additionally, 'heap' is also misspelling. So use
I agree this typo is ugly, but I wonder if some telemetry users
started relying on it...
we should have gotten feedback earlier if telemetry user relied on it.
Because this field has not been displayed since September last year.
Or do we add an announcement in rel_notes?
'Heap_id' to replace 'Head id'.
Fixes: e6732d0d6e26 ("mem: add telemetry infos")
IIUC, the commit that broke displaying "Head id" is actually the one
that limited the set of chars.
Ack, I will add the following commit:
Fixes: 2537fb0c5f34 ("telemetry: limit characters allowed in dictionary
names")
Cc: sta...@dpdk.org
Signed-off-by: Huisong Li <lihuis...@huawei.com>
