[PATCH] delete fork_tcb()

Denys Vlasenko dvlasenk at redhat.com
Wed Jun 22 12:37:53 UTC 2011


On Wed, 2011-06-22 at 02:40 +0400, Dmitry V. Levin wrote:
> On Tue, Jun 21, 2011 at 08:26:07PM +0200, Denys Vlasenko wrote:
> > Hi,
> > 
> > Sometime ago expand_tcbtab/alloc_tcb were modified
> > so that they never return failure (they will abort instead).
> > 
> > This opens up a possibility for further simplifications.
> > 
> > This patch gets rid of fork_tcb() function. It used to do
> > what the comment above it says, but now it doesn't do much:
> > it only sets tcp->flags |= TCB_FOLLOWFORK and maybe calls
> > expand_tcbtab(). The second operation is not necessary, since
> > alloc_tcp will do it itself when needed.
> > 
> > This patch deletes fork_tcb, open-coding tcp->flags |= TCB_FOLLOWFORK
> > where it was formerly called. It also makes nprocs and tcbtabsize
> > variables static. (While at it, I nuked redundant extern char **environ
> > declaration: strace.c had *two* of them...)
> > 
> > Please review.
> 
> Looks good.

Thanks for review, pushed to git (both patches).

-- 
vda






More information about the Strace-devel mailing list