Re: [patch] delete sync.i in ia64_switch_to()

david mosberger
2005-09-07T17:59:45Z

On 9/7/05, Jim Hull <jim.hull@hp.com> wrote:
> That sync.i is not old code - it is required.

Yes, the sync.i was added by Asit at some point precisely for the
reasons you explained. I believe we concluded at the time that the
"mf" wasn't needed since we'd be guaranteed to have a fence in the
execution-path anyhow (IIRC, it was due to locks being
acquired/released on the paths in question). I'm not sure, however,
whether this point has been considered when kernel-preemption support
was added. The folks caring about CONFIG_PREEMPT may want to
reconsider that issue if not.

--david