Bart,
The use of the global variable in the os_unix.py file would seem to introduce a potential concurrency issue if multiple threads are using this code. Consider the case where separate threads are operating on separate user images simultaneously. Currently, the pkg(5) code handles this, but with this fix, there could be a problem with accessing the __been_here dictionary.

Instead of checking on every call, can't the module just call the offending get group name method once as it is loaded? And maybe that call should be in os_sunos.py rather than os_unix.py is that it is only done for Solaris?

Thanks.
Tom


Bart Smaalders wrote:

http://cr.opensolaris.org/~barts/reboot_needed/webrev/

9387 pkg fix should honor reboot-needed flag

Note that this fix is designed to prevent the upgrade
of files that require a reboot on a live image; it doesn't
yet deal w/ automatically creating a new BE if that's the
desired behavior.


begin:vcard
fn:Tom Mueller
n:Mueller;Tom
org:Sun Microsystems, Inc.;Update Center Software
adr:;;21915 Hillandale Dr;Elkhorn;NE;68022;USA
email;internet:[email protected]
title:Senior Staff Engineer
tel;work:877-250-4011
tel;fax:877-250-4011
tel;home:402-916-9943
x-mozilla-html:TRUE
version:2.1
end:vcard

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to