From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 81AAF4BF8C for ; Thu, 18 Jul 2024 18:18:26 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 92FE968D935; Thu, 18 Jul 2024 21:18:23 +0300 (EEST) Received: from relay7-d.mail.gandi.net (relay7-d.mail.gandi.net [217.70.183.200]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 5C2D368D827 for ; Thu, 18 Jul 2024 21:18:17 +0300 (EEST) Received: by mail.gandi.net (Postfix) with ESMTPSA id 7BD4920004 for ; Thu, 18 Jul 2024 18:18:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=niedermayer.cc; s=gm1; t=1721326696; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=lvEVATLlzxuWM8DaH95GzgOwY2/5YvuCWAAGf3XCKMc=; b=HnS2dVE44/8nRY688pFhshEBOvJkjpWEttGQE2KTqhPzDHY0Lzrmz4APS+3cHNpmcMhfv8 Wh84AxI7inuVMDu1qAxIg1c1TH2NPnzU8Or4wDIlkLNEdN1W0GMHC6i/ShyJLKgZ3J03mr kCRnprLjL1qsciQKbxJ55F3G+j95yRzixEtBRwPtW5G63oIog4eNRykujOQsXxPBsYJyJt 8Rpy2tzzQYkHDkE0or5ibHm4Ry39V7XyYI56F4Qju9o9cEQp8VcqeS0eB3YO5bv+wxuCmm wvZZbrHRzD94qiKh2Cknx8+UqQ16NjlLUroz2FFF6ncKMWeWA7kLCnsn0ov94w== Date: Thu, 18 Jul 2024 20:18:15 +0200 From: Michael Niedermayer To: FFmpeg development discussions and patches Message-ID: <20240718181815.GG4991@pb2> References: <20240716171155.31838-1-anton@khirnov.net> <20240716171155.31838-10-anton@khirnov.net> <20240717224205.GY4991@pb2> <172129373901.21847.12392144255635795802@lain.khirnov.net> <20240718144806.GC4991@pb2> <172131667667.21847.10057209425663694866@lain.khirnov.net> MIME-Version: 1.0 In-Reply-To: <172131667667.21847.10057209425663694866@lain.khirnov.net> X-GND-Sasl: michael@niedermayer.cc Subject: Re: [FFmpeg-devel] [PATCH 10/39] lavc/ffv1dec: move the bitreader to stack X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: multipart/mixed; boundary="===============4677034286689521150==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============4677034286689521150== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="zLaDioFsM0hUAvZz" Content-Disposition: inline --zLaDioFsM0hUAvZz Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jul 18, 2024 at 05:31:16PM +0200, Anton Khirnov wrote: > Quoting Michael Niedermayer (2024-07-18 16:48:06) > > On Thu, Jul 18, 2024 at 11:08:59AM +0200, Anton Khirnov wrote: > > > Quoting Michael Niedermayer (2024-07-18 00:42:05) [...] > > > > > > Another factor that matters in efficient cache use is e.g. not having > > > multiple copies of the same constant data scattered around, which you= 're > > > objecting to in my other patches. > >=20 > > copying the actually used small data together per slice > > where its accessed per pixel should improve teh speed per pixel while > > making the per slice code a little slower. now we have 4 slices maybe > > and millions of pixels. Thats why this can give an overall gain >=20 > This all sounds like premature optimization, AKA the root of all evil. > As I said above, I intended to make this code more readable, not faster. > Yet somehow it became faster anyway, which suggests this code is not > very optimized. So then arguing whether this or that specific change > adds or removes a few cycles per frame seems like a waste time to me. Making the code faster is welcome Making the code cleaner is welcome whats faster is objective (mostly) whats cleaner is not objective. So we could agree or disagree on that Things that make the code slower, and dont provide a larger improvment elsewhere i do object to About "premature optimization", i have not seen any optimizations being removed that made the code hard to maintain or work with so this is not a nice statement. And iam the first to suport making the code cleaner and easier to maintain as iam the one maintaining it. Either way, thanks for your efforts in improving the ffv1 code, they are appreciated, my objections are not meant to hinder improvments i just want to make sure it all one by one is an improvment thx -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Any man who breaks a law that conscience tells him is unjust and willingly= =20 accepts the penalty by staying in jail in order to arouse the conscience of= =20 the community on the injustice of the law is at that moment expressing the= =20 very highest respect for law. - Martin Luther King Jr --zLaDioFsM0hUAvZz Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEKAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCZplcZAAKCRBhHseHBAsP q4QPAKCBIKjy7tnxB1y2HSWL6gj91XFELgCfS3tvsaWy1pxqcXRGoTt6PJ9EGqE= =AkEb -----END PGP SIGNATURE----- --zLaDioFsM0hUAvZz-- --===============4677034286689521150== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". --===============4677034286689521150==--