Attention is currently required from: Hoernchen.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/32944 )
Change subject: ms: fix startup & shutdown of blade
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://gerrit.osmocom.org/c/osmo-trx/+/32944/comment/103f0f7f_e0bc227e
PS1, Line 11: This is now fixed.
can you perhaps explain the actual fix?
File Transceiver52M/ms/ms.h:
https://gerrit.osmocom.org/c/osmo-trx/+/32944/comment/deb8df3c_0bf41a66
PS1, Line 334: if (pthread_setaffinity_np(h, sizeof(cpuset), &cpuset) < 0) {
this looks unrelated?
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/32944
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I1cd8790191790f4861a70bc55c8f4c9993fa10c8
Gerrit-Change-Number: 32944
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 05 Jul 2023 14:31:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Hoernchen.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/32943 )
Change subject: ms: prettify scheduling + add odroid
......................................................................
Patch Set 1:
(3 comments)
Patchset:
PS1:
Why not simply provide specific config files for specific platforms and keep the code generic?
File Transceiver52M/ms/ms.h:
https://gerrit.osmocom.org/c/osmo-trx/+/32943/comment/b4ed64ec_d0d593d9
PS1, Line 205: enum target { ODROID = 0, PI4 };
Do we really only want to be able to run this in ODROID or PI4? I don't see the point in having this kind of platform hardcoded here.
https://gerrit.osmocom.org/c/osmo-trx/+/32943/comment/6922eb21_a747f2b6
PS1, Line 291: hw_target(hw_cpus > 4 ? sched_params::target::ODROID : sched_params::target::PI4)
so any platform with more than 4 cpus is considered "ODROID"? why not simply "GENERIC" or "GENERAL" then?
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/32943
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Icaf42fd5f76dc2d53dc526558aa8ceaa9c190f7b
Gerrit-Change-Number: 32943
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 05 Jul 2023 14:25:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Hoernchen.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/32942 )
Change subject: ms: fix the gain init for blade
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Can you explain where thoes 30 and 0 dB figures come from?
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/32942
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ic5a25d6f5606fba599b8144fbec7285047dca3c9
Gerrit-Change-Number: 32942
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 05 Jul 2023 14:17:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment