[illumos-Developer] [REVIEW] 174 libpthread uberdata stale after 6735446

Garrett D'Amore garrett at nexenta.com
Sat Sep 11 15:05:53 PDT 2010


Looks good to me!  I think you should go ahead and start the integration
process.

	- Garrett

On Sat, 2010-09-11 at 17:35 -0400, Gordon Ross wrote:
> This is small enough so I'm not bothering with a webrev
> unless someone asks for one.  Please review.  Thx!
> 
> --- old/usr/src/lib/libc/inc/thr_uberdata.h
> +++ new/usr/src/lib/libc/inc/thr_uberdata.h
> @@ -860,6 +860,8 @@
>  
>  /*
>   * This is data that is global to all link maps (uberdata, aka super-global).
> + * Note: When changing this, please be sure to keep the 32-bit variant of
> + * this in sync.  (see uberdata32_t below)
>   */
>  typedef struct uberdata {
>  	pad_lock_t	_link_lock;
> @@ -1116,6 +1118,7 @@
>  	caddr32_t	atforklist;
>  	caddr32_t	robustlocks;
>  	caddr32_t	robustlist;
> +	caddr32_t	progname;
>  	caddr32_t	tdb_bootstrap;
>  	tdb32_t		tdb;
>  } uberdata32_t;
> 
> 
> _______________________________________________
> Developer mailing list
> Developer at lists.illumos.org
> http://lists.illumos.org/m/listinfo/developer




More information about the Developer mailing list