Bug #9936

atomic ops in syscall_mstate() induce significant overhead

Added by John Levon 3 months ago. Updated 2 months ago.

Status:ClosedStart date:2018-10-29
Priority:NormalDue date:
Assignee:John Levon% Done:

100%

Category:-
Target version:-
Difficulty:Medium Tags:needs-triage

Description

As described by Mateusz Guzik:

https://illumos.topicbox.com/groups/discuss/T602520cd39c57fc9/scalability-issues-due-to-5003-want-kstats-for-per-zone-cpu-usage

the atomic operations added in bug 5003 appear to have a significant cost, at least in this
type of load:

illumos5003.png (4.74 KB) John Levon, 2018-10-29 12:16 PM


Related issues

Related to illumos gate - Feature #5003: want kstats for per-zone cpu usage Closed 2014-07-14

History

#1 Updated by Marcel Telka 3 months ago

  • Related to Feature #5003: want kstats for per-zone cpu usage added

#2 Updated by John Levon 3 months ago

Joyent OS-7335

#3 Updated by Electric Monk 2 months ago

  • % Done changed from 0 to 100
  • Status changed from New to Closed

git commit 2918c4a32d09a835c1eba8b0b02fe1dcb7a83175

commit  2918c4a32d09a835c1eba8b0b02fe1dcb7a83175
Author: John Levon <john.levon@joyent.com>
Date:   2018-11-21T20:07:44.000Z

    9936 atomic ops in syscall_mstate() induce significant overhead
    9942 zone secflags are not initialized correctly
    Reviewed by: Patrick Mooney <patrick.mooney@joyent.com>
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Reviewed by: Jerry Jelinek <jerry.jelinek@joyent.com>
    Reviewed by: Andy Fiddaman <andy@omniosce.org>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Approved by: Richard Lowe <richlowe@richlowe.net>

Also available in: Atom