Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Feb 2003 09:42:44 -0500 (EST)
From:      Daniel Eischen <eischen@pcnet1.pcnet.com>
To:        Alexey Zelkin <phantom@FreeBSD.org.ua>
Cc:        Alfred Perlstein <alfred@FreeBSD.org>, cvs-src@FreeBSD.org
Subject:   Re: cvs commit: src/include pthread.h src/lib/libc_r/uthread uthread_attr_getstack.c uthread_attr_setstack.c src/lib/libpthread/thread thr_attr_getstack.c thr_attr_setstack.c src/lib/libc_r/man pthread_attr.3
Message-ID:  <Pine.GSO.4.10.10302100937430.5059-100000@pcnet1.pcnet.com>
In-Reply-To: <20030210120147.A25796@phantom.cris.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 10 Feb 2003, Alexey Zelkin wrote:
> hi,
> 
> Who reviewed this change ?  Did you told to Dan before doing it ?

Yeah, I want a backout!  Back it out now!  Just kidding :-) :-)

These are added functions in the latest POSIX spec.  They do the
same thing as *_[gs]etstackaddr() and *_[gs]etstacksize().

[In the kindest words imaginable] In the future I would like
to review changes to libc_r.

> And last, that purpose of this functionality ?  (I have seen a diff,
> but it's interesting how these functions are supposed to be used?)
> 
> PS: We have pthread_attr_get_np() which provide functionality similar
> to pthread_attr_getstack() in cleaner and correct way.  Having of
> pthread_attr_setstack() is absolutely crazy idea (IMHO) because using it
> after pthread_create() may (or will) crash application.
> 
> On Mon, Feb 10, 2003 at 12:48:04AM -0800, Alfred Perlstein wrote:
> > alfred      2003/02/10 00:48:04 PST
> > 
> >   Modified files:
> >     include              pthread.h 
> >     lib/libc_r/man       pthread_attr.3 
> >   Added files:
> >     lib/libc_r/uthread   uthread_attr_getstack.c 
> >                          uthread_attr_setstack.c 
> >     lib/libpthread/thread thr_attr_getstack.c thr_attr_setstack.c 
> >   Log:
> >   Add pthread_attr_getstack() and pthread_attr_setstack().
> >   
> >   Submitted by: Craig Rodrigues <rodrigc@attbi.com>
> >   
> >   Revision  Changes    Path
> >   1.27      +4 -0      src/include/pthread.h
> >   1.12      +8 -0      src/lib/libc_r/man/pthread_attr.3
> >   1.1       +59 -0     src/lib/libc_r/uthread/uthread_attr_getstack.c (new)
> >   1.1       +58 -0     src/lib/libc_r/uthread/uthread_attr_setstack.c (new)
> >   1.1       +59 -0     src/lib/libpthread/thread/thr_attr_getstack.c (new)
> >   1.1       +58 -0     src/lib/libpthread/thread/thr_attr_setstack.c (new)

-- 
Dan Eischen


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-src" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.GSO.4.10.10302100937430.5059-100000>