Skip to content

Conversation

@KyleKotowick
Copy link

No description provided.

@KyleKotowick KyleKotowick marked this pull request as ready for review March 13, 2025 15:21
Copy link
Member

@robin-nitrokey robin-nitrokey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally, I would prefer to make Dispatch generic over the message size (maybe with a type alias using the old default of 7609 bytes) so that this crate does not need to be aware of the message size required by specific algorithms. As we have already made the App trait generic over the message size and extracted it into a separate crate, only the runner, trussed-usbip and usbd-ctaphid would be affected by this change. So while it would add some complexity, the scope would be very limited. @sosthene-nitrokey What do you think?

@sosthene-nitrokey
Copy link
Contributor

I was thinking of the same thing. I like the idea of keeping an alias to avoid having the value show up in so many places.

@KyleKotowick
Copy link
Author

@robin-nitrokey @sosthene-nitrokey I'm all for making it generic and removing the need for this change, but this sounds like a structural change in trussed that would be beyond the scope of just PQC implementation. Is that something you'll be making a PR for?

@robin-nitrokey
Copy link
Member

Yeah, we should take that in a separate PR. It should not be a big change.

robin-nitrokey added a commit that referenced this pull request Mar 21, 2025
The fixed buffer size can be problematic for some use cases, see:
- #15

To avoid adding support for different buffer sizes to the crate, we can
just make the dispatch implementation generic over the buffer size.  For
simple cases, a type alias is provided using the old buffer size.
robin-nitrokey added a commit that referenced this pull request Mar 21, 2025
The fixed buffer size can be problematic for some use cases, see:
- #15

To avoid adding support for different buffer sizes to the crate, we can
just make the dispatch implementation generic over the buffer size.  For
simple cases, a type alias is provided using the old buffer size.
@robin-nitrokey
Copy link
Member

Replaced by: #16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants