190537 - Size/Speed optimizations for OS/2

Status: RESOLVED FIXED

(NSS :: Build, enhancement, P2)

Description

[jhp (no longer active)]

By tweaking how we create import libraries on OS/2, we can achieve around 25%
speedup on load of the DLLs.  Also, by taking advantage of the DEF files already
in the tree, we can make it so the DLLs export less functions, reducing size and
maybe increasing speed.

Comment 1

[jhp (no longer active)]

Attachment: patch

Comment 2

[Mike Kaply [:mkaply]]

This patch doesn't work.

nss3.dll doesn't load because it is looking for softokn.dll when all we have is
softokn3.dll.

It's related to the def file used to create the DLL I think.

Comment 3

[jhp (no longer active)]

Attachment: patch v2

I was not adding the LIBRARY_VERSION to the def file.  This was causing the
internal library name not to match the actuall DLL name.

Comment 4

[jhp (no longer active)]

Attachment: patch v2.1

Remove revision string change from nssckapi

Comment 5

[Wan-Teh Chang]

Comment on attachment 114900 [details] [diff] [review]
patch v2.1

In security/coreconf/OS2.mk, could you try defining
PROCESS_MAP_FILE like this:

PROCESS_MAP_FILE = \
    echo LIBRARY $(LIBRARY_NAME)$(LIBRARY_VERSION) INITINSTANCE TERMINSTANCE >
$@;\
    echo PROTMODE >> $@;\
    echo CODE	 LOADONCALL MOVEABLE DISCARDABLE >> $@;\
    echo DATA	 PRELOAD MOVEABLE MULTIPLE NONSHARED >> $@;\
    echo EXPORTS >> $@;\
    grep -v ';+' $(LIBRARY_NAME).def | grep -v ';-' | \
    sed -e 's; DATA ;;' -e 's,;;,,' -e 's,;.*,,' >> $@

and in security/coreconf/rules.mk, removing the following
change?

>@@ -380,6 +363,13 @@
> 
> $(MAPFILE): $(LIBRARY_NAME).def
> 	@$(MAKE_OBJDIR)
>+ifeq ($(OS_ARCH),OS2)
>+	echo LIBRARY $(LIBRARY_NAME)$(LIBRARY_VERSION) INITINSTANCE TERMINSTANCE > $@
>+	echo PROTMODE >> $@
>+	echo CODE    LOADONCALL MOVEABLE DISCARDABLE >> $@
>+	echo DATA    PRELOAD MOVEABLE MULTIPLE NONSHARED >> $@
>+	echo EXPORTS >> $@
>+endif
> 	$(PROCESS_MAP_FILE)

Comment 6

[jhp (no longer active)]

Attachment: patch v2.2

Patch with wtc's comments.

Comment 7

[Wan-Teh Chang]

Comment on attachment 115016 [details] [diff] [review]
patch v2.2

Have you tested your change to the XP_OS2_EMX part
of mozilla/security/coreconf/OS2.mk?  Does gcc understand
the same *.def file format as VACPP?

Your changes to security/coreconf/rules.mk and
security/nss/lib/ckfw/nssck.api are good.  Mike, could
you review Javier's changes to mozilla/security/coreconf/OS2.mk
too?

Comment 8

[Mike Kaply [:mkaply]]

Comment on attachment 115016 [details] [diff] [review]
patch v2.2

Looks good to me. I had Javier sit down and explain it all.

Comment 9

[Wan-Teh Chang]

Patch checked into the NSS TIP (3.8), NSS_3_7_BRANCH (3.7.2),
and NSS_CLIENT_TAG (mozilla 1.4alpha).