From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Fri, 17 Jun 2022 12:28:11 +0200 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1o29Cz-0056SE-UM for lore@lore.pengutronix.de; Fri, 17 Jun 2022 12:28:11 +0200 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1o29D0-0002dm-DR; Fri, 17 Jun 2022 12:28:10 +0200 Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1o29Ch-0002dS-6B; Fri, 17 Jun 2022 12:27:51 +0200 Received: from mol by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1o29Cg-0006zK-Th; Fri, 17 Jun 2022 12:27:50 +0200 Date: Fri, 17 Jun 2022 12:27:50 +0200 From: Michael Olbrich To: Enrico =?iso-8859-1?Q?J=F6rns?= Message-ID: <20220617102750.GQ13709@pengutronix.de> Mail-Followup-To: Enrico =?iso-8859-1?Q?J=F6rns?= , ptxdist@pengutronix.de, jlu@pengutronix.de References: <20220615142217.990960-1-ejo@pengutronix.de> <20220615142217.990960-3-ejo@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain User-Agent: Mutt/1.10.1 (2018-07-13) Subject: Re: [ptxdist] [PATCH 2/5] RAUC: add option to enable streaming support X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: ptxdist@pengutronix.de, jlu@pengutronix.de Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false On Fri, Jun 17, 2022 at 11:47:19AM +0200, Enrico Jörns wrote: > Am Freitag, dem 17.06.2022 um 08:28 +0200 schrieb Michael Olbrich: > > On Wed, Jun 15, 2022 at 04:22:14PM +0200, Enrico Jorns wrote: > > > Enable it by default so that default installation can benefit from this. > > > Can be deactivated for saving space or for deactivating network > > > functionality. > > > Web server must support HTTP range requests for this and NBD support > > > must be enabled in kernel. > > > > > > Signed-off-by: Enrico Jorns > > > --- > > > rules/rauc.in | 11 +++++++++++ > > > rules/rauc.make | 1 + > > > 2 files changed, 12 insertions(+) > > > > > > diff --git a/rules/rauc.in b/rules/rauc.in > > > index aee162b83..832967b44 100644 > > > --- a/rules/rauc.in > > > +++ b/rules/rauc.in > > > @@ -13,6 +13,7 @@ menuconfig RAUC > > > select BUSYBOX_FEATURE_SEAMLESS_GZ if BUSYBOX_TAR > > > select DBUS if RAUC_SERVICE > > > select LIBCURL if RAUC_NETWORK > > > + select LIBNL3 if RAUC_STREAMING > > > select JSON_GLIB if RAUC_JSON > > > select UTIL_LINUX_NG_LIBFDISK if RAUC_GPT > > > select SQUASHFS_TOOLS if RUNTIME > > > @@ -44,6 +45,16 @@ config RAUC_NETWORK > > > Note that network is primarily designed to be used for RAUC'S casync > > > capabilities, not for fetching full bundles. > > > > > > +config RAUC_STREAMING > > > + bool > > > + prompt "Enable streaming support" > > > + default y > > > > Why 'default y'? I think most users won't need this. > > I am not convinced that streaming is not required for most users. I guess that > being able to download bundles form HTTP directly is a use case that is more and > more common. With network and streaming enabled, one can download bundles only > externally or install from USB sticks etc. > > In Yocto we have this enabled by default, too. If it's ptxdist's philosophy to > ship as little features as possible by default (i.e. focus on small size), we > can also make this default n. The thing is, nobody changes the defaults unless they notice something is not working as expected. And enabling things in PTXdist is easy. How does this work anyways? If I call 'rauc install ', will this use streaming implicitly or does it require an explicit option? How about 'default RAUC_NETWORK'? This way those that use networking also get streaming by default. I can do a fixup here if this makes sense. Michael > > > + select RAUC_NETWORK > > > + help > > > + Enables built-in HTTP(S) streaming support that allows RAUC to > > > + access remote bundles via http/https and write images without the > > > + need for intermediate storage on the target. > > > + > > > config RAUC_JSON > > > bool > > > prompt "JSON support" > > > diff --git a/rules/rauc.make b/rules/rauc.make > > > index 46d196bed..820583e59 100644 > > > --- a/rules/rauc.make > > > +++ b/rules/rauc.make > > > @@ -43,6 +43,7 @@ RAUC_CONF_OPT := \ > > > --disable-valgrind \ > > > --$(call ptx/endis,PTXCONF_RAUC_SERVICE)-service \ > > > --$(call ptx/endis,PTXCONF_RAUC_NETWORK)-network \ > > > + --$(call ptx/endis,PTXCONF_RAUC_STREAMING)-streaming \ > > > --$(call ptx/endis,PTXCONF_RAUC_JSON)-json \ > > > --$(call ptx/endis,PTXCONF_RAUC_GPT)-gpt \ > > > --with-systemdunitdir=/usr/lib/systemd/system \ > > > -- > > > 2.30.2 > > > > > > > > > > > > > -- > Pengutronix e.K.                           | Enrico Jörns                | > Embedded Linux Consulting & Support        | https://www.pengutronix.de/ | > Steuerwalder Str. 21                       | Phone: +49-5121-206917-180  | > 31137 Hildesheim, Germany                  | Fax:   +49-5121-206917-9    | > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |