On Fri, 29 Jan 2021 15:34:29 +0100
Pierre Morel <pmorel@xxxxxxxxxxxxx> wrote:
> Measurement block format 1 is made available by the extended
> mesurement block facility and is indicated in the SCHIB by
> the bit in the PMCW.
>
> The MBO is specified in the SCHIB of each channel and the MBO
> defined by the SCHM instruction is ignored.
>
> The test of the MB format 1 is just skip if the feature is
s/skip/skipped/
> not available.
>
> Signed-off-by: Pierre Morel <pmorel@xxxxxxxxxxxxx>
> ---
> s390x/css.c | 51 +++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
(...)
> @@ -208,6 +251,14 @@ static void test_schm(void)
> schm(mb0, SCHM_MBU);
> test_schm_fmt0(mb0);
>
> + mb1 = alloc_io_mem(sizeof(struct measurement_block_format1), 0);
> + if (!mb1) {
I'm wondering whether you should move this into the format 1
invocation, so you won't try to allocate memory if format 1 is not even
supported.
> + report(0, "mesurement_block_format0 allocation");
In any case, s/mesurement_block_format0/measurement_block_format1/ :)
> + goto end_free;
> + }
> + test_schm_fmt1(mb1);
> +
> + free_io_mem(mb1, sizeof(struct measurement_block_format1));
> end_free:
> free_io_mem(mb0, sizeof(struct measurement_block_format0));
> }
