Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 8 Aug 2014 22:30:42 +0000 (UTC)
From:      Warren Block <wblock@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r269743 - head/sys/sys
Message-ID:  <53e54f92.239a.3d3730af@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: wblock (doc committer)
Date: Fri Aug  8 22:30:41 2014
New Revision: 269743
URL: http://svnweb.freebsd.org/changeset/base/269743

Log:
  Update the comments in exec.h with help from jilles.
  
  PR:		180970
  Submitted by:	Carlos Jacobo Puga Medina <cjpugmed@gmail.com>
  Reviewed by:	jilles
  MFC after:	1 week

Modified:
  head/sys/sys/exec.h

Modified: head/sys/sys/exec.h
==============================================================================
--- head/sys/sys/exec.h	Fri Aug  8 22:08:51 2014	(r269742)
+++ head/sys/sys/exec.h	Fri Aug  8 22:30:41 2014	(r269743)
@@ -39,12 +39,17 @@
 #define _SYS_EXEC_H_
 
 /*
- * The following structure is found at the top of the user stack of each
- * user process. The ps program uses it to locate argv and environment
- * strings. Programs that wish ps to display other information may modify
- * it; normally ps_argvstr points to the argv vector, and ps_nargvstr
- * is the same as the program's argc. The fields ps_envstr and ps_nenvstr
- * are the equivalent for the environment.
+ * Before ps_args existed, the following structure, found at the top of
+ * the user stack of each user process, was used by ps(1) to locate
+ * environment and argv strings.  Normally ps_argvstr points to the
+ * argv vector, and ps_nargvstr is the same as the program's argc. The
+ * fields ps_envstr and ps_nenvstr are the equivalent for the environment.
+ *
+ * Programs should now use setproctitle(3) to change ps output.
+ * setproctitle() always informs the kernel with sysctl and sets the
+ * pointers in ps_strings.  The kern.proc.args sysctl first tries p_args.
+ * If p_args is NULL, it then falls back to reading ps_strings and following
+ * the pointers.
  */
 struct ps_strings {
 	char	**ps_argvstr;	/* first of 0 or more argument strings */
@@ -55,6 +60,7 @@ struct ps_strings {
 
 /*
  * Address of ps_strings structure (in user space).
+ * Prefer the kern.ps_strings or kern.proc.ps_strings sysctls to this constant.
  */
 #define	PS_STRINGS	(USRSTACK - sizeof(struct ps_strings))
 #define SPARE_USRSPACE	4096



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?53e54f92.239a.3d3730af>