Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

RE: CVS commit: src/share/man/man9

On Mon, 25 Jan 2010, Paul Goyette wrote:

Module Name:    src
Committed By:   jruoho
Date:           Mon Jan 25 16:16:34 UTC 2010

Modified Files:
        src/share/man/man9: Makefile
Added Files:
        src/share/man/man9: sysmon_taskq.9

Log Message:
Add a simple manual page for the simple sysmon task queue.

ok wiz@

This routine is really targetted specifically for use by the sysmon_envsys(8) facility. This man page seems to imply that it's available for general-purpose use.

If we're going to treat it as a general-purpose routine, we should rename and move the files (kern/kern_taskq.[ch] maybe?). Otherwise, I'd prefer to make this man page more specific to sysmon, and perhaps
add an example of where it is currently used.

Also, there is some semantics in the current implementation where all the tasks in the queue are run before checking the condvar; this might not necessarily be appropriate for a general-purpose taskq.

         * Run through all the tasks before we check for the exit
         * condition; it's probably more important to actually run
         * all the tasks before we exit.

|   Paul Goyette   | PGP DSS Key fingerprint: |  E-mail addresses:      |
| Customer Service | FA29 0E3B 35AF E8AE 6651 |  paul at   |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at |
| Kernel Developer |                          | pgoyette at  |

Home | Main Index | Thread Index | Old Index