Project

General

Profile

Feature #15660

Milestone #15372: art multi-threading phase 1

Feature #15374: Modifying all framework-provided services to be thread safe.

Make FloatingPointControl thread safe

Added by Kyle Knoepfel over 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
02/24/2017
Due date:
% Done:

100%

Estimated time:
16.00 h
Spent time:
Scope:
Internal
Experiment:
-
SSI Package:
art
Duration:

Description

This likely means making the service control only global state and not per-module configuration.

Associated revisions

Revision dab11294 (diff)
Added by Kyle Knoepfel over 2 years ago

Resolve issue #15660: Adjust FloatingPointControl to control global FPU behavior.

History

#1 Updated by Kyle Knoepfel over 2 years ago

It turns out that the floating-point environment is thread_local. This implies that per-module behavior is possible, presuming that the system is aware of multiple streams/threads. This service will take more analysis.

#2 Updated by Kyle Knoepfel over 2 years ago

  • % Done changed from 0 to 10

Changed the callbacks to private access. Creating a ServiceHandle<FloatingPointControl> will yield a handle that has no interface. This is a potentially breaking change for users, and we will list it as such.

Implemented with commit art:fbed0673.

#3 Updated by Kyle Knoepfel over 2 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 10 to 100

An email has been sent to the art users and stakeholders, notifying them of our intention to remove the per-module functionality of this service. Anticipating no objections, I have moved ahead and removed the per-module facility.

Implemented with commit art:dab11294.

#4 Updated by Kyle Knoepfel over 2 years ago

  • Target version set to 1209

#5 Updated by Kyle Knoepfel about 2 years ago

  • Target version changed from 1209 to 2.07.01

#6 Updated by Kyle Knoepfel about 2 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF