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 08A7F4C66C for ; Wed, 31 Jul 2024 12:55:05 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 1B02F68D734; Wed, 31 Jul 2024 15:55:03 +0300 (EEST) Received: from mail-pf1-f178.google.com (mail-pf1-f178.google.com [209.85.210.178]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id C5AFD68BDE1 for ; Wed, 31 Jul 2024 15:54:56 +0300 (EEST) Received: by mail-pf1-f178.google.com with SMTP id d2e1a72fcca58-7104f939aa7so469294b3a.1 for ; Wed, 31 Jul 2024 05:54:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1722430494; x=1723035294; darn=ffmpeg.org; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=/5rASMi8P8kP2kvhpPjU4cF79kBxRM17JyEra8ozrh4=; b=c4yUvI1OsoacCfWfpln7J9AJUlvBkfj5Q6EujZnEX0vuLYUBNm+oUeQTU6J0r9HbEC 9Ed2lLyVEqhy10/4FfvUF3V5NIwXbLxGjwtZDLKHhDBl9AGcYgy3UHwGqTQLVSLtGNij oXXVIjyHsj2Kp8Vy+RVqERkX99LQiqlkI770lmYmrnBKgw8GRcE1bi/4yGZYXsQhDzwj QlwB2r5eMAjrf5I5Md8KiSQDoKOiwk3iFeSPZtv1Wi6rVvEgZ4yWiOkSvvV3jXGHWbKs TbCUr8vsQ9cQBnHvSDtKJ4M5jHYumQpPa+xK1p4OCRm3d5dFgVWDBl3RdxrIrKESCllV UWjQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1722430494; x=1723035294; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=/5rASMi8P8kP2kvhpPjU4cF79kBxRM17JyEra8ozrh4=; b=IjttJap+YbkFYY1lBCcsDJIVNvYMbM6EB3oOJYzhl1qYWAg3LPzbqEMor9rWdON69d K1rBjID7vBafsvEkLFt/VNoOuD6BGZPB9z54yIc7B8I0JASbzfffGyEdWTQK5Ktfy8di 2Ac7LuCaiWsUNPI9wr/bIuUyx9nEa7DtoAuOG9UGBXDmsNy1sUjekcG2fBXk8Oy/U8K/ Vgrtv6SEWdlzQPzVNTAAmma9fWNznlT7o8s9xs1z+Ph5uUxk03A+oiE5ytpkBkBrhTYJ n8MupeGl7neqYrmvuvKSu79MbPRWeV5POC9nDzTpydXZOal1J/9iTDLw3ryCA7HDGjiy 8ZmA== X-Gm-Message-State: AOJu0Yx1xwW6psPqu3TOVANEM8P/szuFtKwsBf/7d/2+n94uw/Bwdfjf 1w25hYDIpmhjD1kMmE2sUyBBqjWAKho4Ydx4x+ytfJjVEYTdTkxFB8/bsA== X-Google-Smtp-Source: AGHT+IGYDXOPoxBhJS+AyeOEyk6IrRChLd3gXJfJuTEvBhynq6CFX1Ah3u8tkD6zpKrKRgwDNijbAQ== X-Received: by 2002:a05:6a21:394b:b0:1c4:a5fe:321b with SMTP id adf61e73a8af0-1c4a5fe33f5mr16991886637.18.1722430493730; Wed, 31 Jul 2024 05:54:53 -0700 (PDT) Received: from localhost.localdomain ([190.194.167.233]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-2cfdc45b4aasm1248695a91.27.2024.07.31.05.54.52 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 31 Jul 2024 05:54:53 -0700 (PDT) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Wed, 31 Jul 2024 09:55:23 -0300 Message-ID: <20240731125523.63406-1-jamrial@gmail.com> X-Mailer: git-send-email 2.45.2 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH] avutil/frame: use the maximum compile time supported alignment for strides 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: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: This puts lavu frame buffer allocator helpers in sync with lavc's decoder frame buffer allocator's STRIDE_ALIGN define. Remove the comment about av_cpu_max_align() while at it as using it is not ideal when CPU flags can be changed mid process. Should fix ticket #11116. Signed-off-by: James Almer --- libavutil/frame.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavutil/frame.c b/libavutil/frame.c index 673a9afb3b..5cbfc6a48b 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c @@ -166,6 +166,8 @@ void av_frame_free(AVFrame **frame) av_freep(frame); } +#define ALIGN (HAVE_SIMD_ALIGN_64 ? 64 : 32) + static int get_video_buffer(AVFrame *frame, int align) { const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(frame->format); @@ -182,7 +184,7 @@ static int get_video_buffer(AVFrame *frame, int align) if (!frame->linesize[0]) { if (align <= 0) - align = 32; /* STRIDE_ALIGN. Should be av_cpu_max_align() */ + align = ALIGN; for (int i = 1; i <= align; i += i) { ret = av_image_fill_linesizes(frame->linesize, frame->format, -- 2.45.2 _______________________________________________ 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".