maria-developers team mailing list archive

[Vicențiu Ciorbaru <cvicentiu@xxxxxxxxx>]

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 <svoj@xxxxxxxxxxx> 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     : maria-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~maria-developers
> More help   : https://help.launchpad.net/ListHelp
>