ARM (32-bit): Thumb never decoded, cmplog logs wrong operands, `pop {pc}` not seen as return
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- rust
- Lĩnh vực
- embedded-iot, testing-qa
Hướng nghiên cứu
Start in src/arch/arm.rs at the listed decoder, operand simplification, and return-detection locations. Check how instruction mode and CPSR.T are represented, then trace compare operands and LDM handling; the report has not been reproduced under SIMICS. Done means Thumb decoding, compare logging, pop {..., pc} return detection, and the noted U8/U16 and Asr cases behave as described, with coverage and cmplog verified on ARM targets.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Where
src/arch/arm.rsaround lines 150, 1016-1019, 1029-1055, 1090-1107, 1128-1131
Problem
The decoder is always ARM mode (set_thumb_mode never called), so Thumb/Cortex-M code gets almost no edge or cmp coverage. ARM-mode data-processing decodes as [Rd, Rn, Op2]; for CMP/CMN/TST/TEQ tsffs logs operands 0 and 1 (r0, Rn) instead of (Rn, Op2). ARM-mode pop {..., pc} is LDM and not matched by last_was_ret. simplify lacks U8/U16; Asr is a logical shift.
Impact
Little useful coverage/cmplog on ARM targets.
Suggested fix
Set Thumb mode per instruction from CPSR.T; skip operand 0 for compare opcodes in ARM mode; treat LDM with pc in the list as a return.
Status
Reported by static code review at 79972db; not yet re-verified line by line. Found by reading the code; not reproduced under SIMICS.
- Ngôn ngữ chính
- Rust
- Star
- 335
- Fork
- 26
- Merge trung bình
- 1 ngày 50 phút
- Pull request đã merge (30 ngày)
- 12
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Có Dockerfile hoặc tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của intel/tsffs
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
Issue tương tự
-
bug UI
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
wardian-app/Wardian#1746 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
jlcodes99/cockpit-tools#3125 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
bug user-sprint
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Vondor/KassenSturz#967 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area/ci
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[BUG] Incremental sync warns "no stored highest_uid" on every round for folders with no new mailĐang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
rustmailer/bichon#411 ·