[PATCH 5/5] KVM: selftests: x86_64: Remove redundant newlines

Sean Christopherson seanjc at google.com
Tue Jan 30 12:26:25 PST 2024


On Wed, Dec 06, 2023, Andrew Jones wrote:
> @@ -162,7 +162,7 @@ static void check_clocksource(void)
>  		goto out;
>  	}
>  
> -	TEST_ASSERT(!strncmp(clk_name, "tsc\n", st.st_size),
> +	TEST_ASSERT(!strncmp(clk_name, "tsc", st.st_size),

This newline is functionally necessary.  It's in the strncmp() (*#$@ sysfs appends
newlines to everything), not the TEST_ASSERT message.  I'll give you a pass and
fixup when applying since I'm guessing you don't have x86 hardware ;-)

I double checked the other arch patches and didn't see anything sneaky like this.



More information about the kvm-riscv mailing list