code.ossystems Code Review - openembedded-core.git/blob

   1 From 7019ba184b828ed7253750cf409fc5760ef90a54 Mon Sep 17 00:00:00 2001
   2 From: Alexander Kanavin <alex.kanavin@gmail.com>
   3 Date: Thu, 9 Jan 2020 17:44:05 +0100
   4 Subject: [PATCH] setup.py: pass missing libraries to Extension for
   5  multiprocessing module
   6
   7 In the following commit:
   8 ...
   9 commit e711cafab13efc9c1fe6c5cd75826401445eb585
  10 Author: Benjamin Peterson <benjamin@python.org>
  11 Date:   Wed Jun 11 16:44:04 2008 +0000
  12
  13     Merged revisions 64104,64117 via svnmerge from
  14     svn+ssh://pythondev@svn.python.org/python/trunk
  15 ...
  16 (see diff in setup.py)
  17 It assigned libraries for multiprocessing module according
  18 the host_platform, but not pass it to Extension.
  19
  20 In glibc, the following commit caused two definition of
  21 sem_getvalue are different.
  22 https://sourceware.org/git/?p=glibc.git;a=commit;h=042e1521c794a945edc43b5bfa7e69ad70420524
  23 (see diff in nptl/sem_getvalue.c for detail)
  24 `__new_sem_getvalue' is the latest sem_getvalue@@GLIBC_2.1
  25 and `__old_sem_getvalue' is to compat the old version
  26 sem_getvalue@GLIBC_2.0.
  27
  28 To build python for embedded Linux systems:
  29 http://www.yoctoproject.org/docs/2.3.1/yocto-project-qs/yocto-project-qs.html
  30 If not explicitly link to library pthread (-lpthread), it will
  31 load glibc's sem_getvalue randomly at runtime.
  32
  33 Such as build python on linux x86_64 host and run the python
  34 on linux x86_32 target. If not link library pthread, it caused
  35 multiprocessing bounded semaphore could not work correctly.
  36 ...
  37 >>> import multiprocessing
  38 >>> pool_sema = multiprocessing.BoundedSemaphore(value=1)
  39 >>> pool_sema.acquire()
  40 True
  41 >>> pool_sema.release()
  42 Traceback (most recent call last):
  43   File "<stdin>", line 1, in <module>
  44 ValueError: semaphore or lock released too many times
  45 ...
  46
  47 And the semaphore issue also caused multiprocessing.Queue().put() hung.
  48
  49 Upstream-Status: Pensing
  50
  51 Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
  52 Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
  53 ---
  54  setup.py | 2 +-
  55  1 file changed, 1 insertion(+), 1 deletion(-)
  56
  57 diff --git a/setup.py b/setup.py
  58 index ec3f2a4..b0f1541 100644
  59 --- a/setup.py
  60 +++ b/setup.py
  61 @@ -1671,7 +1671,7 @@ class PyBuildExt(build_ext):
  62                                     libraries=libs,
  63                                     include_dirs=["Modules/_multiprocessing"]))
  64
  65 -        self.add(Extension('_multiprocessing', multiprocessing_srcs,
  66 +        self.add(Extension('_multiprocessing', multiprocessing_srcs, libraries=['pthread'],
  67                             include_dirs=["Modules/_multiprocessing"]))
  68
  69      def detect_uuid(self):