[PATCH 1/1] stp: rt: replace spin_lock with stp style lock and use STP_ALLOC_FLAGS

David Smith dsmith@redhat.com
Thu Oct 22 16:34:00 GMT 2015


On 10/22/2015 02:46 AM, Zhu Yanjun wrote:
> -rt mode spin lock lead to __might_sleep calltrace.
> Replacing spin lock with stp type raw lock and
> changing STP_ALLOC_SLEEP_FLAGS to STP_ALLOC_FLAGS solves the problem.

In general, this patch looks fine. However, I'm not too sure about the
STP_ALLOC_SLEEP_FLAGS bit. Can show us a backtrace that happens with
your spinlock changes but without the alloc flags changes or explain why
the alloc flags change is necessary? It could be that we're using the
wrong set of flags in the caller.

Thanks.

-- 
David Smith
dsmith@redhat.com
Red Hat
http://www.redhat.com
256.217.0141 (direct)
256.837.0057 (fax)



More information about the Systemtap mailing list