patchDotGNU Portable.NET - Patches: patch #3695, BufferedStream is way to slow

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #3695: BufferedStream is way to slow

Submitter:  Andre Seidelt <Ilu>
Submitted:  Sun 30 Jan 2005 09:49:56 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Open/Closed:  Closed

Wed 02 Feb 2005 12:06:56 PM UTC, comment #2: 

Applied On 02-Feb-2005 (Array.InternalCopy() instead of Array.Copy() method)


Gopal.V <t3rmin4t0r>
Group administrator
Mon 31 Jan 2005 08:53:16 PM UTC, comment #1: 

please do not apply this patch, Rhys suggested a better way and I'll upload a fitting patch soon...

Andre Seidelt <Ilu>
Sun 30 Jan 2005 09:49:56 PM UTC, original submission:  

The reason for this is the usage of Array.Copy() to copy byte-arrays arround. Deserializing ~4mb takes 1:40 with this method resulting in a lot of calls to Array.GetRank() and Array.GetElementType().

As long as Array.Copy is no native method the following patch can reduce the overhead by approx. factor 8 (1:40 without, 0:12 with the patch for me).

The patch was obtained using
iluvatar@elbereth ~/pnet/pnetlib/runtime
$ cvs diff -u3 System/IO/BufferedStream.cs >/tmp/BufferedStream.patch

Andre Seidelt <Ilu>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #8337:  BufferedStream.patch added by Ilu (1017B - application/octet-stream - implements an own ArrayCopy() method for bytes)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 3 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2005-02-02 t3rmin4t0r StatusNone Done
    Open/ClosedOpen Closed
2005-01-30 Ilu Attached File- Added BufferedStream.patch, #4113

Back to the top

Powered by Savane 3.15-94dd.
Corresponding source code