Hi Sergey, Thanks for the review. I would push the fix into 5.5 if there are no buildbot failures. There seems to be something wrong with solaris though. Need to #ifdef the __sync_synchronize() there. The patch generally applies to 5.5 as well.
Since we're having a 5.5 week next, I'll test it for 5.5 as well, then push. Vicentiu On Tue, 2 Aug 2016 at 13:47 Sergey Vojtovich <[email protected]> wrote: > Hi Vicentiu, > > Ok to push. This patch is against 10.2, do you plan to fix previous > versions > separately? > > On Mon, Aug 01, 2016 at 02:46:03PM +0300, vicentiu wrote: > > revision-id: 5486f3458859fa4bd161150b9ac0e9c0f633a0e3 > (mariadb-10.2.1-9-g5486f34) > > parent(s): 08683a726773f8cdf16a4a3dfb3920e5f7842481 > > author: Vicențiu Ciorbaru > > committer: Vicențiu Ciorbaru > > timestamp: 2016-08-01 14:43:41 +0300 > > message: > > > > MDEV-10341: InnoDB: Failing assertion: mutex_own(mutex) - mutex_exit_func > > > > Fix memory barrier issues on releasing mutexes. We must have > Looks like a cutted comment. > > > > > --- > > storage/innobase/include/os0sync.h | 17 ++++++++--------- > > storage/innobase/include/sync0sync.ic | 2 ++ > > storage/xtradb/include/os0sync.h | 16 ++++++++-------- > > storage/xtradb/include/sync0sync.ic | 2 ++ > > 4 files changed, 20 insertions(+), 17 deletions(-) > > Thanks, > Sergey > > _______________________________________________ > Mailing list: https://launchpad.net/~maria-developers > Post to : [email protected] > Unsubscribe : https://launchpad.net/~maria-developers > More help : https://help.launchpad.net/ListHelp >
_______________________________________________ Mailing list: https://launchpad.net/~maria-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~maria-developers More help : https://help.launchpad.net/ListHelp
