* RFC: ASRockRack E3C246D4I port
@ 2021-03-12 2:45 Zev Weiss
2021-03-24 1:50 ` Joel Stanley
0 siblings, 1 reply; 4+ messages in thread
From: Zev Weiss @ 2021-03-12 2:45 UTC (permalink / raw)
To: openbmc
Hello all,
We (Equinix Metal, formerly known as Packet) having been working on a
port of OpenBMC to the ASRockRack E3C246D4I board
(https://www.asrockrack.com/general/productdetail.asp?Model=E3C246D4I-2T,
an AST2500-equipped Xeon E-2100 mini-ITX board), and we're now looking to
get some feedback from the community and hopefully start the process of
getting it merged upstream.
It's currently sitting in a Github branch, since at the moment I'm
mostly seeking fairly broad guidance on what sorts of things will need
to happen to get it into a mergeable form (not sure if gerrit is the
right tool for that). Anyone who'd like to can take a look at:
https://github.com/zevweiss/openbmc/tree/e3c246d4i
Known issues, in approximate descending order of significance (by my
estimate):
- The kernel situation is...kind of messy. This mostly stems from
having a dependency on the PECI code, which is why the branch is
based on the 2.9.0 tag (the last one before the PECI patchset was
dropped from the openbmc kernel) instead of something more recent.
We've got some additional PECI patches on top of that to work around
some limitations in the PECI support on the particular CPU we're
working with, some of which is fairly gross hardcoding for that
specific SKU. There are also a few non-PECI-related kernel patches
that I've posted upstream but not yet incorporated the resulting
review feedback on.
- There are various hacks to get things like x86-power-control and
host-error-monitor compiling and working reasonably, and I'm sort of
skeptical that the approach I took there would be considered
desirable.
- The directory naming is currently meta-equinix/meta-e3c246d4i;
starting out I was a bit unsure whether to name based on the
maintainer/developer of the layer or the maker of the hardware and we
ended up with the former, though I see Ed's in-progress x570d4u port
has things under meta-asrock, so perhaps that should be renamed?
- There are some relatively generic patches for dbus-sensors and
entity-manager that should probably be merged independently.
If anyone can give it a look and let me know what the best way forward
would be for this we'd certainly appreciate it.
Thanks,
Zev Weiss
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: RFC: ASRockRack E3C246D4I port
2021-03-12 2:45 RFC: ASRockRack E3C246D4I port Zev Weiss
@ 2021-03-24 1:50 ` Joel Stanley
2021-03-24 22:15 ` Zev Weiss
0 siblings, 1 reply; 4+ messages in thread
From: Joel Stanley @ 2021-03-24 1:50 UTC (permalink / raw)
To: Zev Weiss; +Cc: openbmc
On Tue, 23 Mar 2021 at 22:25, Zev Weiss <zweiss@equinix.com> wrote:
>
> Hello all,
>
> We (Equinix Metal, formerly known as Packet) having been working on a
> port of OpenBMC to the ASRockRack E3C246D4I board
> (https://www.asrockrack.com/general/productdetail.asp?Model=E3C246D4I-2T,
> an AST2500-equipped Xeon E-2100 mini-ITX board), and we're now looking to
> get some feedback from the community and hopefully start the process of
> getting it merged upstream.
Nice!
>
> It's currently sitting in a Github branch, since at the moment I'm
> mostly seeking fairly broad guidance on what sorts of things will need
> to happen to get it into a mergeable form (not sure if gerrit is the
> right tool for that). Anyone who'd like to can take a look at:
> https://github.com/zevweiss/openbmc/tree/e3c246d4i
>
> Known issues, in approximate descending order of significance (by my
> estimate):
>
> - The kernel situation is...kind of messy. This mostly stems from
> having a dependency on the PECI code, which is why the branch is
> based on the 2.9.0 tag (the last one before the PECI patchset was
> dropped from the openbmc kernel) instead of something more recent.
> We've got some additional PECI patches on top of that to work around
> some limitations in the PECI support on the particular CPU we're
> working with, some of which is fairly gross hardcoding for that
> specific SKU. There are also a few non-PECI-related kernel patches
> that I've posted upstream but not yet incorporated the resulting
> review feedback on.
As you point out, it would be great if someone could take on the PECI
maintainership and get the code merged to mainline.
Out of interest, can you still boot the host without PECI support?
I recommend submitting your device tree now, and getting some level of
support in the openbmc tree. From there you can work to get the other
fixes into the tree.
Cheers,
Joel
>
> - There are various hacks to get things like x86-power-control and
> host-error-monitor compiling and working reasonably, and I'm sort of
> skeptical that the approach I took there would be considered
> desirable.
>
> - The directory naming is currently meta-equinix/meta-e3c246d4i;
> starting out I was a bit unsure whether to name based on the
> maintainer/developer of the layer or the maker of the hardware and we
> ended up with the former, though I see Ed's in-progress x570d4u port
> has things under meta-asrock, so perhaps that should be renamed?
>
> - There are some relatively generic patches for dbus-sensors and
> entity-manager that should probably be merged independently.
>
>
> If anyone can give it a look and let me know what the best way forward
> would be for this we'd certainly appreciate it.
>
>
> Thanks,
> Zev Weiss
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: RFC: ASRockRack E3C246D4I port
2021-03-24 1:50 ` Joel Stanley
@ 2021-03-24 22:15 ` Zev Weiss
0 siblings, 0 replies; 4+ messages in thread
From: Zev Weiss @ 2021-03-24 22:15 UTC (permalink / raw)
To: Joel Stanley; +Cc: openbmc
On Tue, Mar 23, 2021 at 08:50:07PM CDT, Joel Stanley wrote:
>On Tue, 23 Mar 2021 at 22:25, Zev Weiss <zweiss@equinix.com> wrote:
>>
>> Hello all,
>>
>> We (Equinix Metal, formerly known as Packet) having been working on a
>> port of OpenBMC to the ASRockRack E3C246D4I board
>> (https://www.asrockrack.com/general/productdetail.asp?Model=E3C246D4I-2T,
>> an AST2500-equipped Xeon E-2100 mini-ITX board), and we're now looking to
>> get some feedback from the community and hopefully start the process of
>> getting it merged upstream.
>
>Nice!
>
>>
>> It's currently sitting in a Github branch, since at the moment I'm
>> mostly seeking fairly broad guidance on what sorts of things will need
>> to happen to get it into a mergeable form (not sure if gerrit is the
>> right tool for that). Anyone who'd like to can take a look at:
>> https://github.com/zevweiss/openbmc/tree/e3c246d4i
>>
>> Known issues, in approximate descending order of significance (by my
>> estimate):
>>
>> - The kernel situation is...kind of messy. This mostly stems from
>> having a dependency on the PECI code, which is why the branch is
>> based on the 2.9.0 tag (the last one before the PECI patchset was
>> dropped from the openbmc kernel) instead of something more recent.
>> We've got some additional PECI patches on top of that to work around
>> some limitations in the PECI support on the particular CPU we're
>> working with, some of which is fairly gross hardcoding for that
>> specific SKU. There are also a few non-PECI-related kernel patches
>> that I've posted upstream but not yet incorporated the resulting
>> review feedback on.
>
>As you point out, it would be great if someone could take on the PECI
>maintainership and get the code merged to mainline.
>
>Out of interest, can you still boot the host without PECI support?
>
Yes, we're using PECI for monitoring CPU thermals but that's about it;
we can boot the host without it.
>I recommend submitting your device tree now, and getting some level of
>support in the openbmc tree. From there you can work to get the other
>fixes into the tree.
>
Alright, I'll start out with getting some baseline bits and pieces in
place and see about the trickier parts when I get to them.
Thanks,
Zev
^ permalink raw reply [flat|nested] 4+ messages in thread
* RFC: ASRockRack E3C246D4I port
@ 2021-03-17 22:46 Zev Weiss
0 siblings, 0 replies; 4+ messages in thread
From: Zev Weiss @ 2021-03-17 22:46 UTC (permalink / raw)
To: openbmc
[Re-sending, apologies if this ends up duplicated.]
Hello all,
We (Equinix Metal, formerly known as Packet) having been working on a
port of OpenBMC to the ASRockRack E3C246D4I board
(https://www.asrockrack.com/general/productdetail.asp?Model=E3C246D4I-2T,
an AST2500-equipped Xeon E-2100 mini-ITX board), and we're now looking to
get some feedback from the community and hopefully start the process of
getting it merged upstream.
It's currently sitting in a Github branch, since at the moment I'm
mostly seeking fairly broad guidance on what sorts of things will need
to happen to get it into a mergeable form (not sure if gerrit is the
right tool for that). Anyone who'd like to can take a look at:
https://github.com/zevweiss/openbmc/tree/e3c246d4i
Known issues, in approximate descending order of significance (by my
estimate):
- The kernel situation is...kind of messy. This mostly stems from
having a dependency on the PECI code, which is why the branch is
based on the 2.9.0 tag (the last one before the PECI patchset was
dropped from the openbmc kernel) instead of something more recent.
We've got some additional PECI patches on top of that to work around
some limitations in the PECI support on the particular CPU we're
working with, some of which is fairly gross hardcoding for that
specific SKU. There are also a few non-PECI-related kernel patches
that I've posted upstream but not yet incorporated the resulting
review feedback on.
- There are various hacks to get things like x86-power-control and
host-error-monitor compiling and working reasonably, and I'm sort of
skeptical that the approach I took there would be considered
desirable.
- The directory naming is currently meta-equinix/meta-e3c246d4i;
starting out I was a bit unsure whether to name based on the
maintainer/developer of the layer or the maker of the hardware and we
ended up with the former, though I see Ed's in-progress x570d4u port
has things under meta-asrock, so perhaps that should be renamed?
- There are some relatively generic patches for dbus-sensors and
entity-manager that should probably be merged independently.
If anyone can give it a look and let me know what the best way forward
would be for this we'd certainly appreciate it.
Thanks,
Zev Weiss
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-03-24 22:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-12 2:45 RFC: ASRockRack E3C246D4I port Zev Weiss
2021-03-24 1:50 ` Joel Stanley
2021-03-24 22:15 ` Zev Weiss
2021-03-17 22:46 Zev Weiss
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).