Attention is currently required from: jolly.
dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43025?usp=email )
Change subject: V1.2: Update from "EnableUsingDD" to "ImmediateEnable" function
......................................................................
Patch Set 1: -Code-Review
(1 comment)
File src/ipa/libipa/proc_prfle_inst.c:
https://gerrit.osmocom.org/c/onomondo-ipa/+/43025/comment/abf43a87_a0403900… :
PS1, Line 45: ipa_es10b_immediate_enable(ctx);
(this refers to WIP: Add refreshFlag, read the review comments there first)
Here you would pass the refreshFlag together with the (aforementioned) req struct. Use the configured refreshFlag from ctx->cfg->refresh_flag.
(please do not greb the refresh_flag from inside ipa_es10b_immediate_enable, I guess the architecture is a bit unclean here. However, since the refreshFlag is a an input parameter of the ES10b function call, it must be passed through a req parameter.)
(It is also a bit vague what exactly grants the immediate profile enable, I guess when the immediateEnableFlag is not set inside the eUICC, immediateEnable is not granted.)
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43025?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: Id750b2f5ca6c08a013aa30f876aa37e6291a30d5
Gerrit-Change-Number: 43025
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Tue, 21 Jul 2026 16:15:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Attention is currently required from: jolly.
dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43026?usp=email )
Change subject: WIP: Add refreshFlag
......................................................................
Patch Set 1:
(5 comments)
Patchset:
PS1:
I thin you somehow confused immediateEnableFlag and refreshFlag. I also thought for a brief moment that the refreshFlag would be something that the eUICC stores, but that is not the case. The refreshFlag is something that is configured in the IoT device. This also makes sense since different modems and devices may behave differently and so some may require a REFRESH and some may not.
I think this is easy to fix, just drop the changes in context.h and add a request struct you can pass as req to ipa_es10b_immediate_enable() and then you use the refresh_flag from the cfg struct you find in ipad.h.
(however, I must confess that I have not understood everything in detail yet, especially when to grant/deny the immediate enable - but I think this is not in scope of this patch anyway.)
Commit Message:
https://gerrit.osmocom.org/c/onomondo-ipa/+/43026/comment/9c997c74_afd6d44d… :
PS1, Line 9: Ich weiss nicht die Bohne, was das hier soll. TBD!!!
With the refreshFlag you can control if the eUICC shall use a proactive REFRESH command to make sure that the modem re-reads all the file contents it may have cached etc.
You can find more about the REFRESH command in ETSI TS 102 223, section 6.4.7
File src/ipa/libipa/context.h:
https://gerrit.osmocom.org/c/onomondo-ipa/+/43026/comment/e7158ede_0581a359… :
PS1, Line 35: bool refresh;
As far as I can see you do not need this flag. The ConfigureImmediateProfileEnablingRequest nor the configureImmediateEnable PSMO can set a refreshFlag directly inside the eUICC.
File src/ipa/libipa/es10b_immediate_enable.c:
https://gerrit.osmocom.org/c/onomondo-ipa/+/43026/comment/ec5c0ce9_fe2646c3… :
PS1, Line 157: enable_prfle_req.req.refreshFlag = ctx->nvstate.iot_euicc_emu.immediate_enable.refresh;
I think this is not entirely correct. You have define a request struct in es10b_immediate_enable.h like:
struct ipa_es10b_immediate_enable_req {
bool refreshFlag
}
I have written a follow up message in Id750b2f5ca6c08a013aa30f876aa37e6291a30d5 at the position where ipa_es10b_immediate_enable() is called.
(in SGP.32 v.1.0.1 this function had no parameters at all, that is why there is no request struct yet)
https://gerrit.osmocom.org/c/onomondo-ipa/+/43026/comment/a738e448_a3286edb… :
PS1, Line 167: IPA_LOGP_ES10X("ImmediateEnable", LINFO,
You could split this of in a separate patch as it is unrelated.
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43026?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: I7d8f9182b675ca0851e00ef8d36201151f30178f
Gerrit-Change-Number: 43026
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: dexter <pmaier(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Tue, 21 Jul 2026 16:15:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: osmith.
daniel has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43084?usp=email )
Change subject: testenv: support Debian Unstable
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43084?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ib8f04ee1e8b33f6e724ea2a74b1a96f26746e878
Gerrit-Change-Number: 43084
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 21 Jul 2026 15:22:04 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: osmith.
daniel has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43085?usp=email )
Change subject: testenv: init: check distro argument early
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43085?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I9d51c2b236f10f4c10e991873df47b4f3297e6e1
Gerrit-Change-Number: 43085
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 21 Jul 2026 15:21:55 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: fixeria, osmith.
daniel has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43083?usp=email )
Change subject: testenv: Dockerfile: rebar3 version via apt-cache
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43083?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ie3c8812c41648dd5ca655114dec00ae7d48eaba8
Gerrit-Change-Number: 43083
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 21 Jul 2026 15:20:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes