Received-SPF: pass (google.com: domain of linux-kernel+bounces-179609-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Wed, 15 May 2024 09:50:35 +0200
From: Lukasz Majewski <lukma@denx.de>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: Jakub Kicinski <kuba@kernel.org>, netdev@vger.kernel.org, Paolo Abeni
 <pabeni@redhat.com>, Eric Dumazet <edumazet@google.com>, Vladimir Oltean
 <olteanv@gmail.com>, "David S. Miller" <davem@davemloft.net>, Oleksij
 Rempel <o.rempel@pengutronix.de>, Tristram.Ha@microchip.com, Ravi
 Gunasekaran <r-gunasekaran@ti.com>, Simon Horman <horms@kernel.org>, Nikita
 Zhandarovich <n.zhandarovich@fintech.ru>, Murali Karicheri
 <m-karicheri2@ti.com>, Arvid Brodin <Arvid.Brodin@xdin.com>, Dan Carpenter
 <dan.carpenter@linaro.org>, "Ricardo B. Marliere" <ricardo@marliere.net>,
 Casper Andersson <casper.casan@gmail.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] net: hsr: Setup and delete proxy prune timer only when
 RedBox is enabled
Message-ID: <20240515095035.1c6a8e47@wsk>
In-Reply-To: <20240515071448.Vf_t99dI@linutronix.de>
References: <20240514091306.229444-1-lukma@denx.de>
	<20240515064139.-B-_Hf0_@linutronix.de>
	<20240515090904.477c6b5f@wsk>
	<20240515071448.Vf_t99dI@linutronix.de>
Organization: denx.de
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="Sig_/=iNR8my_p0HXF67qrEs5xe0";
 protocol="application/pgp-signature"; micalg=pgp-sha512

--Sig_/=iNR8my_p0HXF67qrEs5xe0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

Hi Sebastian,

> On 2024-05-15 09:09:04 [+0200], Lukasz Majewski wrote:
> > Hi Sebastian, =20
> Hi Lukasz,
>=20
> > My concern is only with resource allocation - when RedBox is not
> > enabled the resources for this particular, not used timer are
> > allocated anyway. =20
>=20
> timer_setup() does not allocate any resources. The initialisation is
> pure static assignment. The timer subsystem does not look at this
> timer until mod_timer() is invoked (or something similar).

Thanks for the clarification.

Considering the above - please just drop this patch.

>=20
> > If this can be omitted - then we can drop the patch.
> >=20
> > Best regards,
> >=20
> > Lukasz Majewski =20
>=20
> Sebastian


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de

--Sig_/=iNR8my_p0HXF67qrEs5xe0
Content-Type: application/pgp-signature
Content-Description: OpenPGP digital signature

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

iQEzBAEBCgAdFiEEgAyFJ+N6uu6+XupJAR8vZIA0zr0FAmZEaUsACgkQAR8vZIA0
zr28cAf/c81TJAjvacDjJZwGsTLd9w7jFwu4PZCY5cfl+cd73FlQSoNiMk/DdRrL
CSOoMxmFXrIT5dn0E9tAZpbphGaAjf8tJdH5E04RVTwiwNx2EgjOj7ePVvo5SX9P
p2JRFNhr0ZMg+AfeX6ybhWpPsgumbDyfbUlCV+DZYskMV0Ir71PKY5bC7tRGl/cV
XQC6F97d3pJFDAezcDoBLN5fQ7jp62/ZDVFoaztJkd0Vj5BTjI4eN/VoEvpxTu5m
EHi1HVUc3v9R0h6vHwpLvyMViIDXWGsCR3XUE+Aeebaz8SspUKVMTBSUv/CapekN
gUDt8jO3skZCNXRpgbxK+oZZFAu1Fg==
=tCGb
-----END PGP SIGNATURE-----

--Sig_/=iNR8my_p0HXF67qrEs5xe0--