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 BFC3742AB6 for ; Tue, 11 Jan 2022 11:20:23 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 1B6DF68AF2A; Tue, 11 Jan 2022 13:20:21 +0200 (EET) Received: from out203-205-221-153.mail.qq.com (out203-205-221-153.mail.qq.com [203.205.221.153]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6375968AEE4 for ; Tue, 11 Jan 2022 13:20:12 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foxmail.com; s=s201512; t=1641900007; bh=hy8XHDF1JpiB8qVW0amZu9aVnGkgky0tInwE9/8Kxzs=; h=From:Subject:Date:References:To:In-Reply-To; b=PMvtnQGiqPns1UNA9P1wlVUIsXTbMRs4/5H9fm1Ou9z3g8TxO610KYd4FY1Q+2oSV gTpqjy7PzioFvAExm8UmVFsIB9IOUKIE9eg7l8jhzj5GINVliC3zmziBoYx2rwv0NE 445YXzX9pg8NALFdV1pEPO4uKUNhOVMl6mJTa5tE= Received: from [192.168.255.10] ([113.108.77.69]) by newxmesmtplogicsvrszb7.qq.com (NewEsmtp) with SMTP id 506A6CB9; Tue, 11 Jan 2022 19:20:06 +0800 X-QQ-mid: xmsmtpt1641900006tjmkcjr1l Message-ID: X-QQ-XMAILINFO: MShfLn39PbN2s2A+4jt7T2m2msdOwZEPFURKiAyzvhzP8AYyA2rZ89HHMbudUf VULW+uqFxTbn3r6Rt6WixAKhvl2alEavTsGWyWTRQeAQTdNCYkt4BEk9LlIvaPeLJPoAaw1KpWNV /cdC58mKzUcysT5Amb+T7OpB9dJj0u3Lml/SZo7J34lUkgyae4Q715Apbi0lKSOqFd5SVzA7LI7Y 3peoQornvlZHrLscFd6Yo+3k0RxEsOBQ+iK1cbyrhp2XJmGA474CVXDqstF7ArkhZBehEBtkRM2l ZEUCF7cxMqEUp7bmm/vVrRmWgWDbNU5Rbz+nsiI0mKZUhRIKftrtOZMun9zkUYV5sE7mBVi1m3VA sTglcDD9a5JsGv4LYVCCXQGgA8u23ALr86mo0WMaqhUOe1K9FtyD4IgVxX2QXYtqj6xU2DJWwCNx IwqajOwaMFUnp6HA25Z6uzKgLCqjZuqx65hcM3C0DACGUtasQdVR+XEX5e0ZUO41NFH3RWScH9rY q8o4f0XdqoEYiI8JN5Z/owgdZGLYLMGpEd/OrBIRyrCCPqiSZ2nW7t0BlJpdlGzJzKPFR3znAY9d ujRpAry59gb8arJhnZnJYBwId6k+WXugU24iBgOQSnCpauZT1V2GjZBiSpVv/0PqqgZ1I2OVrP/K DrMDJoM31sSbgUPfteI9MqwX+Wu2UmK3PzCYTt891hSbi2cVY4G5hoebmCFsIpxZ/fqTlnOX5cjv EEkCPlbujqSywiaQAK5hSFgfxYj0MfdbY5U/R1u0dUfp0ezrSEl1u7yIJeDbs6pkcXtGnjLKhbmQ lI7JcwTfAGDcrOeytBHDwIqZqcjxrMiZFjqEsi7UOwmCFmZ0HqLLuxiLdeayapkDA= From: =?utf-8?B?InpoaWxpemhhbyjotbXlv5fnq4spIg==?= Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.60.0.2.21\)) Date: Tue, 11 Jan 2022 19:20:07 +0800 References: <20220111092338.194468-1-gijs@peskens.net> <20220111103408.201161-1-gijs@peskens.net> To: FFmpeg development discussions and patches In-Reply-To: <20220111103408.201161-1-gijs@peskens.net> X-OQ-MSGID: <6346FD08-0BF7-46BA-9375-C1EB08FE71BA@foxmail.com> X-Mailer: Apple Mail (2.3654.60.0.2.21) Subject: Re: [FFmpeg-devel] [PATCH] libRIST: allow setting fifo size and fail on overflow. 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: > On Jan 11, 2022, at 6:34 PM, Gijs Peskens wrote: > > Introduce fifo_size and overrun_nonfatal params to configure fifo buffer > behavior. > > Use newly introduced RIST_DATA_FLAGS_OVERFLOW flag to check for overrun > and error out in that case. > --- > doc/protocols.texi | 9 +++++++++ > libavformat/librist.c | 37 +++++++++++++++++++++++++++++++++++++ > 2 files changed, 46 insertions(+) > > diff --git a/doc/protocols.texi b/doc/protocols.texi > index d207df0b52..f1acf0cc77 100644 > --- a/doc/protocols.texi > +++ b/doc/protocols.texi > @@ -745,6 +745,15 @@ Set internal RIST buffer size in milliseconds for retransmission of data. > Default value is 0 which means the librist default (1 sec). Maximum value is 30 > seconds. > > +@item fifo_size > +Size of the librist receiver output fifo in number of packets. This must be a > +power of 2. > +Defaults to 8192 (vs the libRIST default of 1024). > + > +@item overrun_nonfatal=@var{1|0} > +Survive in case of libRIST fifo buffer overrun. Default > +value is 0. > + > @item pkt_size > Set maximum packet size for sending data. 1316 by default. > > diff --git a/libavformat/librist.c b/libavformat/librist.c > index 378b635ea7..53f83d3668 100644 > --- a/libavformat/librist.c > +++ b/libavformat/librist.c > @@ -43,6 +43,9 @@ > ((patch) + ((minor)* 0x100) + ((major) *0x10000)) > #define FF_LIBRIST_VERSION FF_LIBRIST_MAKE_VERSION(LIBRIST_API_VERSION_MAJOR, LIBRIST_API_VERSION_MINOR, LIBRIST_API_VERSION_PATCH) > #define FF_LIBRIST_VERSION_41 FF_LIBRIST_MAKE_VERSION(4, 1, 0) > +#define FF_LIBRIST_VERSION_42 FF_LIBRIST_MAKE_VERSION(4, 2, 0) > + > +#define FF_LIBRIST_FIFO_SIZE_DEFAULT 8192 > > typedef struct RISTContext { > const AVClass *class; > @@ -52,6 +55,8 @@ typedef struct RISTContext { > int packet_size; > int log_level; > int encryption; > + int fifo_size; > + int overrun_nonfatal; > char *secret; > > struct rist_logging_settings logging_settings; > @@ -70,6 +75,8 @@ static const AVOption librist_options[] = { > { "main", NULL, 0, AV_OPT_TYPE_CONST, {.i64=RIST_PROFILE_MAIN}, 0, 0, .flags = D|E, "profile" }, > { "advanced", NULL, 0, AV_OPT_TYPE_CONST, {.i64=RIST_PROFILE_ADVANCED}, 0, 0, .flags = D|E, "profile" }, > { "buffer_size", "set buffer_size in ms", OFFSET(buffer_size), AV_OPT_TYPE_INT, {.i64=0}, 0, 30000, .flags = D|E }, > + { "fifo_size", "Set libRIST fifo buffer. Size must be power of 2", OFFSET(fifo_size), AV_OPT_TYPE_INT, {.i64=FF_LIBRIST_FIFO_SIZE_DEFAULT}, 2 << 9, 2 << 15, .flags = D|E }, > + { "overrun_nonfatal", "survive in case of libRIST receiving circular buffer overrun", OFFSET(overrun_nonfatal), AV_OPT_TYPE_INT, {.i64 = 0}, 0, 1, D }, This is not what I mean. We use int type to hold the result of AV_OPT_TYPE_BOOL configuration. If overrun_nonfatal has bool type, it can trigger such error: src/libavutil/opt.c:129:9: runtime error: store to misaligned address 0x7fbf454121a3 for type 'int', which requires 4 byte alignment For AVOption, the type is AV_OPT_TYPE_BOOL. > { "pkt_size", "set packet size", OFFSET(packet_size), AV_OPT_TYPE_INT, {.i64=1316}, 1, MAX_PAYLOAD_SIZE, .flags = D|E }, > { "log_level", "set loglevel", OFFSET(log_level), AV_OPT_TYPE_INT, {.i64=RIST_LOG_INFO}, -1, INT_MAX, .flags = D|E }, > { "secret", "set encryption secret",OFFSET(secret), AV_OPT_TYPE_STRING,{.str=NULL}, 0, 0, .flags = D|E }, > @@ -161,6 +168,20 @@ static int librist_open(URLContext *h, const char *uri, int flags) > if (ret < 0) > goto err; > > + //Prior to 4.2.0 there was a bug in libRIST which made this call always fail. > +#if FF_LIBRIST_VERSION >= FF_LIBRIST_VERSION_42 > + if (flags & AVIO_FLAG_READ) { > + ret = rist_receiver_set_output_fifo_size(s->ctx, s->fifo_size); > + if (ret != 0) { > + goto err; > + } > + } > +#else > + if (s->fifo_size != FF_LIBRIST_FIFO_SIZE_DEFAULT) { > + av_log(h, AV_LOG_ERROR, "libRIST prior to 0.2.7 has a bug which fails setting the fifo buffer size\n"); > + } > +#endif > + > if (((s->encryption == 128 || s->encryption == 256) && !s->secret) || > ((peer_config->key_size == 128 || peer_config->key_size == 256) && !peer_config->secret[0])) { > av_log(h, AV_LOG_ERROR, "secret is mandatory if encryption is enabled\n"); > @@ -223,8 +244,24 @@ static int librist_read(URLContext *h, uint8_t *buf, int size) > return AVERROR_EXTERNAL; > } > > +#if FF_LIBRIST_VERSION >= FF_LIBRIST_VERSION_42 > + if (data_block->flags & RIST_DATA_FLAGS_OVERFLOW == RIST_DATA_FLAGS_OVERFLOW) { > + if (!s->overrun_nonfatal) { > + av_log(h, AV_LOG_ERROR, "Fifo buffer overrun. " > + "To avoid, increase fifo_size URL option. " > + "To survive in such case, use overrun_nonfatal option\n"); > + size = AVERROR(EIO); > + goto out_free; > + } else { > + av_log(h, AV_LOG_WARNING, "Fifo buffer overrun. " > + "Surviving due to overrun_nonfatal option\n"); > + } > + } > +#endif > + > size = data_block->payload_len; > memcpy(buf, data_block->payload, size); > +out_free: > #if FF_LIBRIST_VERSION < FF_LIBRIST_VERSION_41 > rist_receiver_data_block_free((struct rist_data_block**)&data_block); > #else > -- > 2.32.0 > > _______________________________________________ > 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". _______________________________________________ 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".