From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH 2/3] alsa: Fixes Bug#13087 - adds module removal for
 stopping alsa.
Date: Wed, 03 May 2023 13:08:52 +0100
Message-ID: <F8BDD03D-7CE7-4DAA-B318-CAAF08853D6C@ipfire.org>
In-Reply-To: <15b4f52f-d006-41b8-a0b6-23df03046fd9@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2089369282555840675=="
List-Id: <development.lists.ipfire.org>

--===============2089369282555840675==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

What is the benefit of unloading the kernel modules?

Wouldn=E2=80=99t they automatically be reloaded once something triggers udev?

-Michael

> On 3 May 2023, at 12:47, Adolf Belka <adolf.belka(a)ipfire.org> wrote:
>=20
> Hi Arne,
>=20
> On 02/05/2023 10:55, Arne Fitzenreiter wrote:
>> I think this will not work.
>> You have to run "alsctl store" to save the mixer settings before you unloa=
d the modules.
> Thanks for flagging this up. I hadn't thought of that. I will fix it.
>> There are also some *.conf files commented in the rootfiles. Also complain=
s that they are missing since some
>> versions. (still work on my hardware anyways.)
>> Please include it to the package.
> I will find them and uncomment them in the rootfile.
>=20
> What is the best way to submit these updates. Just a v2 update for the patc=
h 2/3 plus another patch for the rootfile update or should I do a v2 patch su=
bmission for the complete set of three patches from the original set together=
 with an additional patch for the rootfile changes?
>=20
> Regards,
>=20
> Adolf.
>> Arne
>> Am 2023-04-21 23:15, schrieb Adolf Belka:
>>> - Stopping alsa or uninstalling it left the sound modules installed
>>> until a reboot was
>>>    carried out. Uninstallation or stopping the kernel modules should
>>> also unload them.
>>>    This patch adds in the modprobe -r commands to unload all the snd
>>> modules installed
>>>    with the start command. The stop command is then added into the
>>> uninstall script to
>>>    remove the modules.
>>>=20
>>> Fixes: Bug#13087
>>> Tested-by: Adolf Belka <adolf.belka(a)ipfire.org>
>>> Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org>
>>> ---
>>>  src/initscripts/packages/alsa | 4 ++++
>>>  1 file changed, 4 insertions(+)
>>>=20
>>> diff --git a/src/initscripts/packages/alsa b/src/initscripts/packages/alsa
>>> index 348e33846..3bfb69f62 100644
>>> --- a/src/initscripts/packages/alsa
>>> +++ b/src/initscripts/packages/alsa
>>> @@ -33,6 +33,10 @@ case "$1" in
>>>          ;;
>>>      stop)
>>>          boot_mesg "Stopping ALSA...    Saving volumes..."
>>> +        modprobe -r snd_pcm_oss >/dev/null 2>&1 || failed=3D1
>>> +        modprobe -r snd_timer >/dev/null 2>&1 || failed=3D1
>>> +        (exit ${failed})
>>> +        evaluate_retval
>>>          loadproc /usr/sbin/alsactl store
>>>          ;;


--===============2089369282555840675==--