mirror of
https://github.com/modernuo/ModernUO
synced 2026-08-11 22:23:06 -04:00
0628902644
Follow-ups to #2559, from a review of the ported idle-sleep/scheduler-health changes. ### Fixes - **`NetState.IsIdle` omitted `_pendingDisconnects`** — `Slice()` drains five queues; the property checked four. The other deferred work (`_connectingQueue`, alive checks, movement throttle) is time-gated and correctly excluded; the disconnect queue was the only ready-work omission. Impact was bounded (≤ one idle wait of delay), but the property's contract is "sleeping cannot strand pending work". - **Neither new setting was clamped** (`Main.cs`): - `server.lateWakeThreshold: -1` made `late <= threshold` false for every sample even at zero late wakes, so from the second sample on, sleeping was re-suspended every second, forever — a permanent full-core spin whose only trace was a nonsense warning ("… at least 8ms late 0 time(s)"). - `server.eventLoopIdleWaitMs: -1` disabled sleeping while the admin gump reported **Healthy** (it tested `== 0`). - Both now clamp to `>= 0` and log a warning naming the configured value. `-1` is a natural thing to reach for given the sibling key's doc says "set very high to disable". - **World snapshots were misattributed to `StolenMs`** — `World.Snapshot` ran outside all five profiler phases, so a 3-second save inside a sample read as ~75% stolen, and `debugging-event-loop.md` teaches stolen = "the host ran something else". The diagnostic pointed operators at buying dedicated CPU for their own largest loop-thread stall. Saves now land in a new `WorldSnapshot` phase; `[LoopStats` iterates `PhaseCount` generically, so the report and CSV pick it up with no changes. - **Admin gump conflated host-forced spin with configured spin** — when the startup probe finds no high-resolution wait support it zeroes the idle wait, after which the gump said "Spinning (configured)" and the operator's config said 2. New `Core.IdleSleepUnsupported` property; the gump now shows "Spinning - host cannot honor short waits" as a distinct fourth verdict. A genuinely configured 0 still reads "configured" (the probe only runs when the configured value was > 0). - **The backoff-ceiling `Error` logged once per process lifetime** — `_loggedBackoffCeiling` never reset, and at the ceiling the method returns before the `Warning`, so a host that recovered (>60s clean streak) and later degraded back to the ceiling never re-logged the one operator-actionable message. The flag now resets with the clean-streak escalation reset. - **Removed the unreachable "already suspended, extend" branch** — no sleeps occur while suspended, so `_lateWakes` stays 0 and every suspended sample early-returns before reaching it; with the threshold clamped it can never fire. If sleep gating ever changes, the normal path handles the case by counting a fresh episode. `dev-docs/debugging-event-loop.md` updated to match (phase list + gump verdict table). ### Verification - `dotnet build` clean (0 warnings) both normally and with `-p:EventLoopProfiling=true` (the snapshot phase only becomes live IL under the profiling flag). |
||
|---|---|---|
| .. | ||
| Buffers | ||
| Client | ||
| Collections | ||
| Comparers | ||
| Compression | ||
| Configuration | ||
| Console | ||
| ContextMenus | ||
| Diagnostics | ||
| Events | ||
| Exceptions | ||
| GarbageCollection | ||
| Geometry | ||
| Items | ||
| Json | ||
| Localization | ||
| Maps | ||
| Menus | ||
| Migrations | ||
| Mobiles | ||
| Network | ||
| PropertyList | ||
| Random | ||
| Regions | ||
| Serialization | ||
| Targeting | ||
| Text | ||
| TileMatrix | ||
| Timer | ||
| TimeZones | ||
| Utilities | ||
| World | ||
| AssemblyHandler.cs | ||
| Attributes.cs | ||
| CityInfo.cs | ||
| Commands.cs | ||
| Effects.cs | ||
| EntityFinalizationTracker.cs | ||
| EventLoopTasks.cs | ||
| ExpansionInfo.cs | ||
| FeatureFlags.cs | ||
| Guild.cs | ||
| HuePicker.cs | ||
| IAccount.cs | ||
| IEntity.cs | ||
| Interfaces.cs | ||
| IVirtualCheckGump.cs | ||
| KeywordList.cs | ||
| Main.cs | ||
| MessageType.cs | ||
| Module.cs | ||
| Party.cs | ||
| Poison.cs | ||
| Prompt.cs | ||
| Race.cs | ||
| SecureTrade.cs | ||
| Serial.cs | ||
| Server.csproj | ||
| Skills.cs | ||
| TileData.cs | ||
| TileList.cs | ||