Attention is currently required from: fixeria, jolly, laforge.
pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/libosmocore/+/42638?usp=email )
Change subject: osmo_io: iofd_txqueue_dequeue(): fix double next->io_len decrement
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
Patchset:
PS2:
Patch LGTM, but I feat the test will trigger eventually non-deterministic results based on read/write internal buffers in the OS.
ACK to making a patch release.
BTW, it would make sense to move the code block "
Fill up empty buffers in dequeued msghdr with buffers from the next msghdr" into a helper function, and actually optimize it to move multiple msgs from the same next msghdr in a tight loop instead of 1 by 1.
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/42638?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia96e25d11d8d683fa9dd0c56bdac257992edd782
Gerrit-Change-Number: 42638
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Apr 2026 08:07:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Attention is currently required from: lynxis lazus.
Hoernchen has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42619?usp=email )
Change subject: octsim: usb_descriptor: reduce IRQ endpoint max packet size to 4 bytes
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> err, nevermind, it's 16 for hs.
...but wait a second, we're a fs device, but this is the hs number? that would allocate a bit more than needed...
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42619?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I615ff16d9bc0e8d39156015e41b4abe4502aaca1
Gerrit-Change-Number: 42619
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Fri, 17 Apr 2026 15:53:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Hoernchen <ewild(a)sysmocom.de>
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>
Attention is currently required from: lynxis lazus.
Hoernchen has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42619?usp=email )
Change subject: octsim: usb_descriptor: reduce IRQ endpoint max packet size to 4 bytes
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> If you're worried about scheduling set bInterval to the recommended value by the ccid spec, which is […]
err, nevermind, it's 16 for hs.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42619?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I615ff16d9bc0e8d39156015e41b4abe4502aaca1
Gerrit-Change-Number: 42619
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Fri, 17 Apr 2026 15:48:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Hoernchen <ewild(a)sysmocom.de>
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>
Attention is currently required from: lynxis lazus.
Hoernchen has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42619?usp=email )
Change subject: octsim: usb_descriptor: reduce IRQ endpoint max packet size to 4 bytes
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> As I understand the IRQs on USB, the EP size is relevant for scheduling the bus and the bus will res […]
If you're worried about scheduling set bInterval to the recommended value by the ccid spec, which is 255 - it's sim card presence after all, not something that changes rapidly...
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42619?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I615ff16d9bc0e8d39156015e41b4abe4502aaca1
Gerrit-Change-Number: 42619
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Fri, 17 Apr 2026 15:46:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Hoernchen <ewild(a)sysmocom.de>
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>
Attention is currently required from: laforge.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/libosmocore/+/42638?usp=email )
Change subject: osmo_io: iofd_txqueue_dequeue(): fix double next->io_len decrement
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
> this definitely sounds like a patch release?
Ack, I can tag one when this patch is merged.
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/42638?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia96e25d11d8d683fa9dd0c56bdac257992edd782
Gerrit-Change-Number: 42638
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Fri, 17 Apr 2026 14:57:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>