2000-10-26 22:45:56

by Jeremy Fitzhardinge

[permalink] [raw]
Subject: [PATCH] address-space identification for /proc

Hi,

/proc has no way to indicate whether tasks share an address space.
This one-liner patch adds a new ASID: field to /proc/<pid>/status so
there's some way to see address-space sharing between tasks.

While this is hardly a bug-fix, it is a pretty useful thing to know
which is otherwise completely absent.

J


Attachments:
(No filename) (0.00 B)
(No filename) (240.00 B)
Download all attachments

2000-10-26 22:52:57

by Jeremy Fitzhardinge

[permalink] [raw]
Subject: Re: [PATCH] address-space identification for /proc

On Thu, Oct 26, 2000 at 03:45:27PM -0700, I wrote:
> + buffer += sprintf("ASID: %p\n", mm);

Obviously, this should be:

+ buffer += sprintf("ASID:\t%p\n", mm);

for consistency.

J


Attachments:
(No filename) (183.00 B)
(No filename) (240.00 B)
Download all attachments

2000-10-26 23:16:32

by Johannes Erdfelt

[permalink] [raw]
Subject: Re: [PATCH] address-space identification for /proc

On Thu, Oct 26, 2000, Jeremy Fitzhardinge <[email protected]> wrote:
> On Thu, Oct 26, 2000 at 03:45:27PM -0700, I wrote:
> > + buffer += sprintf("ASID: %p\n", mm);
>
> Obviously, this should be:
>
> + buffer += sprintf("ASID:\t%p\n", mm);

and even more obvious:

+ buffer += sprintf(buffer, "ASID:\t%p\n", mm);

Actually putting it into the buffer would be useful as well :)

JE

2000-10-27 01:22:10

by Jeremy Fitzhardinge

[permalink] [raw]
Subject: Re: [PATCH] address-space identification for /proc

On Thu, Oct 26, 2000 at 07:01:26PM -0400, Johannes Erdfelt wrote:
> and even more obvious:
>
> + buffer += sprintf(buffer, "ASID:\t%p\n", mm);
>
> Actually putting it into the buffer would be useful as well :)

That serves me right for hand-editing patches.

J
--
Repeat to self: I am not Linus


Attachments:
(No filename) (298.00 B)
(No filename) (240.00 B)
Download all attachments