PCR 6638 - Platform Resource Library systemic event notification needed.
Summary: Platform Resource Library systemic event notification needed.
Status: ASSIGNED
Alias: None
Product: Kernel
Classification: Deos
Component: Kernel (show other PCRs)
Version: mainline
Hardware: All Deos
: Any Upcoming
: Enhancement
Target Milestone: mainline
Assignee: .Kernel
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2010-07-08 08:55 MST by alarson
Modified: 2025-03-27 10:27 MST (History)
1 user (show)

See Also:
Impact Assessment: ---
Organization: DDC-I, Inc.
rroffelsen: Requirements+
rroffelsen: Code+
alarson: TestCases?
alarson: TestProcedures?
alarson: Other?


Attachments

Note You need to log in before you can comment on or make changes to this PCR.
Description alarson 2010-07-08 08:55:00 MST
Platform resource libraries are required to respond to systemic events
powerTransientEx, and frameResyncEx, but they get no notification from
the kernel that such an event has occurred.  The original design
intent was that the library would detect the device state change and
respond accordingly.  However, getting the device to become quiescent
prior to shutdown is desirable, e.g., to handle non-powerTransientEx
related mode changes.  Some mechanism to handle this is desirable.
Comment 1 alarson 2010-07-08 09:00:06 MST
One way to accomplish this goal in a backward compatible manner is to
have the resource library set a hook function in the
platformResourceDescription passed to it, or equivalently, to have the
library call back into the kernel to register the function.  The
events the library might need is still not clear.  Passing
powerTransientEx and frameResyncEx doesn't sound right since those
events happen during recovery, not prior to the event.  Perhaps a
simple "shutting down" notification is adequate.
Comment 2 deosbugs.ccb 2010-07-08 14:08:05 MST
CCB visited this PCR on 2010-07-08.
Comment 3 deosbugs.ccb 2010-07-29 13:05:08 MST
CCB visited this PCR on 2010-07-29.
Comment 4 deosbugs.ccb 2010-09-30 14:52:30 MST
CCB 2 visited this PCR on 2010-09-30.
Comment 5 deosbugs.ccb 2010-12-17 10:18:16 MST
CCB visited this PCR on 2010-12-17.
Comment 6 deosbugs.ccb 2011-03-07 17:39:58 MST
CCB visited this PCR on 2011-03-07
Comment 7 deosbugs.ccb 2011-04-11 14:38:28 MST
CCB visited this PCR on 2011-04-11
Comment 8 deosbugs.ccb 2011-05-16 15:21:03 MST
CCB2 visited this PCR on 2011-05-16
Comment 9 deosbugs.ccb 2011-08-17 15:11:19 MST
CCB visited this PCR on 2011-08-17
Comment 10 deosbugs.ccb 2011-11-04 15:53:03 MST
CCB 2 visited this PCR on 2011-11-04
Comment 11 deosbugs.ccb 2012-01-31 13:46:17 MST
CCB visited this PCR on 2012-01-31
Comment 12 deosbugs.ccb 2012-05-15 08:41:09 MST
CCB visited this PCR on 2012-05-15
Comment 13 deosbugs.ccb 2012-06-08 13:17:36 MST
CCB 2 visited this PCR on 2012-06-08
Comment 14 deosbugs.ccb 2012-08-24 09:16:53 MST
CCB visited this PCR on 2012-08-24
Comment 15 deosbugs.ccb 2012-09-07 10:39:02 MST
CCB visited this PCR on 2012-09-07
Comment 16 rroffelsen 2012-09-17 16:01:17 MST
CCB visited this PCR on 2012-09-17
Comment 17 deosbugs.ccb 2012-11-19 12:20:14 MST
CCB visited this PCR on 2012-11-19
Comment 18 deosbugs.ccb 2012-11-26 17:46:37 MST
CCB visited this PCR on 2012-11-26
Comment 19 deosbugs.ccb 2013-02-12 21:18:02 MST
CCB visited this PCR on 2013-02-12
Comment 20 deosbugs.ccb 2013-03-18 14:35:59 MST
CCB visited this PCR on 2013-03-18
Comment 21 deosbugs.ccb 2013-05-15 10:12:47 MST
CCB visited this PCR on 2013-05-15
Comment 22 deosbugs.ccb 2013-07-19 11:26:25 MST
CCB visited this PCR on 2013-07-19
Comment 23 deosbugs.ccb 2013-11-15 17:40:41 MST
CCB visited this PCR on 2013-11-15
Comment 24 deosbugs.ccb 2014-05-20 10:14:44 MST
CCB 2 visited this PCR on 2014-05-20
Comment 25 deosbugs.ccb 2014-07-14 15:12:20 MST
CCB visited this PCR on 2014-07-14
Comment 26 deosbugs.ccb 2014-11-11 13:17:21 MST
CCB 2 visited this PCR on 2014-11-11
Comment 27 deosbugs.ccb 2014-11-17 08:58:32 MST
CCB visited this PCR on 2014-11-17
Comment 28 deosbugs.ccb 2016-04-18 12:56:55 MST
CCB visited this PCR on 2016-04-18
Comment 29 deosbugs.ccb 2016-06-20 13:09:53 MST
CCB visited this PCR on 2016-06-20
Comment 30 deosbugs.ccb 2017-02-01 09:38:18 MST
CCB visited this PCR on 2017-02-01-59501
Comment 31 deosbugs.ccb 2017-06-28 13:01:23 MST
CCB visited this PCR on 2017-06-28-69227
Comment 32 deosbugs.ccb 2017-07-06 11:07:25 MST
CCB visited this PCR on 2017-07-06-58325
Comment 33 deosbugs.ccb 2021-03-26 09:36:48 MST
CCB visited this PCR on 2021-03-26-57787
Comment 34 deosbugs.ccb 2021-04-05 09:53:54 MST
CCB visited this PCR on 2021-04-05-59141
Comment 35 deosbugs.ccb 2023-08-14 11:06:25 MST
CCB visited this PCR on 2023-08-14-64795
Comment 36 deosbugs.ccb 2023-08-14 12:49:56 MST
This is a desired feature that should be included in an upcoming release if time allows.  It is being taken off of HOLD for consideration for Kismet, since shutdown hook feature is being included.
Comment 37 alarson 2023-12-13 17:41:37 MST
Another item noted by Ron is that the PRLs have no way to raise fatal errors.
Comment 38 deosbugs.ccb 2024-01-23 15:34:18 MST
CCB visited this PCR on 2024-01-23-80895
Comment 39 deosbugs.ccb 2024-04-04 13:19:24 MST
CCB visited this PCR on 2024-04-04-71881
Comment 40 rroffelsen 2024-06-24 10:42:24 MST
Committed SVNRevision 94293.  Applies to PCR 11408, PCR 15060, PCR 6638.

- Added support for kernel mode interceptor libraries. To support this several changes where made:
-- Added getLibrarySymbolAddressPPI().
-- All kernel interface functions are now exported as weak symbols.
-- PRLs are now loaded as if they are in the kernel's needed list before the PAL.
-- Kernel execution now halts if loading a PRL fails for any reason other than the PRL library file not being found.
-- PRL registration function symbol lookup now includes all kernel mode libraries.
-- The kernel now provides weak definitions of all DEOSOPTIONALPALPPI functions.
- Added call to optional PAL provided PALshutdown() function.
- Fix various code trace tag errors that where preventing collection of code trace tags from some files.
Comment 41 Ronald Rische 2024-06-25 16:00:54 MST
Committed SVNRevision 94316.  Applies to PCR 11408, PCR 15060, PCR 6638.

Add new libraryStatus values to test libraryStatus2Str() function for kernel 11.4.0
Comment 42 rroffelsen 2024-07-12 13:55:01 MST
Committed SVNRevision 94532.  Applies to PCR 11408, PCR 15060, PCR 6638.

- Fix PRL initialization function calling bug introduced in SVNRevision 94293
- Resolve some outstanding image code TODOs.
Comment 43 deosbugs.ccb 2024-08-30 06:57:03 MST
CCB visited this PCR on 2024-08-30-49739
Comment 44 deosbugs.ccb 2024-08-30 09:42:00 MST
Developer asserts changes are acceptable for 11.4.0
Comment 45 deosbugs.ccb 2024-12-11 13:04:57 MST
CCB visited this PCR on 2024-12-11-68907
Comment 46 alarson 2025-01-30 13:59:34 MST
Committed SVNRevision 96950.  Applies to PCR 11408, PCR 15060, PCR 6638.

Enable building of kernel instrumented
variant.  The changes made in SVNRevision 94293 introduced a pattern rule for
all .o files that depended on them being C++ files.  Some were assembly so
some extra logic was required.

I verified the binaries generated with the previous and updated code/makefile.mk
were identical.
Comment 47 deosbugs.ccb 2025-03-27 09:32:22 MST
CCB visited this PCR on 2025-03-27-57605
Comment 48 deosbugs.ccb 2025-03-27 10:27:25 MST
Developer asserts updates are acceptable for release 11.5.1.