Skip to content

[Docs] Update IPC, shared memory, directory renaming in features.md#2130

Merged
kailun-qin merged 1 commit into
masterfrom
kailun-qin/update-features-doc
May 21, 2025
Merged

[Docs] Update IPC, shared memory, directory renaming in features.md#2130
kailun-qin merged 1 commit into
masterfrom
kailun-qin/update-features-doc

Conversation

@kailun-qin

@kailun-qin kailun-qin commented May 9, 2025

Copy link
Copy Markdown
Contributor

Description of the changes

Fixes #2125, #2128 and #2110.

How to test this PR?

CI.


This change is Reviewable

@mkow mkow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion


-- commits line 2 at r1:

Suggestion:

[Docs] Update IPC, shared memory and directory renaming in features.md

Signed-off-by: Kailun Qin <kailun.qin@intel.com>
@kailun-qin kailun-qin force-pushed the kailun-qin/update-features-doc branch from e9280f7 to 1f539cc Compare May 20, 2025 14:53

@kailun-qin kailun-qin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @mkow)


-- commits line 2 at r1:
Done.

@mkow mkow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@kailun-qin

Copy link
Copy Markdown
Contributor Author

Jenkins, retest Jenkins-SGX-22.04-Sanitizers please (TCS limit exceeded in rlimit_nofile_4k, happens from time to time, unrelated to the PR)

@kailun-qin

kailun-qin commented May 21, 2025

Copy link
Copy Markdown
Contributor Author

Jenkins, retest Jenkins-Direct-24.04 please (clearly unrelated to the PR)

Note that this failed pipeline showed the same (random) failure patterns (see https://localhost:8080/job/graphene-direct-24.04/243/console and the snippet below) as those observed in #1795, i.e., some issues already hidden in our current master branch. cc @mkow

23:24:43  ----------------------------- Captured stderr call -----------------------------
23:24:43  [0.483] [P1:libos] error: Internal memory fault at 0x4826b26c8990 (0x7fffffe306a3, VMID = 1, TID = 0)
23:24:43  - generated xml file: /home/jenkins/workspace/graphene-direct-24.04/libos/test/regression/libos-regression.xml -
23:24:43  =========================== short test summary info ============================
23:24:43  FAILED test_libos.py::TC_01_Bootstrap::test_201_exec_same - subprocess.CalledProcessError: Command '['/home/jenkins/workspace/graphene-direct-24.04/usr/lib/x86_64-linux-gnu/gramine/direct/loader', '/home/jenkins/workspace/graphene-direct-24.04/usr/lib/x86_64-linux-gnu/gramine/direct/libpal.so', 'init', 'exec_same', 'arg_#0', 'arg_#1', 'arg_#2', 'arg_#3', 'arg_#4', 'arg_#5', 'arg_#6', 'arg_#7', 'arg_#8', 'arg_#9', 'arg_#10', 'arg_#11', 'arg_#12', 'arg_#13', 'arg_#14', 'arg_#15', 'arg_#16', 'arg_#17', 'arg_#18', 'arg_#19', 'arg_#20', 'arg_#21', 'arg_#22', 'arg_#23', 'arg_#24', 'arg_#25', 'arg_#26', 'arg_#27', 'arg_#28', 'arg_#29', 'arg_#30', 'arg_#31', 'arg_#32', 'arg_#33', 'arg_#34', 'arg_#35', 'arg_#36', 'arg_#37', 'arg_#38', 'arg_#39', 'arg_#40', 'arg_#41', 'arg_#42', 'arg_#43', 'arg_#44', 'arg_#45', 'arg_#46', 'arg_#47', 'arg_#48', 'arg_#49']' returned non-zero exit status 1.

@kailun-qin kailun-qin merged commit 1f539cc into master May 21, 2025
27 checks passed
@kailun-qin kailun-qin deleted the kailun-qin/update-features-doc branch May 21, 2025 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants