Hold GIL while allocating memory for PythonString.

[tatyana-krasnukha]

Summary

Swig wraps C++ code into SWIG_PYTHON_THREAD_BEGIN_ALLOW; ... SWIG_PYTHON_THREAD_END_ALLOW;
Thus, lldb crashs with "Fatal Python error: Python memory allocator called without holding the GIL" when calls lldb_SB***___str__ function.

[tatyana-krasnukha]

%feature("nothreadallow") looks more appropriate in this case. GetDescription method is quite cheap, no need to handle the GIL.

[granata.enrico]

I don't work on LLDB anymore. I am sure Greg will do a fine job reviewing!

[clayborg]

Not sure about this one. IIUC we now wouldn't take the GIL for these functions now and hope that the str() function doesn't do something that would require thread safety?

[tatyana-krasnukha]

Yes, and %feature("nothreadallow") doesn't allow to release GIL before calling these functions - it prevents Swig from wrapping them into SWIG_PYTHON_THREAD_BEGIN_ALLOW .. SWIG_PYTHON_THREAD_END_ALLOW block.

[clayborg]

What kind of scenario causes this crash to happen? Seems like a hack?

[tatyana-krasnukha]

I have a third party module which implements a custom command for LLDB. The command calls str(target), str(process), etc. for logging.

Anyway, every call to these functions will cause this crash to happen, because PythonString is allocated after releasing the lock.

[tatyana-krasnukha]

Use more readable aliases to set end clear the feature.

scripts/Python/python-extensions.swig

 
 %extend lldb::SBAddress {
+        %nothreadallow;
         PyObject *lldb::SBAddress::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBBlock {
+        %nothreadallow;
         PyObject *lldb::SBBlock::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBBreakpoint {
+        %nothreadallow;
         PyObject *lldb::SBBreakpoint::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
         
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 
 }
 %extend lldb::SBBreakpointLocation {
+        %nothreadallow;
         PyObject *lldb::SBBreakpointLocation::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelFull);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 
 %extend lldb::SBBreakpointName {
+        %nothreadallow;
         PyObject *lldb::SBBreakpointName::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 
 %extend lldb::SBBroadcaster {
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 
 %extend lldb::SBCommandReturnObject {
+        %nothreadallow;
         PyObject *lldb::SBCommandReturnObject::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
         
         /* the write() and flush() calls are not part of the SB API proper, and are solely for Python usage
         they are meant to make an SBCommandReturnObject into a file-like object so that instructions of the sort
         print >>sb_command_return_object, "something"
         will work correctly */
 
         void lldb::SBCommandReturnObject::write (const char* str)
         {
             if (str)
                 $self->Printf("%s",str);
         }
         void lldb::SBCommandReturnObject::flush ()
         {}
 }
 %extend lldb::SBCompileUnit {
+        %nothreadallow;
         PyObject *lldb::SBCompileUnit::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 %extend lldb::SBData {
+        %nothreadallow;
         PyObject *lldb::SBData::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBDebugger {
+        %nothreadallow;
         PyObject *lldb::SBDebugger::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBDeclaration {
+        %nothreadallow;
         PyObject *lldb::SBDeclaration::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
         
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 
 }
 %extend lldb::SBError {
+        %nothreadallow;
         PyObject *lldb::SBError::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBFileSpec {
+        %nothreadallow;
         PyObject *lldb::SBFileSpec::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBFrame {
+        %nothreadallow;
         PyObject *lldb::SBFrame::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBFunction {
+        %nothreadallow;
         PyObject *lldb::SBFunction::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
         
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 
 }
 %extend lldb::SBInstruction {
+        %nothreadallow;
         PyObject *lldb::SBInstruction::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBInstructionList {
+        %nothreadallow;
         PyObject *lldb::SBInstructionList::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBLineEntry {
+        %nothreadallow;
         PyObject *lldb::SBLineEntry::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
         
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 
 %extend lldb::SBMemoryRegionInfo {
+        %nothreadallow;
         PyObject *lldb::SBMemoryRegionInfo::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 
 %extend lldb::SBModule {
+        %nothreadallow;
         PyObject *lldb::SBModule::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
         
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 
 %extend lldb::SBModuleSpec {
+        %nothreadallow;
         PyObject *lldb::SBModuleSpec::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 
 %extend lldb::SBModuleSpecList {
+        %nothreadallow;
         PyObject *lldb::SBModuleSpecList::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 
 %extend lldb::SBProcess {
+        %nothreadallow;
         PyObject *lldb::SBProcess::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBSection {
+        %nothreadallow;
         PyObject *lldb::SBSection::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
         
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
%extend lldb::SBStream {
         {
             if (str)
                 $self->Printf("%s",str);
         }
         void lldb::SBStream::flush ()
         {}
 }
 %extend lldb::SBSymbol {
+        %nothreadallow;
         PyObject *lldb::SBSymbol::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
     %pythoncode %{
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 %extend lldb::SBSymbolContext {
+        %nothreadallow;
         PyObject *lldb::SBSymbolContext::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBSymbolContextList {
+        %nothreadallow;
         PyObject *lldb::SBSymbolContextList::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 
 %extend lldb::SBTarget {
+        %nothreadallow;
         PyObject *lldb::SBTarget::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 
 %extend lldb::SBType {
+        %nothreadallow;
         PyObject *lldb::SBType::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBTypeCategory {
+        %nothreadallow;
         PyObject *lldb::SBTypeCategory::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBTypeFilter {
+        %nothreadallow;
         PyObject *lldb::SBTypeFilter::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 %extend lldb::SBTypeFormat {
+        %nothreadallow;
         PyObject *lldb::SBTypeFormat::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBTypeMember {
+        %nothreadallow;
         PyObject *lldb::SBTypeMember::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBTypeMemberFunction {
+        %nothreadallow;
         PyObject *lldb::SBTypeMemberFunction::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBTypeEnumMember {
+        %nothreadallow;
         PyObject *lldb::SBTypeEnumMember::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBTypeNameSpecifier {
+        %nothreadallow;
         PyObject *lldb::SBTypeNameSpecifier::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 %extend lldb::SBTypeSummary {
+        %nothreadallow;
         PyObject *lldb::SBTypeSummary::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 %extend lldb::SBTypeSynthetic {
+        %nothreadallow;
         PyObject *lldb::SBTypeSynthetic::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelBrief);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 %extend lldb::SBThread {
+        %nothreadallow;
         PyObject *lldb::SBThread::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
     %pythoncode %{ 
         def __eq__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return False 
             
             return getattr(_lldb,self.__class__.__name__+"___eq__")(self, rhs)
             
         def __ne__(self, rhs):
             if not isinstance(rhs, type(self)): 
                 return True 
             
             return getattr(_lldb,self.__class__.__name__+"___ne__")(self, rhs)
     %}
 }
 %extend lldb::SBValue {
+        %nothreadallow;
         PyObject *lldb::SBValue::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBValueList {
+        %nothreadallow;
         PyObject *lldb::SBValueList::__str__ (){
                 lldb::SBStream description;
                 const size_t n = $self->GetSize();
                 if (n)
                 {
                     for (size_t i=0; i<n; ++i)
                         $self->GetValueAtIndex(i).GetDescription(description);
                 }
                 else
                 {
                     description.Printf("<empty> lldb.SBValueList()");
                 }
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 %extend lldb::SBWatchpoint {
+        %nothreadallow;
         PyObject *lldb::SBWatchpoint::__str__ (){
                 lldb::SBStream description;
                 $self->GetDescription (description, lldb::eDescriptionLevelVerbose);
                 const char *desc = description.GetData();
                 size_t desc_len = description.GetSize();
                 if (desc_len > 0 && (desc[desc_len-1] == '\n' || desc[desc_len-1] == '\r'))
                     --desc_len;
                 if (desc_len > 0)
                     return lldb_private::PythonString(llvm::StringRef(desc, desc_len)).release();
                 else
                     return lldb_private::PythonString("").release();
         }
+        %clearnothreadallow;
 }
 
 
 // %extend lldb::SBDebugger {
 //         // FIXME: We can't get the callback and baton
 //         PyObject *lldb::SBDebugger (){
 //             // Only call Py_XDECREF if we have a Python object (or NULL)
 //             if (LLDBSwigPythonCallPythonLogOutputCallback == $self->GetLogOutPutCallback())