|
Re: [PATCH] Add performance monitoring unit extension
Hi Greg,
I think there is some confusion here.
The “counter_info” does not related to the mhpmevent CSR value. Rather, the “counter_info” only provides more details about a counter
Hi Greg,
I think there is some confusion here.
The “counter_info” does not related to the mhpmevent CSR value. Rather, the “counter_info” only provides more details about a counter
|
By
Anup Patel
·
#397
·
|
|
Re: [PATCH] Add performance monitoring unit extension
Hi Tommy,
Your suggestion looks good to me as well.
I will update to use counter_info[XLEN-1] as “counter type”.
Regards,
Anup
Hi Tommy,
Your suggestion looks good to me as well.
I will update to use counter_info[XLEN-1] as “counter type”.
Regards,
Anup
|
By
Anup Patel
·
#396
·
|
|
Re: [PATCH] Add performance monitoring unit extension
Hi Brian,
Sure, I will include an explicit statement about “SBI PMU extension providing a perf compatible event …” (like you suggested).
Regards,
Anup
Hi Brian,
Sure, I will include an explicit statement about “SBI PMU extension providing a perf compatible event …” (like you suggested).
Regards,
Anup
|
By
Anup Patel
·
#395
·
|
|
Re: [PATCH] Add performance monitoring unit extension
Ok, that sounds completely reasonable.
--Sean
Ok, that sounds completely reasonable.
--Sean
|
By
Sean Anderson
·
#394
·
|
|
Re: Agenda for Platform Spec call on 2 Nov 2020
Additional discussion at [1]. My objections are particularly motivated
by the RFC implementation of this spec for opensbi which just does
hang:
wfi
j hang
if it doesn't have a more specific
Additional discussion at [1]. My objections are particularly motivated
by the RFC implementation of this spec for opensbi which just does
hang:
wfi
j hang
if it doesn't have a more specific
|
By
Sean Anderson
·
#393
·
|
|
Re: [PATCH] Add performance monitoring unit extension
I want to ask or encourage that this not be done. Here's why:
It will be desirable for new CPU implementations to adopt a mhpmevent CSR format that matches what is naturally coming through OpenSBI -
I want to ask or encourage that this not be done. Here's why:
It will be desirable for new CPU implementations to adopt a mhpmevent CSR format that matches what is naturally coming through OpenSBI -
|
By
Greg Favor
·
#392
·
|
|
Re: [PATCH] Add performance monitoring unit extension
Yes.
--Sean
By
Sean Anderson
·
#391
·
|
|
Re: [PATCH] Add performance monitoring unit extension
That wouldn’t help 64-bit targets much. Did you mean bit XLEN-1?
Tommy
That wouldn’t help 64-bit targets much. Did you mean bit XLEN-1?
Tommy
|
By
Tommy Thorn <tommy.thorn@...>
·
#390
·
|
|
Agenda for Platform Spec call on 2 Nov 2020
# Profiles and Platform Spec Technical Subcommittee (P2TS) Agenda: 2 Nov
# 2020
## Meeting Info: weekly on Mondays (09:00 PST, 16:00 UTC)
Details:
# Profiles and Platform Spec Technical Subcommittee (P2TS) Agenda: 2 Nov
# 2020
## Meeting Info: weekly on Mondays (09:00 PST, 16:00 UTC)
Details:
|
By
Al Stone
·
#389
·
|
|
Re: Add OpenSBI performance monitoring unit extension
Anup,
For HARDWARE events, event_idx.code encodes a standard set of events starting from encoding '0'. Insofar as it would be very nice for this to be writeable as is into an mhpmevent CSR without
Anup,
For HARDWARE events, event_idx.code encodes a standard set of events starting from encoding '0'. Insofar as it would be very nice for this to be writeable as is into an mhpmevent CSR without
|
By
Greg Favor
·
#388
·
|
|
Re: [PATCH 1/1] Proposal to assign SBI Implementation ID 5 to Diosix
It's so great to see more and more Rust implementations!
Awesome! I like how you have split the crates out. Any plan on
submitting them to crates.io? With more and more embedded Rust work
ongoing it
It's so great to see more and more Rust implementations!
Awesome! I like how you have split the crates out. Any plan on
submitting them to crates.io? With more and more embedded Rust work
ongoing it
|
By
Alistair Francis <alistair.francis@...>
·
#387
·
|
|
Re: [PATCH 1/1] Proposal to assign SBI Implementation ID 5 to Diosix
By
Anup Patel
·
#386
·
|
|
[PATCH 1/1] Proposal to assign SBI Implementation ID 5 to Diosix
Hello all,
This is a proposal to request SBI Implementation ID 5 for the
open-source Diosix project: https://github.com/diodesign/diosix
Diosix is a bare-metal hypervisor written in Rust for 32- and
Hello all,
This is a proposal to request SBI Implementation ID 5 for the
open-source Diosix project: https://github.com/diodesign/diosix
Diosix is a bare-metal hypervisor written in Rust for 32- and
|
By
Chris Williams
·
#385
·
|
|
Re: Proposal v2: Intro to the Spec and Profiles
OK, these are good answers, but to reword it:
We expect all Linux, hypervisors, and distros can be compiled to work with all optional features of a required extension (including privileged and
OK, these are good answers, but to reword it:
We expect all Linux, hypervisors, and distros can be compiled to work with all optional features of a required extension (including privileged and
|
By
Allen Baum
·
#384
·
|
|
Re: [PATCH 2/4] sbi: Define some terminology
Right, but the SBI spec is considering the S-mode execution environment.
Whether it's still a hart to M-mode is irrelevant (and if the hart was
powered down or physically removed it might not be a
Right, but the SBI spec is considering the S-mode execution environment.
Whether it's still a hart to M-mode is irrelevant (and if the hart was
powered down or physically removed it might not be a
|
By
Sean Anderson
·
#383
·
|
|
Re: [PATCH 1/4] sbi: Add a bookmark for the HSM extension
Reviewed-by: Atish Patra <atish.patra@...>
--
Regards,
Atish
Reviewed-by: Atish Patra <atish.patra@...>
--
Regards,
Atish
|
By
atishp@...
·
#382
·
|
|
Re: [PATCH 4/4] sbi: Specify the initial state
I think this belongs in the SBI specification. There are multiple issues
about information expected to be in this ([1], [2]). In addition, some
concepts such as the HSM, or the placement of the hartid
I think this belongs in the SBI specification. There are multiple issues
about information expected to be in this ([1], [2]). In addition, some
concepts such as the HSM, or the placement of the hartid
|
By
Sean Anderson
·
#381
·
|
|
Re: [PATCH 4/4] sbi: Specify the initial state
Does it need to be specified in SBI specification or platform
specification ? Platform specification anyways should have a section
about state of machine while entering M/S mode.
Keeping the same
Does it need to be specified in SBI specification or platform
specification ? Platform specification anyways should have a section
about state of machine while entering M/S mode.
Keeping the same
|
By
atishp@...
·
#380
·
|
|
Re: [PATCH 3/4] sbi: Specify the instruction set
I am fine with either way.
--
Regards,
Atish
I am fine with either way.
--
Regards,
Atish
|
By
atishp@...
·
#379
·
|
|
Re: [PATCH 2/4] sbi: Define some terminology
There is a brief valid hart definition in binary encoding section as
well.
"any of the hartid from hart_mask is not valid i.e. either the hartid
is not enabled by the platform or is not available to
There is a brief valid hart definition in binary encoding section as
well.
"any of the hartid from hart_mask is not valid i.e. either the hartid
is not enabled by the platform or is not available to
|
By
atishp@...
·
#378
·
|