Closed Bug 17031 Opened 25 years ago Closed 25 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: 25 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.