Re: [kvm-unit-tests PATCH v1 5/5] s390x: css: testing measurement block format 1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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));
>  }




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux