From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id C1DF9384C357; Tue, 13 Dec 2022 23:01:45 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C1DF9384C357 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1670972505; bh=tBEad7YiAJ4V2bjSZgM7F7DlDyhzrXEzghcaW7KTjUs=; h=From:To:Subject:Date:In-Reply-To:References:From; b=bEUCRAiC1n91LyGUD+AOJQR3Zt8xFO0f5fFCrBh6I7tkdu6Rn2jP2KnImWjLKAlHp W7I4JjXp1pI2+NJ1THNzlKmOahjVhTik0k5Ydb5uHX4QYKDkrByrtzcWAisWLwtm63 btuYU1AtRvngc9v51NLGxQZH4Yb4Gn5zvtNESy2E= From: "goldstein.w.n at gmail dot com" To: glibc-bugs@sourceware.org Subject: [Bug libc/29863] Segmentation fault in memcmp-sse2.S if memory contents can concurrently change Date: Tue, 13 Dec 2022 23:01:45 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: glibc X-Bugzilla-Component: libc X-Bugzilla-Version: 2.36 X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: goldstein.w.n at gmail dot com X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: unassigned at sourceware dot org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://sourceware.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 List-Id: https://sourceware.org/bugzilla/show_bug.cgi?id=3D29863 --- Comment #13 from Noah Goldstein --- (In reply to K.S. Bhaskar from comment #12) > While I can't speak to the other databases, the YottaDB code base goes ba= ck > much further than 25 years. Under the name GT.M, it first saw live > production use in 1986. Its use of optimistic concurrency control was > inspired by Kung & Robinson's seminal paper > (https://www.eecs.harvard.edu/~htk/publication/1981-tods-kung-robinson.pd= f). > I personally know the original author of the code, as well as someone who > has worked on the code base since the mid 1980s. In addition to those > mentioned by Narayanan, the code has over the decades run on several other > CPU architectures and operating systems. >=20 > I hope this suffices as a use case. Can you test the following dif: ``` @@ -308,7 +308,7 @@ L(ret_nonzero_vec_end_0): setg %dl leal -1(%rdx, %rdx), %eax # else - addl %edx, %eax + addq %rdx, %rax movzbl (VEC_SIZE * -1 + SIZE_OFFSET)(%rsi, %rax), %ecx movzbl (VEC_SIZE * -1 + SIZE_OFFSET)(%rdi, %rax), %eax subl %ecx, %eax ``` If that fixes things I'm okay making the change. I'm opposed to explicitly supporting it, but am happy to try and make this failure from the unsupported usage less dramatic. --=20 You are receiving this mail because: You are on the CC list for the bug.=