Closed Bug 17031 Opened 25 years ago Closed 24 years ago

Necko protocol DLLs should be combined.

Categories

(Core :: Networking, defect, P3)

Product:
Core
Component:
Networking
Platform:
All
Mac System 8.5
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: sfraser_bugs, Assigned: warrensomebody)

References

Details

(Keywords: perf) 

Need to glob together all the small necko protocol DLLs, to make loading more
efficient.
Blocks: 17033
Status: NEW → ASSIGNED
Target Milestone: M12
I thought we had a bug on this already, but I can't find it now.
Blocks: 18471
Blocks: 18951
This bug is marked as for the Mac but I'm assuming this is something we want to
do across platforms?
Yes, I think we should do this xp.
Target Milestone: M12 → M14
This isn't essential for dogfood. Moving to m14/beta.
Bulk move of all Necko (to be deleted component) bugs to new Networking

component.
Keywords: perf
Summary: [Perf] Necko protocol DLLs should be combined. → Necko protocol DLLs should be combined.
Do you expect this to have a significant effect on load-time, enough to put this 
bug on the beta1 radar?
Keywords: beta1
Question from PDT: Is this still an issue? Did we try pulling libs together for 
Mac during October and November, and conclude it was not the win we hoped for??

Thanks,
Jim
We talked about this in the performance meeting today and think it will still be 
a win on Windows and Linux.
PDT wants to know why this is expected to be a win on Mac and Linux when it 
wasn't on Mac.
Adding [NEED INFO] to Status Summary.
Whiteboard: [NEED INFO]
Info from Simon...

Pros:

1. Reduce # items in components folder, reduce enumeration overhead
2. Eliminate per-library loading overhead
3. 'Cleanliness'

Cons:

1. Harder to update just one protocol
2. May cause loading of more unused code at startup

Not essential for beta; do it at your leisure.
Whiteboard: [NEED INFO] → [PDT-][NEED INFO]
Moving to M15.
Target Milestone: M14 → M15
Marking beta2.
Keywords: beta2
Want input from shrimp project as to which protocols they really need. That will 
help decide the best packaging. 

Assigning to selmer to get the input.
Assignee: warren → selmer
Status: ASSIGNED → NEW
Gayatri, this is yours now.  Can you make a quick estimate based on the info you 
got from Varada?
Assignee: selmer → gayatrib
Getting off M15 radar, adding amusil to cc:
Target Milestone: M15 → M16
Status: NEW → ASSIGNED
Keywords: nsbeta2
Taking this back. 
Clearing [NEED INFO] to get on re-eval query for PDT.
Whiteboard: [PDT-][NEED INFO] → [PDT-]
Here's what I think we're going to do:

nknet.dll (new) will contain what is currently in:
  necko.dll
  nkcache.dll
  nkhttp.dll
  nkfile.dll
  nkdata.dll
  nkjar.dll
  nkres.dll
  mimetype.dll
  nkabout.dll
  cnvts.dll
  stremcnv.dll

Then nknet2.dll will contain the rest:
  nkdatetm.dll
  nkfinger.dll
  nkftp.dll
  nkkyword.dll (maybe this should go in the first one, or separate)
Assignee: gayatrib → warren
Status: ASSIGNED → NEW
Keywords: beta1
Whiteboard: [PDT-]
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
It's in. They're called necko.dll and necko2.dll. Keyword is in the first one.
No longer blocks: 18471
No longer blocks: 18951
verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.