Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757635Ab0LTVty (ORCPT ); Mon, 20 Dec 2010 16:49:54 -0500 Received: from mail-iw0-f174.google.com ([209.85.214.174]:48537 "EHLO mail-iw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752587Ab0LTVtx (ORCPT ); Mon, 20 Dec 2010 16:49:53 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=dWg1k5OnvCiumZMxddszRTm+iBnLQRJnr7AZne6ursgUYG3NBKfwrGpqMu5zp5dQmS jfEGiUZOU39YZ2pPCF1LFZukkzrZ9fA+pb1CBA3z3fyeav2BYOkowNXFiaLDj6bjNJdK JFmvmW1MaCB9kyFFsoxVw+1U3w+MtuZqFER5g= MIME-Version: 1.0 In-Reply-To: <1292875718-7980-2-git-send-email-sboyd@codeaurora.org> References: <1292649385-28771-1-git-send-email-sboyd@codeaurora.org> <1292875718-7980-1-git-send-email-sboyd@codeaurora.org> <1292875718-7980-2-git-send-email-sboyd@codeaurora.org> Date: Mon, 20 Dec 2010 16:49:52 -0500 Message-ID: Subject: Re: [PATCH 1/3] hvc_dcc: Fix bad code generation by marking assembly volatile From: Arnaud Lacombe To: Stephen Boyd Cc: Greg Kroah-Hartman , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Tony Lindgren , Arnd Bergmann , Nicolas Pitre , Daniel Walker Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 672 Lines: 17 Hi, On Mon, Dec 20, 2010 at 3:08 PM, Stephen Boyd wrote: > Without marking the asm __dcc_getstatus() volatile my compiler > decides [...] What compiler ? That might be a usefull information to know, espectially 5 years from now when tracing code history. There has been similar issue with gcc 4.5 recently. AFAIK, in the same idea, the final change has been to mark the asm volatile. Thanks, - Arnaud -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/