Differential D1105

Implement ppoll() system call.
ClosedPublic
Actions

Authored by dchagin on Nov 4 2014, 4:14 PM.

Tags

None

Referenced Files

	F131933688: D1105.id3028.diff
	Sun, Oct 12, 7:55 AM

	Unknown Object (File)
	Sun, Oct 5, 9:36 AM

	Unknown Object (File)
	Fri, Oct 3, 3:20 PM

	Unknown Object (File)
	Thu, Sep 25, 10:42 PM

	Unknown Object (File)
	Tue, Sep 23, 10:43 AM

	Unknown Object (File)
	Sun, Sep 21, 7:41 AM

	Unknown Object (File)
	Mon, Sep 15, 7:42 AM

	Unknown Object (File)
	Mon, Sep 15, 7:42 AM

Subscribers

Details

Reviewers

Group Reviewers

Linux Emulation

Commits

rS283451: Implement ppoll() system call.

Summary

(need more test, ltp passed)

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dchagin updated this revision to Diff 2267.Nov 4 2014, 4:14 PM

dchagin retitled this revision from to Implement ppoll() system call..

dchagin updated this object.

dchagin added a reviewer: Linux Emulation.Nov 4 2014, 5:44 PM

trasz added a subscriber: trasz.Dec 8 2014, 11:09 AM

trasz added inline comments.

sys/amd64/linux/syscalls.master
458 ↗	(On Diff #2267)	Misplaced space, should be "struct pollfd *fds".
sys/compat/linux/linux_misc.c
2189 ↗	(On Diff #2267)	if (args->sset != NULL) {
2197 ↗	(On Diff #2267)	Why call SIGDELSET? A comment would be nice.
2201 ↗	(On Diff #2267)	if (args->tsp != NULL) {

Fixed, thanks! btw, also for all of Linuxulator's sigprocmask syscall family should be remove SIGKILL|SIGSTOP from new signal mask.
Will be fixed soon.

whoops, kern_sigprocmask() already call SIG_CANTMASK for new mask, so remove my SIGDELSET.

trasz added inline comments.Mar 21 2015, 5:18 PM

sys/compat/linux/linux_misc.c
2224 ↗	(On Diff #3028)	What is this for? The sys_poll() implementation doesn't seem to do anything like this.

dchagin added inline comments.Mar 24 2015, 6:24 PM

sys/compat/linux/linux_misc.c
2224 ↗	(On Diff #3028)	The Linux ppoll() system call modifies its timeout argument.

trasz accepted this revision.Mar 25 2015, 11:21 AM

trasz added a reviewer: trasz.

trasz added inline comments.

sys/compat/linux/linux_misc.c
2224 ↗	(On Diff #3028)	A comment explaining this would be nice. Still, looks ok.

This revision is now accepted and ready to land.Mar 25 2015, 11:21 AM

Closed by commit rS283451: Implement ppoll() system call. (authored by dchagin). · Explain WhyMay 24 2015, 4:59 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

amd64/

linux/

1 line

syscalls.master

3 lines

linux32/

linux32_dummy.c

1 line

syscalls.master

3 lines

compat/

linux/

53 lines

i386/

linux/

1 line

syscalls.master

3 lines

Diff 5655

head/sys/amd64/linux/linux_dummy.c

Loading...

head/sys/amd64/linux/syscalls.master

Loading...

head/sys/amd64/linux32/linux32_dummy.c

Loading...

head/sys/amd64/linux32/syscalls.master

Loading...

head/sys/compat/linux/linux_misc.c

Loading...

head/sys/i386/linux/linux_dummy.c

Loading...

head/sys/i386/linux/syscalls.master

Loading...