From patchwork Fri May 7 06:46:38 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 27637 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a6b:b214:0:0:0:0:0 with SMTP id b20csp273430iof; Thu, 6 May 2021 23:47:46 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxFmnldIBUhxD5YMn/d1Ixu2xqRlxhMfimvxsU8WFHA4iHs24R65WwvLEK+zR5+ze7szkMW X-Received: by 2002:a17:906:b0cb:: with SMTP id bk11mr8496903ejb.310.1620370066739; Thu, 06 May 2021 23:47:46 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id p5si2906399edi.291.2021.05.06.23.47.46; Thu, 06 May 2021 23:47:46 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b="jAY82ws/"; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 7F73E680A0F; Fri, 7 May 2021 09:47:18 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR02-HE1-obe.outbound.protection.outlook.com (mail-oln040092068080.outbound.protection.outlook.com [40.92.68.80]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id B434168094F for ; Fri, 7 May 2021 09:47:16 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=df9hpk8OZX5hDWwUUX4xPUyMgRw7Rkr9cvWzc2w942QPUIlQlVPKSrkzk6Da/wZqNsgFM/o+B4jIkzPM5FN2PrvTuawEnw20nx1IYzu/p41mOyjxbCHuA998z/1iuQMTLl+qH628Qe3/elqxMOKLvG1J8Ns/qXZeUA5K8ViNdaZblp2nVWiClGBI/RWkpTdvhdRJa9rTti52tpw/oSIG+Pv3/oeBypWpjTx+tKFBLprquSmqVNCcO4q+UwjduXtmO11Z5jvm0vR2mWM/laIVelIQm50I67p6saD+JUJxPcyDStDNpK6WrtHYKk/V+1eyCKK7mVsUxfv7dWct8VF82g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/ogdTkja29FJ9h9iwMdJUWzNl9Uw5QAUCYzd7nSJj2A=; b=YWbE3S0+N0WG6DMxPNsnACAEn4oVkS7T8fCLkJi676erypPq3CSj63xByF6f+Py+ZEfEsWDY1PTYYZEk/pZambS9T7rnR6JDwIxESorJ0Sm9soHJH42TCBiu3vJ+ZizQzr9uUs3HA89LTQN9DNOhzaQxUROpVOqXV1hiZdZI06sLgRnumNX84jxRJQZVZrBkkYIxCtry1rUMh2oM4EMf3SR0vk3lSE5qpgeB5A+CUB8g9HOQbCMHSdNIXuFa3gWcDqFQ3emIL8vL0OSTzllXfoBVkPxqcqsX/oJY47TxG7EXI6ldgTH+PSYdyQ0gSko06Bo08gsnoJU9lOAhF3o10g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/ogdTkja29FJ9h9iwMdJUWzNl9Uw5QAUCYzd7nSJj2A=; b=jAY82ws/7whT0KzYNPHkrjil1zMU6NrTZvOpdtOv30fnBmSQBWyX2nhUdgGNyp6fLjQIrxwNEI/Lnvj850S54svJTUuheznepIoiDc6A2VeoONfxVkMc/3GNuc8shbvZcjRWSjHkz6OuWhh2SrLmgtsTpR+NAfAXs4Aw9YqE1Kbnj9y05hh1upZwlL5bl5HNybASQUw1VAp8/NtoMErM9v3Ja/gxP4QNJ7TTRT2wmzBJ0WB46UQlAV6rO9f2YrQTFg8ds90a1Rbo+q8eIQpe/F6H2XwiPbDPBFCo3zyrC64E/TrC0fICXiOuL10V6QdcGGmxZajeAw+oJggjrf0e/Q== Received: from HE1EUR02FT024.eop-EUR02.prod.protection.outlook.com (2a01:111:e400:7e1d::4e) by HE1EUR02HT079.eop-EUR02.prod.protection.outlook.com (2a01:111:e400:7e1d::371) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25; Fri, 7 May 2021 06:47:14 +0000 Received: from HE1PR0301MB2154.eurprd03.prod.outlook.com (2a01:111:e400:7e1d::52) by HE1EUR02FT024.mail.protection.outlook.com (2a01:111:e400:7e1d::181) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25 via Frontend Transport; Fri, 7 May 2021 06:47:14 +0000 X-IncomingTopHeaderMarker: OriginalChecksum:744B6B95B6D4F9B68826C0D8899D2EFDBD38E96C906294F6477E0628C0AE9D98; UpperCasedChecksum:0C49CF5997F4F59CB53C4322B98EF00ACE2017FAD2B790DF12F35142EA21B851; SizeAsReceived:7602; Count:48 Received: from HE1PR0301MB2154.eurprd03.prod.outlook.com ([fe80::45bb:c44f:2b75:23b7]) by HE1PR0301MB2154.eurprd03.prod.outlook.com ([fe80::45bb:c44f:2b75:23b7%5]) with mapi id 15.20.4108.027; Fri, 7 May 2021 06:47:14 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 7 May 2021 08:46:38 +0200 Message-ID: X-Mailer: git-send-email 2.27.0 In-Reply-To: References: X-TMN: [OWk6qcx2MNzgX07tDy0QiRp497jnatnRRXAqEXosVJk=] X-ClientProxiedBy: AM0PR06CA0117.eurprd06.prod.outlook.com (2603:10a6:208:ab::22) To HE1PR0301MB2154.eurprd03.prod.outlook.com (2603:10a6:3:2a::22) X-Microsoft-Original-Message-ID: <20210507064647.362502-10-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from sblaptop.fritz.box (188.193.170.150) by AM0PR06CA0117.eurprd06.prod.outlook.com (2603:10a6:208:ab::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25 via Frontend Transport; Fri, 7 May 2021 06:47:14 +0000 X-MS-PublicTrafficType: Email X-IncomingHeaderCount: 48 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-Correlation-Id: 685d8443-13e6-47e0-d0d5-08d91123f2d0 X-MS-Exchange-SLBlob-MailProps: gjx25WM8ZNWUr2JHLiAqcgwWwb6KluLH9lhJPzcSP1IRERdunlh0aR9Gg8OSZEDuLSZlAzG68nak7rhDJM3bON0HKy6kFiazHItFv2UETMHi3bx1xKk87sdvY/VVF/Rsctxq6zOKGx71ilU8wn6pnGMO9+KW/SRGvLZLAEZ95dujwSfYBrA+mBbgdruLmr2BlNKd4yluozcbeb+Z15jPag8FQfujE+7/oNLMVoEccKvvHfHbTPlz0xrspyAhypGh8bKmp4VNfJhRkoFKWakaNjGicYLixqvEElyMNrp144R1tCPhrQiu9zx41FlU+Gpmxf6u5TcdBG8uhSmE97h1y2mfix93f2Y2FQtBfAL8tmnC/qr+W5zcQwj5uKf49lggf446bAS2BVdjHlH7GTY0FqWDE1vFCFQx62ybZntDnmUFCXrNhVGMLlB0bpd1VGLmDpIwygHJFXdk6pCPM7YZU1ObPgQ+7a8R3ySBGVYoEQvAeYbz2Sa2NZ2LUfD2+qT8eLLudK91NuBdgPlIjTL8tAKVOto3dEK/hw3g+LA5xowVvxn67aM7g4WDn7DMGtgPWGpWql9IGxxyuKffVJRGKc4dCsY83j/6fXAhMn1cYHuzIHuFSJMzSjD0sq7JJ7iv4psKjBUqq0mBQVnaNaEZFMWU+aeSVtAgS2zgcZYyMly0tJ0WqOCrIbOMyZfOc/Kpkdb9YlmGZXHgpnQlebYEES+YefLEwapeLMJKO+zeEgI= X-MS-TrafficTypeDiagnostic: HE1EUR02HT079: X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: iW3rpwxIkyNTIsrONidWIIpcJDkheJ3G7DptwLyzRUfQWTtmWQRq2woMk7fd1M7l4Vz6tUh8vLXCdDqoZb7PSqw7yXShkfeueU8F9ctVx6ZdLGxiDezKisjjKAHovWUUN4nhYpy5Bh3cHG0O75HbY7KEWNh5zwrEbGcfvRfqiO9+HM+IiIP+cvEAjaZPsuG0U0qdevqIbtUWtzx88gRqWewLFShvtbLKS5JSBrn9qkHAzwF71SmKzzZHAYb7rATFJB09YEnHj94+HjyXnWrY+SMYygqQxUEZiJC3967NMVlE6fMuqcqBCWxgba9TUga6Z6NNG21Lcql3ltuboK7TZqhG5ptx6xVp1tEI1TdSqlBgD3C22Wiwfx5NB6vuXtod1FS5Ys4+hjLsykTQ4E/EXA== X-MS-Exchange-AntiSpam-MessageData: q+7GR0s6O7+3ASWQk3viEMnSQktYv5YaNFDeKlyj7Ky9iKr4/o+NM13jtT1S2PsLDP8kCqdoYT31pmGY5wRNf6QSEPLVJLCB4dSkMd2vGYQebdtsi9AqI+g8OOOS47kK31LPjyaASSvGCrnswpByDw== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 685d8443-13e6-47e0-d0d5-08d91123f2d0 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 May 2021 06:47:14.7133 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-AuthSource: HE1EUR02FT024.eop-EUR02.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: Internet X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1EUR02HT079 Subject: [FFmpeg-devel] [PATCH 11/20] avcodec/vorbisenc: Combine codebooks, avoid relocations 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 Cc: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: mFlLNVBvZ30B The Vorbis encoder has an array of a structure containing all the ingredients for a codebook; this includes a pointer to the actual codebook and some even have a pointer to an array containing quant values. Each of these real codebooks is an array of its own. These pointers lead to relocations and therefore the array will be placed in .data.rel.ro and not in .rodata. This commit avoids the pointers altogether by combining all the actual codebooks into one big array; the actual codebooks are now accessed consecutively by incrementing the pointer used to access them by the length of the actual codebook that has just been dealt with (said length is contained in the structure describing the codebook). There is no downside to this given that these codebooks are already only used once during init. Signed-off-by: Andreas Rheinhardt --- libavcodec/vorbis_enc_data.h | 216 +++++++++++++++-------------------- libavcodec/vorbisenc.c | 8 +- 2 files changed, 100 insertions(+), 124 deletions(-) diff --git a/libavcodec/vorbis_enc_data.h b/libavcodec/vorbis_enc_data.h index a51aaec978..eab8adbb76 100644 --- a/libavcodec/vorbis_enc_data.h +++ b/libavcodec/vorbis_enc_data.h @@ -23,15 +23,12 @@ #include -static const uint8_t codebook0[] = { +static const uint8_t codebooks[] = { + /* Codebook 0 - 16 elements */ 2, 10, 8, 14, 7, 12, 11, 14, 1, 5, 3, 7, 4, 9, 7, 13, -}; - -static const uint8_t codebook1[] = { + /* Codebook 1 - 8 elements */ 1, 4, 2, 6, 3, 7, 5, 7, -}; - -static const uint8_t codebook2[] = { + /* Codebook 2 - 256 elements */ 1, 5, 7, 21, 5, 8, 9, 21, 10, 9, 12, 20, 20, 16, 20, 20, 4, 8, 9, 20, 6, 8, 9, 20, 11, 11, 13, 20, 20, 15, 17, 20, 9, 11, 14, 20, 8, 10, 15, 20, 11, 13, 15, 20, 20, @@ -50,17 +47,13 @@ static const uint8_t codebook2[] = { 19, 18, 20, 15, 16, 20, 20, 17, 17, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, -}; - -static const uint8_t codebook3[] = { + /* Codebook 3 - 64 elements */ 2, 3, 7, 13, 4, 4, 7, 15, 8, 6, 9, 17, 21, 16, 15, 21, 2, 5, 7, 11, 5, 5, 7, 14, 9, 7, 10, 16, 17, 15, 16, 21, 4, 7, 10, 17, 7, 7, 9, 15, 11, 9, 11, 16, 21, 18, 15, 21, 18, 21, 21, 21, 15, 17, 17, 19, 21, 19, 18, 20, 21, 21, 21, 20, -}; - -static const uint8_t codebook4[] = { + /* Codebook 4 - 128 elements */ 5, 5, 5, 5, 6, 5, 6, 5, 6, 5, 6, 5, 6, 5, 6, 5, 6, 5, 6, 5, 6, 5, 6, 5, 7, 5, 7, 5, 7, 5, 7, 5, 8, 6, 8, 6, 8, 6, 9, 6, 9, 6, 10, 6, 10, @@ -70,15 +63,11 @@ static const uint8_t codebook4[] = { 14, 11, 14, 12, 14, 13, 14, 13, 14, 14, 15, 16, 15, 15, 15, 14, 15, 17, 21, 22, 22, 21, 22, 22, 22, 22, 22, 22, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, -}; - -static const uint8_t codebook5[] = { + /* Codebook 5 - 32 elements */ 2, 5, 5, 4, 5, 4, 5, 4, 5, 4, 6, 5, 6, 5, 6, 5, 6, 5, 7, 5, 7, 6, 8, 6, 8, 6, 8, 6, 9, 6, 9, 6, -}; - -static const uint8_t codebook6[] = { + /* Codebook 6 - 96 elements */ 8, 5, 8, 4, 9, 4, 9, 4, 9, 4, 9, 4, 9, 4, 9, 4, 9, 4, 9, 4, 9, 4, 8, 4, 8, 4, 9, 5, 9, 5, 9, 5, 9, 5, 9, 6, 10, 6, 10, 7, 10, 8, 11, 9, 11, @@ -86,15 +75,11 @@ static const uint8_t codebook6[] = { 15, 15, 16, 16, 15, 16, 16, 16, 15, 18, 16, 15, 17, 17, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, -}; - -static const uint8_t codebook7[] = { + /* Codebook 7 - 32 elements */ 1, 5, 5, 5, 5, 5, 5, 5, 6, 5, 6, 5, 6, 5, 6, 5, 6, 6, 7, 7, 7, 7, 8, 7, 8, 8, 9, 8, 10, 9, 10, 9, -}; - -static const uint8_t codebook8[] = { + /* Codebook 8 - 96 elements */ 4, 3, 4, 3, 4, 4, 5, 4, 5, 4, 5, 5, 6, 5, 6, 5, 7, 5, 7, 6, 7, 6, 8, 7, 8, 7, 8, 7, 9, 8, 9, 9, 9, 9, 10, 10, 10, 11, 9, 12, 9, 12, 9, 15, 10, @@ -102,49 +87,35 @@ static const uint8_t codebook8[] = { 11, 14, 12, 13, 13, 14, 14, 13, 14, 15, 14, 16, 13, 13, 14, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 15, 15, -}; - -static const uint8_t codebook9[] = { + /* Codebook 9 - 17 elements */ 4, 5, 4, 5, 3, 5, 3, 5, 3, 5, 4, 4, 4, 4, 5, 5, 5, -}; - -static const uint8_t codebook10[] = { + /* Codebook 10 - 32 elements */ 3, 3, 4, 3, 4, 4, 4, 4, 5, 5, 5, 5, 5, 6, 5, 7, 5, 8, 6, 8, 6, 9, 7, 10, 7, 10, 8, 10, 8, 11, 9, 11, -}; - -static const uint8_t codebook11[] = { - 3, 7, 3, 8, 3, 10, 3, 8, 3, 9, 3, 8, 4, 9, 4, - 9, 5, 9, 6, 10, 6, 9, 7, 11, 7, 12, 9, 13, 10, 13, - 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, - 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, - 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, - 12, 12, 12, -}; - -static const uint8_t codebook12[] = { + /* Codebook 11 - 78 elements */ + 3, 7, 3, 8, 3, 10, 3, 8, 3, 9, 3, 8, 4, 9, 4, + 9, 5, 9, 6, 10, 6, 9, 7, 11, 7, 12, 9, 13, 10, 13, + 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, + 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, + 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, + 12, 12, 12, + /* Codebook 12 - 17 elements */ 4, 5, 4, 5, 4, 5, 4, 5, 3, 5, 3, 5, 3, 5, 4, 5, 4, -}; - -static const uint8_t codebook13[] = { + /* Codebook 13 - 32 elements */ 4, 2, 4, 2, 5, 3, 5, 4, 6, 6, 6, 7, 7, 8, 7, 8, 7, 8, 7, 9, 8, 9, 8, 9, 8, 10, 8, 11, 9, 12, 9, 12, -}; - -static const uint8_t codebook14[] = { + /* Codebook 14 - 78 elements */ 2, 5, 2, 6, 3, 6, 4, 7, 4, 7, 5, 9, 5, 11, 6, 11, 6, 11, 7, 11, 6, 11, 6, 11, 9, 11, 8, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 10, 10, 10, 10, 10, 10, -}; - -static const uint8_t codebook15[] = { + /* Codebook 15 - 100 elements */ 5, 6, 11, 11, 11, 11, 10, 10, 12, 11, 5, 2, 11, 5, 6, 6, 7, 9, 11, 13, 13, 10, 7, 11, 6, 7, 8, 9, 10, 12, 11, 5, 11, 6, 8, 7, 9, 11, 14, 15, 11, 6, 6, 8, 4, @@ -152,9 +123,7 @@ static const uint8_t codebook15[] = { 10, 7, 7, 8, 6, 5, 5, 7, 9, 9, 11, 8, 8, 11, 8, 7, 6, 6, 7, 9, 12, 11, 10, 13, 9, 9, 7, 7, 7, 9, 11, 13, 12, 15, 12, 11, 9, 8, 8, 8, -}; - -static const uint8_t codebook16[] = { + /* Codebook 16 - 1641 elements, real length 6561 */ 2, 4, 4, 0, 0, 0, 0, 0, 0, 5, 6, 6, 0, 0, 0, 0, 0, 0, 5, 6, 6, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, @@ -265,9 +234,7 @@ static const uint8_t codebook16[] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 6, 8, 8, 0, 0, 0, 0, 0, 0, 8, 9, 9, 0, 0, 0, 0, 0, 0, 8, 9, 8, -}; - -static const uint8_t codebook17[] = { + /* Codebook 17 - 443 elements, real length 625 */ 2, 5, 5, 0, 0, 0, 5, 5, 0, 0, 0, 5, 5, 0, 0, 0, 7, 8, 0, 0, 0, 0, 0, 0, 0, 5, 6, 6, 0, 0, 0, 7, 7, 0, 0, 0, 7, 7, 0, 0, 0, 10, 10, 0, 0, @@ -298,9 +265,7 @@ static const uint8_t codebook17[] = { 0, 9, 9, 0, 0, 0, 9, 9, 0, 0, 0, 10, 10, 0, 0, 0, 0, 0, 0, 0, 8, 10, 10, 0, 0, 0, 9, 9, 0, 0, 0, 9, 9, 0, 0, 0, 10, 10, -}; - -static const uint8_t codebook18[] = { + /* Codebook 18 - 105 elements, real length 625 */ 2, 4, 3, 6, 6, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 4, 4, 4, 6, 6, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, @@ -308,26 +273,20 @@ static const uint8_t codebook18[] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 6, 6, 6, 9, 9, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 6, 6, 7, 9, 9, -}; - -static const uint8_t codebook19[] = { + /* Codebook 19 - 68 elements, real length 81 */ 2, 3, 3, 6, 6, 0, 0, 0, 0, 0, 4, 4, 6, 6, 0, 0, 0, 0, 0, 4, 4, 6, 6, 0, 0, 0, 0, 0, 5, 5, 6, 6, 0, 0, 0, 0, 0, 0, 0, 6, 6, 0, 0, 0, 0, 0, 0, 0, 7, 8, 0, 0, 0, 0, 0, 0, 0, 7, 7, 0, 0, 0, 0, 0, 0, 0, 9, 9, -}; - -static const uint8_t codebook20[] = { + /* Codebook 20 - 81 elements */ 1, 3, 4, 6, 6, 7, 7, 9, 9, 0, 5, 5, 7, 7, 7, 8, 9, 9, 0, 5, 5, 7, 7, 8, 8, 9, 9, 0, 7, 7, 8, 8, 8, 8, 10, 10, 0, 0, 0, 8, 8, 8, 8, 10, 10, 0, 0, 0, 9, 9, 9, 9, 10, 10, 0, 0, 0, 9, 9, 9, 9, 10, 10, 0, 0, 0, 10, 10, 10, 10, 11, 11, 0, 0, 0, 0, 0, 10, 10, 11, 11, -}; - -static const uint8_t codebook21[] = { + /* Codebook 21 - 289 elements */ 2, 3, 3, 6, 6, 7, 7, 8, 8, 8, 8, 9, 9, 10, 10, 11, 10, 0, 5, 5, 7, 7, 8, 8, 9, 9, 9, 9, 10, 10, 10, 10, 11, 11, 0, 5, 5, 7, 7, 8, 8, 9, 9, 9, 9, @@ -348,18 +307,14 @@ static const uint8_t codebook21[] = { 0, 0, 0, 0, 0, 0, 0, 11, 11, 12, 12, 12, 12, 13, 13, 13, 13, 0, 0, 0, 0, 0, 0, 0, 0, 0, 12, 12, 12, 12, 13, 13, 13, 13, -}; - -static const uint8_t codebook22[] = { + /* Codebook 22 - 81 elements */ 1, 4, 4, 7, 6, 6, 7, 6, 6, 4, 7, 7, 10, 9, 9, 11, 9, 9, 4, 7, 7, 10, 9, 9, 11, 9, 9, 7, 10, 10, 11, 11, 10, 12, 11, 11, 6, 9, 9, 11, 10, 10, 11, 10, 10, 6, 9, 9, 11, 10, 10, 11, 10, 10, 7, 11, 11, 11, 11, 11, 12, 11, 11, 6, 9, 9, 11, 10, 10, 11, 10, 10, 6, 9, 9, 11, 10, 10, 11, 10, 10, -}; - -static const uint8_t codebook23[] = { + /* Codebook 23 - 121 elements */ 2, 4, 4, 6, 6, 7, 7, 7, 7, 8, 8, 10, 5, 5, 6, 6, 7, 7, 8, 8, 8, 8, 10, 5, 5, 6, 6, 7, 7, 8, 8, 8, 8, 10, 6, 6, 7, 7, 8, 8, 8, 8, 8, 8, 10, @@ -369,9 +324,7 @@ static const uint8_t codebook23[] = { 10, 10, 10, 8, 8, 8, 8, 8, 8, 10, 10, 10, 10, 10, 9, 9, 8, 8, 9, 8, 10, 10, 10, 10, 10, 8, 8, 8, 8, 8, 8, -}; - -static const uint8_t codebook24[] = { + /* Codebook 24 - 169 elements */ 1, 4, 4, 6, 6, 7, 7, 8, 8, 9, 9, 10, 10, 6, 5, 5, 7, 7, 8, 8, 8, 8, 9, 9, 10, 10, 7, 5, 5, 7, 7, 8, 8, 8, 8, 9, 9, 11, 10, 0, 8, 8, 8, 8, 9, @@ -384,14 +337,10 @@ static const uint8_t codebook24[] = { 11, 11, 11, 11, 12, 12, 13, 13, 0, 0, 0, 0, 0, 12, 12, 12, 12, 13, 13, 14, 13, 0, 0, 0, 0, 0, 13, 13, 12, 12, 13, 12, 14, 13, -}; - -static const uint8_t codebook25[] = { + /* Codebook 25 - 25 elements */ 2, 4, 4, 5, 5, 6, 5, 5, 5, 5, 6, 4, 5, 5, 5, 6, 5, 5, 5, 5, 6, 6, 6, 5, 5, -}; - -static const uint8_t codebook26[] = { + /* Codebook 26 - 169 elements */ 1, 4, 4, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 4, 9, 8, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 2, 9, 7, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, @@ -404,9 +353,7 @@ static const uint8_t codebook26[] = { 12, 12, 12, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, -}; - -static const uint8_t codebook27[] = { + /* Codebook 27 - 225 elements */ 1, 4, 4, 6, 6, 7, 7, 8, 7, 9, 9, 10, 10, 10, 10, 6, 5, 5, 7, 7, 8, 8, 10, 8, 11, 10, 12, 12, 13, 13, 6, 5, 5, 7, 7, 8, 8, 10, 9, 11, 11, 12, 12, 13, 12, @@ -422,9 +369,7 @@ static const uint8_t codebook27[] = { 17, 18, 18, 17, 18, 13, 12, 13, 10, 12, 11, 14, 14, 14, 14, 17, 18, 18, 18, 18, 15, 16, 12, 12, 13, 10, 14, 12, 14, 15, 18, 18, 18, 16, 17, 16, 14, 12, 11, 13, 10, 13, 13, 14, 15, -}; - -static const uint8_t codebook28[] = { + /* Codebook 28 - 289 elements */ 2, 5, 5, 6, 6, 7, 7, 7, 7, 7, 7, 8, 8, 8, 8, 8, 8, 10, 6, 6, 7, 7, 8, 7, 8, 8, 8, 8, 8, 9, 9, 9, 9, 9, 10, 6, 6, 7, 7, 7, 7, 8, 8, 8, 8, @@ -447,45 +392,72 @@ static const uint8_t codebook28[] = { 9, 10, 10, 10, }; +static const uint8_t quant_tables[] = { + /* Codebook 16 */ + 1, 0, 2, + /* Codebook 17 */ + 2, 1, 3, 0, 4, + /* Codebook 18 */ + 2, 1, 3, 0, 4, + /* Codebook 19 */ + 4, 3, 5, 2, 6, 1, 7, 0, 8, + /* Codebook 20 */ + 4, 3, 5, 2, 6, 1, 7, 0, 8, + /* Codebook 21 */ + 8, 7, 9, 6, 10, 5, 11, 4, 12, 3, 13, 2, 14, 1, 15, 0, 16, + /* Codebook 22 */ + 1, 0, 2, + /* Codebook 23 */ + 5, 4, 6, 3, 7, 2, 8, 1, 9, 0, 10, + /* Codebook 24 */ + 6, 5, 7, 4, 8, 3, 9, 2, 10, 1, 11, 0, 12, + /* Codebook 25 */ + 2, 1, 3, 0, 4, + /* Codebook 26 */ + 6, 5, 7, 4, 8, 3, 9, 2, 10, 1, 11, 0, 12, + /* Codebook 27 */ + 7, 6, 8, 5, 9, 4, 10, 3, 11, 2, 12, 1, 13, 0, 14, + /* Codebook 28 */ + 8, 7, 9, 6, 10, 5, 11, 4, 12, 3, 13, 2, 14, 1, 15, 0, 16, +}; + static const struct { int dim; int len; int real_len; - const uint8_t *clens; int lookup; float min; float delta; - const uint8_t *quant; } cvectors[] = { - { 2, 16, 16, codebook0, 0 }, - { 2, 8, 8, codebook1, 0 }, - { 2, 256, 256, codebook2, 0 }, - { 2, 64, 64, codebook3, 0 }, - { 2, 128, 128, codebook4, 0 }, - { 2, 32, 32, codebook5, 0 }, - { 2, 96, 96, codebook6, 0 }, - { 2, 32, 32, codebook7, 0 }, - { 2, 96, 96, codebook8, 0 }, - { 2, 17, 17, codebook9, 0 }, - { 2, 32, 32, codebook10, 0 }, - { 2, 78, 78, codebook11, 0 }, - { 2, 17, 17, codebook12, 0 }, - { 2, 32, 32, codebook13, 0 }, - { 2, 78, 78, codebook14, 0 }, - { 2, 100, 100, codebook15, 0 }, - { 8, 1641, 6561, codebook16, 1, -1.0, 1.0, (const uint8_t[]){ 1, 0, 2, } }, - { 4, 443, 625, codebook17, 1, -2.0, 1.0, (const uint8_t[]){ 2, 1, 3, 0, 4, } }, - { 4, 105, 625, codebook18, 1, -2.0, 1.0, (const uint8_t[]){ 2, 1, 3, 0, 4, } }, - { 2, 68, 81, codebook19, 1, -4.0, 1.0, (const uint8_t[]){ 4, 3, 5, 2, 6, 1, 7, 0, 8, } }, - { 2, 81, 81, codebook20, 1, -4.0, 1.0, (const uint8_t[]){ 4, 3, 5, 2, 6, 1, 7, 0, 8, } }, - { 2, 289, 289, codebook21, 1, -8.0, 1.0, (const uint8_t[]){ 8, 7, 9, 6, 10, 5, 11, 4, 12, 3, 13, 2, 14, 1, 15, 0, 16, } }, - { 4, 81, 81, codebook22, 1, -11.0, 11.0, (const uint8_t[]){ 1, 0, 2, } }, - { 2, 121, 121, codebook23, 1, -5.0, 1.0, (const uint8_t[]){ 5, 4, 6, 3, 7, 2, 8, 1, 9, 0, 10, } }, - { 2, 169, 169, codebook24, 1, -30.0, 5.0, (const uint8_t[]){ 6, 5, 7, 4, 8, 3, 9, 2, 10, 1, 11, 0, 12, } }, - { 2, 25, 25, codebook25, 1, -2.0, 1.0, (const uint8_t[]){ 2, 1, 3, 0, 4, } }, - { 2, 169, 169, codebook26, 1, -1530.0, 255.0, (const uint8_t[]){ 6, 5, 7, 4, 8, 3, 9, 2, 10, 1, 11, 0, 12, } }, - { 2, 225, 225, codebook27, 1, -119.0, 17.0, (const uint8_t[]){ 7, 6, 8, 5, 9, 4, 10, 3, 11, 2, 12, 1, 13, 0, 14, } }, - { 2, 289, 289, codebook28, 1, -8.0, 1.0, (const uint8_t[]){ 8, 7, 9, 6, 10, 5, 11, 4, 12, 3, 13, 2, 14, 1, 15, 0, 16, } }, + { 2, 16, 16, 0 }, + { 2, 8, 8, 0 }, + { 2, 256, 256, 0 }, + { 2, 64, 64, 0 }, + { 2, 128, 128, 0 }, + { 2, 32, 32, 0 }, + { 2, 96, 96, 0 }, + { 2, 32, 32, 0 }, + { 2, 96, 96, 0 }, + { 2, 17, 17, 0 }, + { 2, 32, 32, 0 }, + { 2, 78, 78, 0 }, + { 2, 17, 17, 0 }, + { 2, 32, 32, 0 }, + { 2, 78, 78, 0 }, + { 2, 100, 100, 0 }, + { 8, 1641, 6561, 1, -1.0, 1.0 }, + { 4, 443, 625, 1, -2.0, 1.0 }, + { 4, 105, 625, 1, -2.0, 1.0 }, + { 2, 68, 81, 1, -4.0, 1.0 }, + { 2, 81, 81, 1, -4.0, 1.0 }, + { 2, 289, 289, 1, -8.0, 1.0 }, + { 4, 81, 81, 1, -11.0, 11.0 }, + { 2, 121, 121, 1, -5.0, 1.0 }, + { 2, 169, 169, 1, -30.0, 5.0 }, + { 2, 25, 25, 1, -2.0, 1.0 }, + { 2, 169, 169, 1, -1530.0, 255.0 }, + { 2, 225, 225, 1, -119.0, 17.0 }, + { 2, 289, 289, 1, -8.0, 1.0 }, }; static const struct { diff --git a/libavcodec/vorbisenc.c b/libavcodec/vorbisenc.c index 5c10e49b3f..d8d7d4d4eb 100644 --- a/libavcodec/vorbisenc.c +++ b/libavcodec/vorbisenc.c @@ -272,6 +272,7 @@ static int create_vorbis_context(vorbis_enc_context *venc, vorbis_enc_floor *fc; vorbis_enc_residue *rc; vorbis_enc_mapping *mc; + const uint8_t *clens, *quant; int i, book, ret; venc->channels = avctx->channels; @@ -286,6 +287,8 @@ static int create_vorbis_context(vorbis_enc_context *venc, // codebook 0..14 - floor1 book, values 0..255 // codebook 15 residue masterbook // codebook 16..29 residue + clens = codebooks; + quant = quant_tables; for (book = 0; book < venc->ncodebooks; book++) { vorbis_enc_codebook *cb = &venc->codebooks[book]; int vals; @@ -300,8 +303,9 @@ static int create_vorbis_context(vorbis_enc_context *venc, cb->codewords = av_malloc_array(cb->nentries, sizeof(uint32_t)); if (!cb->lens || !cb->codewords) return AVERROR(ENOMEM); - memcpy(cb->lens, cvectors[book].clens, cvectors[book].len); + memcpy(cb->lens, clens, cvectors[book].len); memset(cb->lens + cvectors[book].len, 0, cb->nentries - cvectors[book].len); + clens += cvectors[book].len; if (cb->lookup) { vals = cb_lookup_vals(cb->lookup, cb->ndimensions, cb->nentries); @@ -309,7 +313,7 @@ static int create_vorbis_context(vorbis_enc_context *venc, if (!cb->quantlist) return AVERROR(ENOMEM); for (i = 0; i < vals; i++) - cb->quantlist[i] = cvectors[book].quant[i]; + cb->quantlist[i] = *quant++; } else { cb->quantlist = NULL; }