Re: [問題] 一個讀取 uart 的 process

作者: gn00618777 (非常念舊)   2021-06-06 18:15:15
※ 引述《gn00618777 (非常念舊)》之銘言:
: 版友好
: 我是在 Linux 寫 C。之前寫了一個 process ,是 1個 byte 1 個 byte 讀取的
: blocking 程式,每讀1個byte就檢查是否是 header,但被說寫得不好。
: 於是乎再寫了一個 non-blocking的,主要是先蒐集我想要的長度後,再檢查是否是
: header。
: protocol header 如下:
: byte0 sig1 (0xab)
: byte1 sig2 (0xba)
: byte2 id
: byte3 packet length
: byte4 seq num
: 概念上,我會讀取 uart fd,讀到的 rcv_len 加總起來,若有我達到的長度
: 就去檢查這5個byte是不是有 header。 只要看到 0xab 0xba 我就認定是一個
: header 的起始。也有可能這2個byte出現在這5個byte的任何地方,甚至 byte4
: 會是 0xab,此時我們就有可能需要再讀取1個byte來做判定是否有header。當我發
: 現一個header時,我就會從此header到結尾整個位移到packet的起始。並回傳 left
: 值,此left值代表意思是說,我還需要讀取幾個byte來做判定。 我感覺我這寫法滿
: 囉嗦的,WAIT_HEADER status這樣寫,WAIT_PAYLOAD status也會這樣寫,後面勢必
: 一大坨,但又想不出啥更好的方法,所以想來求助一下版友看有沒有更好的寫法?
: 這是我的範例 code: https://reurl.cc/bzrez3
: 懇請建議,謝謝。
寫code駑鈍..
我參考了版友建議,改了寫法。
我用了類似生產者消費者方式。當 select uart 有資料時,讀取 sizeof(rbuf) = 256
的長度。根據uart處於何種狀態(MSG_WAIT_ID1, MSG_WAIT_ID2....),來決定抓取的數量
完整header共5byte: ID1, ID2 , OP, SEQ, LEN
假設 select 到了 5 個 byte了,我就去一個byte一個byte去檢查,如果這五個byte是
xx,xx,ID1,ID2,OP ,跑了我的 find_header 後就會求出目前 state 狀態為
MSG_WAIT_SEQ,根據此狀態發現,我還需要再抓取2byte,才足夠去判定他是不是一個
完整的 5 個 byte header。
先姑且不論效率和coding style ,我想求問的是以下:
rcv = FRONT_UART_read(fd, rbuf + idx, sizeof(rbuf));
有沒有好的改寫方式? 我怕的是此行,如果我 idx 停留在 255,當 uart device
搞我送出 256 byte資料,我不就超出記憶體範圍了?
還有請問我 99~106 有沒有邏輯上的錯誤呢? 萬分感謝..
https://reurl.cc/NrZ4We
作者: Lipraxde (Lipraxde)   2021-06-06 21:54:00
恩...它第三個參數是放 buffer size 吧?
作者: gn00618777 (非常念舊)   2021-06-06 23:53:00
你的意思是說,我不該寫死,應該動態調整讀取長度?
作者: Lipraxde (Lipraxde)   2021-06-07 00:18:00
動態調整嗎...我覺得應該講成給的 buffer size 有多少,就該填多少

Links booklink

Contact Us: admin [ a t ] ucptt.com