Bug 61195

Comment 1 Alex Dupre 2004-01-11 11:08:46 UTC

Responsible Changed
From-To: freebsd-doc->ale

I'll take this.

Comment 2 Simon L. B. Nielsen 2004-01-11 14:39:24 UTC

On 2004.01.11 14:36:52 +0800, Xin LI wrote:

> Index: NOTES
> ===================================================================
> RCS file: /home/mirror/freebsd/ncvs/src/sys/i386/conf/NOTES,v
> retrieving revision 1.1112
> diff -u -r1.1112 NOTES
> --- NOTES 3 Jan 2004 18:43:37 -0000 1.1112
> +++ NOTES 11 Jan 2004 06:25:33 -0000
> @@ -231,8 +231,9 @@
>  # the CPU fraction reserved to userland with the sysctl variable
>  # kern.polling.user_frac (default 50, range 0..100).
>  #
> -# Only the "dc" "fxp" and "sis" devices support this mode of operation at
> -# the time of this writing.
> +# Only the dc(4), em(4), fwe(4), fxp(4), nge(4), re(4), rl(4) and
> +# sis(4) devices support this mode of operation at the time of this
> +# writing.


Perhaps notes should just refer to the polling manual page for the
device list.  Then we don't have to maintain two lists, but that's up to
ale (who took the PR).  Just my 0.02euro cent :).

-- 
Simon L. Nielsen
FreeBSD Documentation Team

----- Original Message ----- 
From: "Simon L. Nielsen" <simon@FreeBSD.org>
To: "Xin LI" <delphij@frontfree.net>
Cc: <FreeBSD-gnats-submit@FreeBSD.org>
Sent: Sunday, January 11, 2004 10:39 PM
Subject: Re: docs/61195: [PATCH] Document more polling(4) supported devices

> Perhaps notes should just refer to the polling manual page for the
> device list.  Then we don't have to maintain two lists, but that's up to
> ale (who took the PR).  Just my 0.02euro cent :).

I agree. What about this patch? ;-)

Index: NOTES
===================================================================
RCS file: /home/mirror/freebsd/ncvs/src/sys/i386/conf/NOTES,v
retrieving revision 1.1112
diff -u -r1.1112 NOTES
--- NOTES 3 Jan 2004 18:43:37 -0000 1.1112
+++ NOTES 11 Jan 2004 14:44:52 -0000
@@ -231,9 +231,8 @@
 # the CPU fraction reserved to userland with the sysctl variable
 # kern.polling.user_frac (default 50, range 0..100).
 #
-# Only the "dc" "fxp" and "sis" devices support this mode of operation at
-# the time of this writing.
-
+# Not all devices support this mode of operation at the time of this
+# writing. You are encouraged to consult polling(4) for details.
 options DEVICE_POLLING

Comment 4 Alex Dupre 2004-01-11 22:08:59 UTC

State Changed
From-To: open->closed

Patch committed, thanks!