Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

Re: kern/159281: [PATCH] Linux-like /proc/swaps for linprocfs

0 views
Skip to first unread message

Robert Millan

unread,
Jul 31, 2011, 12:56:51 PM7/31/11
to
2011/7/30 Kostik Belousov <kost...@gmail.com>:
> Second, my proposal contains a flaw. Namely, if some swap device was removed
> between calls to swap_info and swap_devname calls, we get mangled list.

Ok, I see that you fixed this by unifying those functions.

> The third problem, which is not fixed, and which I do not want to handle,
> is that the swap devices list can be changed between calls to swap_devname(),
> changing device indexes and thus making the output mangled.

You say you don't want to handle this, but AFAICT from the patch you
already did. Or did I miss something?

> Should the swap device name be separated from 'unknown' word by
> space or by tab ?

Linux' /proc/swaps prints spaces after the first column and tabs for
the rest. I think it's better to do the same just in case, but I
don't think it's relevant either way.

> I updated your patch, hopefully fixing the issues. Do you have comments
> or objections ?

No. Thanks for fixing those problems.

--
Robert Millan
_______________________________________________
freebsd-...@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-emulation
To unsubscribe, send any mail to "freebsd-emulat...@freebsd.org"

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-...@muc.de

0 new messages