945136 - disable drmserver and installd

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Andreas Gal :gal]

A chipset vendor requested that we disable starting drmserver and installd via build/core/user_tags.mk since we use neither. Ideally we shouldn't build and install them either.

Comment 1

[Keven Kuo [:kkuo]]

Hi! Thomas,

Over to you. Thanks.

--
Keven

Comment 2

[James Zhang (Spreadtrum)]

Attachment: 945136.patch

I have verified on fugu.

Comment 3

[thomas tsai]

how about these 3 .so? They are still built and included in the rom.
	libfwdlockengine.so
	libdrm1_jni.so
	libdrm1.so

Comment 4

[thomas tsai]

any comment if all these DRM related are removed?
    drmserver.so
    libdrmframework.so
    libdrmframework_jni.so
libfwdlockengine.so
	libdrm1_jni.so
	libdrm1.so

Comment 5

[James Zhang (Spreadtrum)]

(In reply to thomas tsai from comment #3)
> how about these 3 .so? They are still built and included in the rom.
> 	libfwdlockengine.so
> 	libdrm1_jni.so
> 	libdrm1.so

Remove these 3 .so will cause build error.

Comment 6

[Andreas Gal :gal]

We can add a repo that exports dummy versions of those libraries. We did that for other Dalvik and Jni stuff too. Saved memory on disk and ram.

Comment 7

[Marco Chen [:mchen]]

Currently /system/bin/drmserver is running on the system.
We need to remove it from init.rc too.

Comment 8

[Marco Chen [:mchen]]

(In reply to Andreas Gal :gal from comment #6)
> We can add a repo that exports dummy versions of those libraries. We did
> that for other Dalvik and Jni stuff too. Saved memory on disk and ram.

We need to find out the dependency and to see whether we can remove that dependency chain entirely.
Or maybe we can do fakedrmXXX to expose necessary APIs.

Comment 9

[James Zhang (Spreadtrum)]

(In reply to Marco Chen [:mchen] from comment #7)
> Currently /system/bin/drmserver is running on the system.
We need to remove
> it from init.rc too.

If drmserver hasn't been built, we need not remove it from init.rc.

Comment 10

[Michael Wu [:mwu]]

In general *_jni.so libraries aren't necessary. I remember removing all of them, though.

Comment 11

[James Zhang (Spreadtrum)]

(In reply to Michael Wu [:mwu] from comment #10)
> In general *_jni.so libraries aren't necessary. I remember removing all of
> them, though.

Please check B2G/out/target/product/<device>/installed-files.txt, you can find many *jni*.so.

Comment 12

[James Zhang (Spreadtrum)]

Michael, please remove all unnecessary jni so to shrink ROM size, it's very important to tarako device.

Comment 13

[Marco Chen [:mchen]]

(In reply to James Zhang from comment #12)
> Michael, please remove all unnecessary jni so to shrink ROM size, it's very
> important to tarako device.

Hi James,

May I know why shrink ROM size is important to tarako device too (not only RAM)?
Because these JNI libraries are only about 200k Bytes only.

Comment 14

[James Zhang (Spreadtrum)]

(In reply to Marco Chen [:mchen] [PTO from 2013/12/24 ~ 2014/01/02] from comment #13)
> (In reply to James Zhang from comment #12)
> > Michael, please remove all unnecessary jni so to shrink ROM size, it's very
> > important to tarako device.
> 
> Hi James,
> 
> May I know why shrink ROM size is important to tarako device too (not only
> RAM)?
> Because these JNI libraries are only about 200k Bytes only.

I have removed all unused JNI libraries on gonk side. We also need shrink ROM size to 100M for fastboot.

Comment 15

[James Zhang (Spreadtrum)]

Fixed on gonk side, please sync tarako code.

Comment 16

[Marco Chen [:mchen]]

(In reply to James Zhang from comment #14)
> I have removed all unused JNI libraries on gonk side. We also need shrink
> ROM size to 100M for fastboot.

Could I know the detail of what there is a limitation of 100MB on fastboot?
If it is caused by memory then could we do traffic control?

(Thanks for your kindly response. I just want to make sure the reason of shrinking ROM size.)

Comment 17

[James Zhang (Spreadtrum)]

(In reply to Marco Chen [:mchen] [PTO from 2013/12/24 ~ 2014/01/02] from comment #16)
> (In reply to James Zhang from comment #14)
> > I have removed all unused JNI libraries on gonk side. We also need shrink
> > ROM size to 100M for fastboot.
> 
> Could I know the detail of what there is a limitation of 100MB on fastboot?
> If it is caused by memory then could we do traffic control?i
> 
> (Thanks for your kindly response. I just want to make sure the reason of
> shrinking ROM size.)

We use u-boot for fastboot,  it'll store the image in ram and the available memory is about 100MB.

Comment 18

[Andrew Overholt [:overholt]]

What's the status here?  Does this still need to be a 1.3 blocker or have the fixes landed?

Comment 19

[James Zhang (Spreadtrum)]

fixed on my side.

Comment 20

[Joe Cheng [:jcheng] (please needinfo)]

add [POVB] and 1.3T+ so it gets tracked for tarako

Comment 21

[[:fabrice] Fabrice Desré]

Marking 1.3t fixed per comment 19