-
Notifications
You must be signed in to change notification settings - Fork 22
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
Conversation
ken4647
commented
Dec 25, 2024
•
edited
Loading
edited
- support Multiprocess for aarch64 in EL1 support Multiprocess for aarch64 in EL1 #142
- add unixsocket without real inode add unixsocket without real inode #145
- add poll state POLLHUP add poll stete POLLHUP #159
- support loopback Multiprocess loopback #158
Merge dev into main
Merge dev into main
Merge dev into main
Merge dev into main
support Multiprocess for aarch64 in EL1
fix bugs for multiprocess in aarch64.
Multiprocess loopback
add unixsocket without real inode
fix bug for unexpected pagefault when nested fork.
add unix socket drop
Multiprocess loopback
add poll stete POLLHUP
#[cfg(not(feature = "fs"))] | ||
unsafe { | ||
dst.write_bytes(0, size); | ||
} |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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); |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
api/ruxos_posix_api/src/imp/pipe.rs
Outdated
@@ -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)); |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there are anthoer change is add loopback device #158 |
6b4a4cc
to
c75a5d9
Compare