commaai / commaai/msgq

Remove fixed max reader number

Open
#702 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
255
Forks
294
Avg merge
27m
Merged PRs (30d)
7

Description

This is a constant footgun for fork maintainers, so we can assume it's even more so for people trying to casually hack on openpliot or msgq. https://github.com/commaai/msgq/pull/681

An immediate idea is to split the readers out into a separate file on `/dev/shm/`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading linked pull request #681 and inspect the reader bookkeeping and shared-memory layout in the msgq repository. Evaluate the proposed separate reader file under /dev/shm/, then confirm that reader counts are no longer constrained by a fixed maximum and that existing pub/sub behavior remains intact.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
distributed-systems
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.