Re: [PATCH] perf list: fix short description for some cache events

[Ian Rogers]

On Wed, Feb 21, 2024 at 1:20 AM Thomas Richter <tmricht@xxxxxxxxxxxxx> wrote:
>
> Correct the short description of the following events:
> DCW_REQ, DCW_REQ_CHIP_HIT, DCW_REQ_DRAWER_HIT, DCW_REQ_IV,
> DCW_ON_CHIP, DCW_ON_CHIP_IV, DCW_ON_CHIP_CHIP_HIT,
> DCW_ON_CHIP_DRAWER_HIT, CW_ON_MODULE, DCW_ON_DRAWER,
> DCW_OFF_DRAWER, IDCW_ON_MODULE_IV, IDCW_ON_MODULE_CHIP_HIT,
> IDCW_ON_MODULE_DRAWER_HIT, IDCW_ON_DRAWER_IV, IDCW_ON_DRAWER_CHIP_HIT,
> IDCW_ON_DRAWER_DRAWER_HIT, IDCW_OFF_DRAWER_IV, IDCW_OFF_DRAWER_CHIP_HIT,
> IDCW_OFF_DRAWER_DRAWER_HIT, ICW_REQ, ICW_REQ_IV, CW_REQ_CHIP_HIT,
> ICW_REQ_DRAWER_HIT, ICW_ON_CHIP, ICW_ON_CHIP_IV, ICW_ON_CHIP_CHIP_HIT,
> ICW_ON_CHIP_DRAWER_HIT, ICW_ON_MODULE and ICW_OFF_DRAWER.
>
> The second Cache should be L2-Cache.
>
> Output before (display diff of the first four events)
>   # perf list -d
>   DCW_REQ
>        [Directory Write Level 1 Data Cache from Cache. Unit: cpum_cf]
>   DCW_REQ_CHIP_HIT
>        [Directory Write Level 1 Data Cache from Cache with Chip HP \
>                Hit. Unit: cpum_cf]
>   DCW_REQ_DRAWER_HIT
>        [Directory Write Level 1 Data Cache from Cache with Drawer \
>                HP Hit. Unit: cpum_cf]
>   DCW_REQ_IV
>        [Directory Write Level 1 Data Cache from Cache with Intervention. \
>                Unit: cpum_cf]
>
> Output after:
>   # perf list -d
>   DCW_REQ
>        [Directory Write Level 1 Data Cache from L2-Cache. Unit: cpum_cf]
>   DCW_REQ_CHIP_HIT
>        [Directory Write Level 1 Data Cache from L2-Cache with Chip HP \
>                Hit. Unit: cpum_cf]
>   DCW_REQ_DRAWER_HIT
>        [Directory Write Level 1 Data Cache from L2-Cache with Drawer \
>                HP Hit. Unit: cpum_cf]
>   DCW_REQ_IV
>        [Directory Write Level 1 Data Cache from L2-Cache with \
>                Intervention. Unit: cpum_cf]
>
> Fixes: 7f76b3113068 ("perf list: Add IBM z16 event description for s390")
> Reported-by: Andreas Krebbel <krebbel@xxxxxxxxxxxxx>
> Signed-off-by: Thomas Richter <tmricht@xxxxxxxxxxxxx>
> Acked-by: Andreas Krebbel <krebbel@xxxxxxxxxxxxx>

Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>

Thanks,
Ian