From owner-svn-src-all@FreeBSD.ORG Thu Jan 6 21:14:43 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1233) id CAC8A106576E; Thu, 6 Jan 2011 21:14:43 +0000 (UTC) Date: Thu, 6 Jan 2011 21:14:43 +0000 From: Alexander Best To: Garrett Cooper Message-ID: <20110106211443.GA48244@freebsd.org> References: <201101041413.p04EDA4f038360@svn.freebsd.org> <20110106204157.GA41314@freebsd.org> <20110106211017.GA46874@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20110106211017.GA46874@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Konstantin Belousov Subject: Re: svn commit: r216955 - head/usr.sbin/rtprio X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 06 Jan 2011 21:14:43 -0000 On Thu Jan 6 11, Alexander Best wrote: > On Thu Jan 6 11, Garrett Cooper wrote: > > On Thu, Jan 6, 2011 at 12:41 PM, Alexander Best wrote: > > > > ... > > > > > this causes problems when pid is -0: > > > > > > [id|rt]prio -t -0 and [id|rt]prio 10 -0 will try to run "0" via execvp(). > > > beforehand however this will also trigger rtprio(). > > > > > > a better solution would be to do: > > > > > >                if (argv[2][0] == '-') { > > >                        proc = parseint(argv[2] + 1, "pid"); > > >                        if (rtprio(RTP_SET, proc, &rtp) != 0) > > >                                err(1, "RTP_SET"); > > >                } else { > > >                        execvp(argv[2], &argv[2]); > > >                                err(1, "%s", argv[2]); > > >                } > > > > How did you get a pid of -0? > > pid 0 stands for the current process. see rptio(2). rtprio(2) that is. ;) > > cheers. > alex > > > Thanks, > > -Garrett > > -- > a13x -- a13x