Re: [PATCH v2 17/24] selftests/resctrl: Replace count_bits with count_consecutive_bits()

From: Reinette Chatre
Date: Fri Apr 21 2023 - 20:20:28 EST


Hi Ilpo,

On 4/18/2023 4:44 AM, Ilpo Järvinen wrote:
> CAT and CMT tests depends on masks being continuous.

The term used in the spec is "contiguous", using it here
may help to convey the goal.

>
> Replace count_bits with more appropriate variant that counts
> consecutive bits.

Could you please elaborate why this is more appropriate and
why this is necessary? What is wrong with current solution?

Please note that cbm_mask in resctrl will always be contiguous.

Reinette