Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#73 closed defect (fixed)

login slow with new os2_randget() in os2ea.c

Reported by: herwigb Owned by: herwigb
Priority: major Milestone: Samba Server for eCS (OS/2) 1.0.1
Component: Samba Server Version: 3.0.33
Keywords: login slow os2_randget() Cc: herwig.bauernfeind@…

Description

The new os2_randget() function in os2ea.c slows down login (and possibly other operations) considerably.

For each login the randbyte_hrtimer() subroutine is executed exactly 129 times, a DosSleep?(5) occurs for every execution, this sums up to a delay of almost 0,7 secs per login.

My suggestion as a fix would be to remove the randbyte_hrtimer() portion completely (reducing DosSleep?(5) to DosSleep?(1) is better, but still slow) as randbyte_hrtimer() is only one of 3 components and the remaining randbyte_sysinfo() and randbyte_perf() still provide more "randomness" than what was available before Changeset #160 and the resulting login speed is perfectly acceptable this way.

An appropriate unified dif is attached.

Attachments (1)

os2ea.zip (757 bytes) - added by herwigb 11 years ago.
os2_randget() with randbyte_hrtimer() removed

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by herwigb

os2_randget() with randbyte_hrtimer() removed

comment:1 Changed 10 years ago by herwigb

  • Owner changed from psmedley to herwigb

comment:2 Changed 10 years ago by herwigb

  • Resolution set to fixed
  • Status changed from new to closed

comment:3 Changed 10 years ago by herwigb

fixed with changeset #162

comment:5 Changed 10 years ago by herwigb

  • Version set to 3.0.33
Note: See TracTickets for help on using tickets.