From patchwork Thu Jun 19 16:51:38 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Siddhesh Poyarekar X-Patchwork-Id: 1571 Received: (qmail 1200 invoked by alias); 19 Jun 2014 16:50:20 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 1188 invoked by uid 89); 19 Jun 2014 16:50:19 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.6 required=5.0 tests=AWL, BAYES_00, SPF_HELO_PASS, SPF_PASS, T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Date: Thu, 19 Jun 2014 22:21:38 +0530 From: Siddhesh Poyarekar To: libc-alpha@sourceware.org Subject: [PATCH] Fix build warning in pthread_rwlock_* Message-ID: <20140619165137.GR7238@spoyarek.pnq.redhat.com> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.22.1-rc1 (2013-10-16) Hi, The first argument of elision_adapt and that of ELISION_*LOCK have different signs since __elision_rwcount is singned char and the argument of elision_adapt is uint8_t. Modified elision_adapt to accept int8_t instead of uint8_t. Siddhesh * sysdeps/x86/nptl/elide.h (elision_adapt): Make first argument type int8_t. diff --git a/sysdeps/x86/nptl/elide.h b/sysdeps/x86/nptl/elide.h index 19f27e5..3979146 100644 --- a/sysdeps/x86/nptl/elide.h +++ b/sysdeps/x86/nptl/elide.h @@ -26,7 +26,7 @@ /* Adapt elision with ADAPT_COUNT and STATUS and decide retries. */ static inline bool -elision_adapt(uint8_t *adapt_count, unsigned int status) +elision_adapt(int8_t *adapt_count, unsigned int status) { if (status & _XABORT_RETRY) return false;