Ticket #273 (closed defect: invalid)

Opened 5 years ago

Last modified 5 years ago

nogil check for usage of new temps

Reported by: dagss Owned by: dagss
Priority: major Milestone: 0.11.2
Component: Parsing Keywords:
Cc:

Description

In ModuleNode?.py, around line 1350, there used to be a check for whether Python temporaries were used in functions declared nogil. This was not implemented for the new temp system (so likely a bug in 0.11 here).

I haven't checked whether this is a problem since the last gil refactoring. At any rate the GIL check should probably be moved to the new gil checker transform instead.

I'm also thinking that perhaps checking for the type of the temps is a bit "indirect" way of doing things, and perhaps it is more robust to simply filter the allowed operations in a function.

Change History

Changed 5 years ago by dagss

See also #205. And, to some degree, #210.

Changed 5 years ago by dagss

It is Nodes.py:1396

Changed 5 years ago by robertwb

  • milestone changed from 0.11.1 to 0.11.2

Changed 5 years ago by scoder

Is this really a problem? What operations can there be that allocate a Python temp but do not require the GIL themselves?

Changed 5 years ago by dagss

Basically I was just assuming that the check existed there for a reason (and decided to make a note rather than forget about it).

But it may be that we should just close this as invalid and count on any such strange cases to show up sometime and be handled in a more proper way.

Changed 5 years ago by dagss

  • status changed from new to closed
  • resolution set to invalid

See Stefan's comment.

Changed 5 years ago by dagss

  • status changed from closed to reopened
  • resolution invalid deleted

Changed 5 years ago by dagss

  • owner changed from somebody to dagss
  • status changed from reopened to new

Changed 5 years ago by dagss

  • status changed from new to closed
  • resolution set to invalid
Note: See TracTickets for help on using tickets.