Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 13 Jul 2023 21:39:25 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Takashi Iwai <tiwai@suse.de>
Cc:     =?iso-8859-1?B?Ik7tY29sYXMgRi4gUi4gQS4i?= Prado 
        <nfraprado@collabora.com>, Jaroslav Kysela <perex@perex.cz>,
        kernel@collabora.com,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>,
        Shuah Khan <shuah@kernel.org>, Takashi Iwai <tiwai@suse.com>,
        alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
        linux-kselftest@vger.kernel.org
Subject: Re: [PATCH 2/2] kselftest/alsa: pcm-test: Decrease stream duration
 from 4 to 2 seconds
Message-ID: <66c81c16-737d-4ec6-b590-a6b1070b991b@sirena.org.uk>
References: <443f697b-fecf-6e8e-0b76-65257aff7da8@perex.cz>
 <9069ad0c-d166-4620-a3de-a36ab233cab0@sirena.org.uk>
 <5c2d5213-5299-44f1-9611-26002c8a5d3a@notapiano>
 <87352krcz5.wl-tiwai@suse.de>
 <f5cab2c2-1638-4d19-aff3-d46ed34b857e@sirena.org.uk>
 <87wmzwptu0.wl-tiwai@suse.de>
 <06b8bfde-e4f1-48ea-aa3e-35d2fe5df046@sirena.org.uk>
 <87wmz8i746.wl-tiwai@suse.de>
 <87ef1544-c1aa-4cce-82f2-60c6f1c2b1c8@sirena.org.uk>
 <87ilaofb9s.wl-tiwai@suse.de>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="rajlGC2kTISHyIRr"
Content-Disposition: inline
In-Reply-To: <87ilaofb9s.wl-tiwai@suse.de>
Precedence: bulk


--rajlGC2kTISHyIRr
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Thu, Jul 13, 2023 at 10:47:43AM +0200, Takashi Iwai wrote:
> Mark Brown wrote:

> > # default.time3.0.0.0.PLAYBACK - 44.1kHz stereo large periods
> > # default.time3.0.0.0.PLAYBACK hw_params.RW_INTERLEAVED.S16_LE.44100.2.16383.131064 sw_params.131064
> > not ok 10 default.time3.0.0.0.PLAYBACK
> > # time mismatch: expected 2000ms got 2229

> > reliably (the actual time drifts by a few ms).  The other boards I've
> > got coverage of seem fine, and I didn't check any broader CI yet.

> Interesting.  With the current patch, we rather extended the margin in
> proportion; formerly 4 sec +/- 0.1s, now 2 sec +/- 0.1s.  And it
> exceeded out of sudden.

Right.

> I guess this rather caught a problem of the driver itself.

Well, there's doubtless something driver/hardware related going on but
I'm not sure if it's a problem there or not.  The results from the 4s
runtime were:

# default.time3.0.0.0.PLAYBACK - 44.1kHz stereo large periods
# default.time3.0.0.0.PLAYBACK hw_params.RW_INTERLEAVED.S16_LE.44100.2.16383.131064 sw_params.131064
ok 10 default.time3.0.0.0.PLAYBACK

so the same buffer sizes and so on, and the period size is only 10ms
unless I miscalculated which should be quite a long way off from the
100ms of margin we give ourselves.  It does seem a little suspect that
it's the large periods test that's falling over though.

--rajlGC2kTISHyIRr
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmSwYPwACgkQJNaLcl1U
h9AAjAf9G1zzfng/beUy+UkABu6GsH4CgzFwfSum/wi7FgrnLo8yNYSH8M5RU/qf
dxoSuJF2Jf7glSBeBxFI+1lBUoEI0YmTuYez1QFQcdaOybZ1R0SgfrPFZ9sj6+14
vdSgMfk98QkgcnU0V0HHnUrSNkXP6Nd2y9hkukv6tTzEsICWnBLC8w4WS3qu92fQ
Z65B7dti1tOCTydBPjirhzybNy8qm3xOgw+tdIBPvEM1HMgSSYcZQIm3r2Mk3+EK
fEjhT0ol+azg2AvW/V78L/JaaeapItkoWfvB8VtI3IWFt1JjbjL0l3PoXQR4ZNGy
Pc4j3nYqATnS6J9OsLLgMBWzYKt6xg==
=9JRd
-----END PGP SIGNATURE-----

--rajlGC2kTISHyIRr--