4 replies to this topic

[skaman]

Hi,
i attach some patch who could be of interest to openpli team.

etxx00-disable-media-attach.diff
disabling this feature an "hard depency" will be created between the dvb module and the right frontend and tuners. So, when you install a driver (for example kernel-module-dvb-usb-af9035) it will install also af9033 frontend and the tuners modules

etxx00-postinst.diff
if no kernel is present avoid the upgrade procedure (otherwise the flash will be deleted without new kernel over)

nscd-init.diff
this is a dirty patch.. simply replace the original initd (for bash) with a new one more simple

nscd-postinst-prerm-postrm.diff
add the right opkg postinst, prerm and postrm script for start and stop nscd automatically

twisted-zope-dependency.diff
twisted-core require zope.. otherwise i experienced some green screen on my box

Attached Files

•  oe-patches.zip   7.25KB   38 downloads

Edited by skaman, 22 November 2011 - 02:14.

[pieterg]

Thanks!

etxx00-disable-media-attach.diff
disabling this feature an "hard depency" will be created between the dvb module and the right frontend and tuners. So, when you install a driver (for example kernel-module-dvb-usb-af9035) it will install also af9033 frontend and the tuners modules

Nice one, that's going to save us a lot of autoload entries.

etxx00-postinst.diff
if no kernel is present avoid the upgrade procedure (otherwise the flash will be deleted without new kernel over)

I'm not too worried about a missing kernel image, as postinst only runs after the install, and we know what's in the ipk
But I'll actually replace the /proc/stb check as well.
Having this check means that when drivers fail to load because the kernel has the wrong version, the kernel would fail to update, so things would not fix themselves.

nscd-init.diff
this is a dirty patch.. simply replace the original initd (for bash) with a new one more simple

nscd-postinst-prerm-postrm.diff
add the right opkg postinst, prerm and postrm script for start and stop nscd automatically

OK, we'll discuss whether it makes sense to have ncsd in our image.

twisted-zope-dependency.diff
twisted-core require zope.. otherwise i experienced some green screen on my box

Though python-zopeinterface is installed in our image by default (dependency of twisted-web, dependency of e2-plugin-systemplugins-softwaremanager), I'll add this dependency just in case it'll no longer be provided through other deps in the future.

[Gennar1]

The MEDIA_ATTACH patch works fine and will solve a lot of dependency problems.
But it has a weird effect one the em28xx driver, so now all demod/tuners drivers supported by em28xx are automatically installed and loaded, and they are a lot:

Module				  Size  Used by
em28xx_dvb			  8502  2
em28xx				 87116  1 em28xx_dvb
v4l2_common			 5066  1 em28xx
videodev			   86414  2 em28xx,v4l2_common
videobuf_vmalloc	    4039  1 em28xx
videobuf_core		  15978  2 em28xx,videobuf_vmalloc
tveeprom			   12009  1 em28xx
mt352				   5381  1 em28xx_dvb
tda18271			   51777  1 em28xx_dvb
tuner_xc2028		   19986  1 em28xx_dvb
s5h1409				 9138  1 em28xx_dvb
drxk				   46954  1 em28xx_dvb
s921				    5302  1 em28xx_dvb
drxd				   20499  1 em28xx_dvb
tda10023			    6343  1 em28xx_dvb
lgdt3305			   14600  1 em28xx_dvb
a8293				   1692  1 em28xx_dvb
cxd2820r			   21453  1 em28xx_dvb
tda10071			   13544  1 em28xx_dvb
tda18271c2dd		   15747  1 em28xx_dvb
tuner_simple		   13546  1 em28xx_dvb
tuner_types			 7921  1 tuner_simple
zl10353				 6461  1 em28xx_dvb
lgdt330x			    7594  1 em28xx_dvb

I tried to remove some unused module but as expected the em28xx driver refuses to load with "Unknown symbol" errors.
Not a big problem anyway.

[pieterg]

Yes, that's the purpose of MEDIA_ATTACH, to avoid those hard symbol dependencies
But since we already listed most/all frontend/demod drivers as dependencies in autoload entries, we were actually replicating the behaviour without MEDIA_ATTACH.
So disabling MEDIA_ATTACH is the nicer solution I guess, even if that means we end up loading even more modules now.

[skaman]

etxx00-postinst.diff
if no kernel is present avoid the upgrade procedure (otherwise the flash will be deleted without new kernel over)

I'm not too worried about a missing kernel image, as postinst only runs after the install, and we know what's in the ipk
But I'll actually replace the /proc/stb check as well.
Having this check means that when drivers fail to load because the kernel has the wrong version, the kernel would fail to update, so things would not fix themselves.

My experience was on first boot. After system flash the image start and at first boot opkg run all postinst script and erase my flash.. at reboot the boot loader stuck on missing kernel.
I'm using a more updated version of opkg.. so maybe the problem appear only to me? Theorically the opkg version shouldn't matter

twisted-zope-dependency.diff
twisted-core require zope.. otherwise i experienced some green screen on my box

Though python-zopeinterface is installed in our image by default (dependency of twisted-web, dependency of e2-plugin-systemplugins-softwaremanager), I'll add this dependency just in case it'll no longer be provided through other deps in the future.

This explain my issue.. i removed the default software manager