arch/ceva: cannot be selected, has no board, and tests the removed CONFIG_DISABLE_SIGNALS

Đang mở
  • Có gì không ổn?
#21,075 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ó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
25/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
c
Lĩnh vực
embedded-iot

Hướng nghiên cứu

Start by checking arch/Kconfig, arch/ceva/Kconfig, and the listed signal guards in arch/ceva; compare them with other architectures’ CONFIG_ENABLE_ALL_SIGNALS guards. The issue offers two possible resolutions: make the port selectable and buildable by adding a board, or remove it. First determine whether the port is still in use; done means the selected path is resolved, including the stale signal configuration references if the port remains.

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

Mô tả

Description

arch/ceva (added in ee916bdb91c, 2022-01) cannot be built in the tree:

  • arch/Kconfig has no ARCH_CEVA option and does not source arch/ceva/Kconfig (which starts with if ARCH_CEVA). So no configuration can select the architecture.
  • No board under boards/ uses it, and no CI test list builds it.

The code is also out of date. 13 lines in 9 files test CONFIG_DISABLE_SIGNALS, which NuttX removed in 7.30 ("Signals can no longer be disabled"):

  • arch/ceva/include/xc5/irq.h:131, arch/ceva/include/xm6/irq.h:134
  • arch/ceva/src/common/ceva_schedulesigaction.c:38, :149
  • arch/ceva/src/common/ceva_sigdeliver.c:37, :98
  • arch/ceva/src/common/ceva_signal_dispatch.c:33
  • arch/ceva/src/common/ceva_svcall.c:281, :322
  • arch/ceva/src/common/ceva_userspace.c:217
  • arch/ceva/src/xc5/xc5_signal_handler.S:32, arch/ceva/src/xm6/xm6_signal_handler.S:32

The other architectures guard the same code with CONFIG_ENABLE_ALL_SIGNALS (0ea686bc5b3, partial signal support). With CONFIG_ENABLE_PARTIAL_SIGNALS or CONFIG_DISABLE_ALL_SIGNALS, ceva_userspace.c:218 and ceva_svcall.c:295 use userspace_s.signal_handler, which include/nuttx/userspace.h then does not declare.

Nobody can build or test a change to these files in the tree. Two ways forward:

  1. If the port is still in use: add ARCH_CEVA to arch/Kconfig and a board, then change the signal guards to CONFIG_ENABLE_ALL_SIGNALS.
  2. If not: remove arch/ceva.
Verification
  • I have verified before submitting the report.
Ngôn ngữ chính
C
Star
4.1k
Fork
1.8k
Merge trung bình
1 ngày 5 giờ
Pull request đã merge (30 ngày)
841

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

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 apache/nuttx

Tất cả issue của apache/nuttx

Issue tương tự

Thêm issue về C

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.