`new` or `init` function should probably be marked as `unsafe`
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- rust
- Ambito
- operating-systems
Direzione di ricerca
Iniziare individuando le implementazioni di Mouse::new e Mouse::init e il codice che accede alle porte 0x60 e 0x64. Stabilire se l’inizializzazione debba diventare unsafe o un’operazione autonoma, quindi aggiornare l’API e la relativa documentazione sulla sicurezza; il lavoro è completato quando il design scelto impedisce l’accesso concorrente non documentato alle porte, preservando Mouse come parser ove applicabile.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Currently, there is no unsafe stopping someone from doing something like this:
let mut m1 = Mouse::new();
let mut m2 = Mouse::new();
// At the same time
m1.init();
m2.init();
Which would mess things up if the same ports are being accessed simultaneously.
I would suggest: making the init function unsafe:
/// # Safety
/// Reads and writes to ports 0x60 and 0x64. You must ensure that these ports are not being accessed concurrently by other code.
And another idea: What if the init function (and the other functions such as was a standalone function instead of being a method of Mouse? This way, we can leave Mouse as a pure, safe parser for incoming mouse data while making it more obvious that the init function modifies global state.
- Lingua principale
- Rust
- Stelle
- 14
- Fork
- 9
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di rust-osdev/ps2-mouse
-
Don't use callback functionsAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
rust-osdev/ps2-mouse#4 · 1 reazione ·
Tutte le issue di rust-osdev/ps2-mouse
Issue simili
-
area:cli bug good first issue priority:high
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
rtk-ai/rtk#4249 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
ActivityWatch/activitywatch#1463 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno