-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Showing
1 changed file
with
24 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,24 @@ | ||
* Threads in /proc/[pid]/task/[n] | ||
Known bugs to be fixed | ||
---------------------- | ||
|
||
* The non-owned processes sometimes show up with INT_MAX as their owner, | ||
instead of opt_anon_uid. This is likely to be a libps problem. | ||
|
||
Improvements and new features | ||
----------------------------- | ||
|
||
* There is a lot of dynamic memory allocation going on and it comes with a | ||
cost in performance. We could try to limit such allocation, as long as it | ||
keeps the inner interface simple and preserves the read/readdir semantics | ||
(performance is probably not critical for a proc filesystem.) | ||
One way would be to add an (optional) "needed_length" field to | ||
procfs_node_ops, and arrange to pass a sufficent buffer in (*contents, | ||
*contents_len) when get_contents is called. Then the user-provided buffer | ||
might be used directly under some circumstances. | ||
|
||
* Add thread directories as [pid]/task/[n]. This shouldn't be too hard if we | ||
use "process" nodes for threads, and provide an "exists" hook for the "task" | ||
entry itself so that it's disabled in thread nodes. It might prove necessary | ||
to have "optional" libps flags for some content generators, though, since | ||
some of them might be missing for threads. | ||
|