Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge multiprocess into dev #160

Merged
merged 30 commits into from
Jan 15, 2025
Merged

Merge multiprocess into dev #160

merged 30 commits into from
Jan 15, 2025

Conversation

ken4647
Copy link
Contributor

@ken4647 ken4647 commented Dec 25, 2024

  1. support Multiprocess for aarch64 in EL1 support Multiprocess for aarch64 in EL1 #142
  2. add unixsocket without real inode add unixsocket without real inode #145
  3. add poll state POLLHUP add poll stete POLLHUP #159
  4. support loopback Multiprocess loopback #158

caodg and others added 28 commits March 29, 2024 06:08
support Multiprocess for aarch64 in EL1
fix bugs for multiprocess in aarch64.
fix bug for unexpected pagefault when nested fork.
@ken4647 ken4647 marked this pull request as draft December 25, 2024 17:36
@ken4647 ken4647 requested a review from lhw2002426 January 9, 2025 05:50
@ken4647 ken4647 marked this pull request as ready for review January 13, 2025 08:28
#[cfg(not(feature = "fs"))]
unsafe {
dst.write_bytes(0, size);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there misuse of tab and block?

Copy link
Contributor Author

@ken4647 ken4647 Jan 14, 2025

Choose a reason for hiding this comment

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

these tab and blocks is generated by cmd make fmt. I think it is OK?

removing_vaddr.push(vaddr);
off_pool.push(off);
Copy link
Contributor

Choose a reason for hiding this comment

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

Why del off_pool in release_pages_mapped

Copy link
Contributor Author

@ken4647 ken4647 Jan 15, 2025

Choose a reason for hiding this comment

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

it may be removed by mistask, and it should not be removed there. I have fix it in latest commit.

@@ -112,7 +116,9 @@ impl Pipe {
}

pub fn write_end_close(&self) -> bool {
Arc::strong_count(&self.buffer) == 1
let write_end_count = Arc::weak_count(&self.buffer);
// error!("Pipe::write_end_close <= buffer: {:#?} {:#?}", write_end_count, Arc::as_ptr(&self.buffer));
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we del this comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, I will remove it soon

@@ -8,8 +8,9 @@
*/

const fn align_up(val: usize) -> usize {
const PAGE_SIZE: usize = 0x1000;
(val + PAGE_SIZE - 1) & !(PAGE_SIZE - 1)
// should be the same as align_size in percpu.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this align_size fixed? use config may be beffter?

Copy link
Contributor Author

@ken4647 ken4647 Jan 14, 2025

Choose a reason for hiding this comment

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

it should be same with the size in linker.ld.s. it is not easy to add into ruxconfig, as it is at the level of crate, while ruxconfig is at the level of module, which is higher than level crate.
image

@lhw2002426
Copy link
Contributor

  1. support Multiprocess for aarch64 in EL1 support Multiprocess for aarch64 in EL1 #142
  2. add unixsocket without real inode add unixsocket without real inode #145
  3. add poll state POLLHUP add poll stete POLLHUP #159

there are anthoer change is add loopback device #158

@ken4647 ken4647 force-pushed the dev branch 5 times, most recently from 6b4a4cc to c75a5d9 Compare January 14, 2025 18:12
@lhw2002426 lhw2002426 merged commit 06589bf into syswonder:dev Jan 15, 2025
10 checks passed
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.

5 participants