From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ffmpeg-devel-bounces@ffmpeg.org>
Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100])
	by master.gitmailbox.com (Postfix) with ESMTPS id 677634C615
	for <ffmpegdev@gitmailbox.com>; Sun,  6 Apr 2025 23:44:26 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 0B250687D59;
	Mon,  7 Apr 2025 02:44:22 +0300 (EEST)
Received: from relay9-d.mail.gandi.net (relay9-d.mail.gandi.net
 [217.70.183.199])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id DB896687BFE
 for <ffmpeg-devel@ffmpeg.org>; Mon,  7 Apr 2025 02:44:15 +0300 (EEST)
Received: by mail.gandi.net (Postfix) with ESMTPSA id 3783A44268
 for <ffmpeg-devel@ffmpeg.org>; Sun,  6 Apr 2025 23:44:15 +0000 (UTC)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=niedermayer.cc;
 s=gm1; t=1743983055;
 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=X0gFnXsntLCwdV6plOwVGFT0Esm3xPbczbzXbOCtV2Q=;
 b=mYdGI5pd2sWX7/s4qLirZJ4X18gYLaP4YgzFE8TD3bNbgUwEk2eZDJVMI57KI83NcVx2sp
 QLs2qHlON046GNxhCJOQnXEn8MMUNXWdIfVdwsCEAWYpYP2KQvmDxJCg7bpGHdg3q+IJoI
 oaPZFHG+vHIWmwh3noDQyuC3AgUXE8GLUj7oLtSPCY88WWVgpElgUEWr1WYhHXu358h2ex
 2zmRxV4GuiSqmxGhNoL40orBhbmeEhHfBpgEbupIFN0vJd54/voObueeVvT8UCF8nPCjxk
 ae9rdaDn35Um+/Cz8NwHnh52nbBQfbErIzCuGlLvOL9dFGqIjwUeazMQmoCNSA==
Date: Mon, 7 Apr 2025 01:44:12 +0200
From: Michael Niedermayer <michael@niedermayer.cc>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Message-ID: <20250406234412.GL4991@pb2>
References: <20250403105049.883948-1-michael@niedermayer.cc>
 <20250403105049.883948-4-michael@niedermayer.cc>
 <cfa3f9f9-ef74-450b-96d1-bd40dabc8bbe@gmail.com>
MIME-Version: 1.0
In-Reply-To: <cfa3f9f9-ef74-450b-96d1-bd40dabc8bbe@gmail.com>
X-GND-State: clean
X-GND-Score: -70
X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeefvddrtddtgdduleekieegucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuifetpfffkfdpucggtfgfnhhsuhgsshgtrhhisggvnecuuegrihhlohhuthemuceftddunecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenfghrlhcuvffnffculdeftddmnecujfgurhepfffhvffukfhfgggtuggjsehgtderredttddvnecuhfhrohhmpefoihgthhgrvghlucfpihgvuggvrhhmrgihvghruceomhhitghhrggvlhesnhhivgguvghrmhgrhigvrhdrtggtqeenucggtffrrghtthgvrhhnpeeigeektdejudffjefhteegjedtgeettefggedthfejgfevhfetgeekjedtvdfhveenucfkphepgedurdeiiedrieejrdduudefnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehinhgvthepgedurdeiiedrieejrdduudefpdhhvghloheplhhotggrlhhhohhsthdpmhgrihhlfhhrohhmpehmihgthhgrvghlsehnihgvuggvrhhmrgihvghrrdgttgdpnhgspghrtghpthhtohepuddprhgtphhtthhopehffhhmphgvghdquggvvhgvlhesfhhfmhhpvghgrdhorhhg
X-GND-Sasl: michael@niedermayer.cc
Subject: Re: [FFmpeg-devel] [PATCH 4/4] avcodec/ffv1enc: avoid slices larger
 than 360x288 if no value is specified
X-BeenThere: ffmpeg-devel@ffmpeg.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org>
List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe>
List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel>
List-Post: <mailto:ffmpeg-devel@ffmpeg.org>
List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help>
List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe>
Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Content-Type: multipart/mixed; boundary="===============3594079254709542143=="
Errors-To: ffmpeg-devel-bounces@ffmpeg.org
Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org>
Archived-At: <https://master.gitmailbox.com/ffmpegdev/20250406234412.GL4991@pb2/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>


--===============3594079254709542143==
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="9wQDOiVmH5t0P2mC"
Content-Disposition: inline


--9wQDOiVmH5t0P2mC
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Apr 06, 2025 at 08:29:42PM -0300, James Almer wrote:
> On 4/3/2025 7:50 AM, Michael Niedermayer wrote:
> > This improves speed by providing more independent things for more CPUs
> >=20
> > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> > ---
> >   libavcodec/ffv1enc.c                             |  6 +++++-
> >   .../ref/fate/matroska-mastering-display-metadata | 16 ++++++++--------
> >   2 files changed, 13 insertions(+), 9 deletions(-)
> >=20
> > diff --git a/libavcodec/ffv1enc.c b/libavcodec/ffv1enc.c
> > index ce3f8e023e5..221344794e2 100644
> > --- a/libavcodec/ffv1enc.c
> > +++ b/libavcodec/ffv1enc.c
> > @@ -581,7 +581,11 @@ int ff_ffv1_encode_determine_slices(AVCodecContext=
 *avctx)
> >                   if (  ff_need_new_slices(avctx->width , s->num_h_slic=
es, s->chroma_h_shift)
> >                       ||ff_need_new_slices(avctx->height, s->num_v_slic=
es, s->chroma_v_shift))
> >                       continue;
> > -            if (avctx->slices =3D=3D s->num_h_slices * s->num_v_slices=
 && avctx->slices <=3D MAX_SLICES || !avctx->slices)
> > +            if (avctx->slices =3D=3D s->num_h_slices * s->num_v_slices=
 && avctx->slices <=3D MAX_SLICES)
> > +                return 0;
> > +            if (maxw*maxh > 360*288)
> > +                continue;
> > +            if (!avctx->slices)
> >                   return 0;
> >           }
> >       }
> > diff --git a/tests/ref/fate/matroska-mastering-display-metadata b/tests=
/ref/fate/matroska-mastering-display-metadata
> > index 91ce6a05584..6a2ff15b1b2 100644
> > --- a/tests/ref/fate/matroska-mastering-display-metadata
> > +++ b/tests/ref/fate/matroska-mastering-display-metadata
> > @@ -1,7 +1,7 @@
> > -ad5e3c4e338599c81ef7d0f9ae25f871 *tests/data/fate/matroska-mastering-d=
isplay-metadata.matroska
> > -1669589 tests/data/fate/matroska-mastering-display-metadata.matroska
> > +c1e5e2ecf433cf05af8556debc7d4d0b *tests/data/fate/matroska-mastering-d=
isplay-metadata.matroska
> > +1669773 tests/data/fate/matroska-mastering-display-metadata.matroska
> >   #extradata 0:        4, 0x040901a3
> > -#extradata 3:      200, 0x506463a8
> > +#extradata 3:      202, 0xfce96279
>=20
> Why did extradata change? Slice dimension value changes?

yes, any reason you belive theres an issue ?

thx

[...]

--=20
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle=20

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

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

iF0EABEKAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCZ/MRyQAKCRBhHseHBAsP
qxcCAJ9SKdfNqM8bnPiViTA6E63yxO9hgQCfS6r8d3p21coQTpbxv6miM0BcSy0=
=S7Ao
-----END PGP SIGNATURE-----

--9wQDOiVmH5t0P2mC--

--===============3594079254709542143==
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".

--===============3594079254709542143==--