Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

msp430: memset and memcpy in core don't write anything, so globals aren't initialized

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#1,101 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
75/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
zig
Lĩnh vực
embedded-iot

Hướng nghiên cứu

Đọc các phần triển khai memset và memcpy trong core/src/cpus/msp430.zig, sau đó tái hiện hành vi khởi động bằng các lệnh trình mô phỏng MSP430 được cung cấp và thiết lập examples/texasinstruments/msp430/src/empty.zig. Sửa các phép so sánh vòng lặp và luồng điều khiển để .bss được xóa và .data được sao chép; xác nhận rằng bản dump bộ nhớ hiển thị 34 12 tại 0x200 và 00 00 tại 0x204.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

msp430

memset and memcpy in core/src/cpus/msp430.zig return without writing anything. The startup code (initialize_system_memories) uses them to clear .bss and copy .data, so on MSP430 global variables start with whatever is in RAM. This affects both boards (msp430x.zig uses the same functions).

Both loops compare a register with itself, because MOV is src, dst:

  • memset: MOV R12, R5 and ADD R5, R14 leave the end in R14, but the loop does CMP R5, R12 (start against start), so it exits right away.
  • memcpy: MOV R5, R12 overwrites the destination with R5, so CMP R5, R12 also exits right away. It also has no JMP memcpy_loop, and SUB R12, R14 changes R14 instead of restoring R12.

Reproduce on 8862a281 with Zig 0.17.0, in the mspdebug simulator (I don't have MSP430 hardware). Replace examples/texasinstruments/msp430/src/empty.zig with:

const std = @import("std");
const microzig = @import("microzig");

pub const std_options = microzig.std_options(.{});

comptime {
    _ = microzig.export_startup();
}

pub const panic = std.debug.no_panic;

var initialized: u16 = 0x1234; // .data
var zeroed: u16 = 0; // .bss

pub fn main() void {
    std.mem.doNotOptimizeAway(&initialized);
    std.mem.doNotOptimizeAway(&zeroed);
}

cd examples/texasinstruments/msp430
zig build -Doptimize=ReleaseSmall
# initialized is at 0x200 and zeroed at 0x204; 0xc02a is microzig_main
mspdebug -q sim "prog zig-out/firmware/empty_MSP430G2553.elf" "fill 0x200 16 0xaa" "reset" "setbreak 0xc02a" "run" "md 0x200 8"

Result:

00200: aa aa aa aa aa aa aa aa

Expected 34 12 at 0x200 and 00 00 at 0x204. With both loops fixed (end in R14, compare the cursor against it, jump back), the same program gives:

00200: 34 12 00 00 00 00 00 00

I can open a PR with the fix if you want.

Ngôn ngữ chính
Zig
Star
2.3k
Fork
198
Merge trung bình
2 ngày 12 giờ
Pull request đã merge (30 ngày)
28

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của ZigEmbeddedGroup/microzig

Tất cả issue của ZigEmbeddedGroup/microzig

Issue tương tự

Thêm issue về Embedded & IoT

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.