From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takuya Yoshikawa Date: Wed, 12 May 2010 09:25:10 +0000 Subject: Re: [RFC][PATCH 7/12 not tested yet] PPC: introduce __set_bit() like Message-Id: <4BEA73F6.7020400@oss.ntt.co.jp> List-Id: References: <20100504215645.6448af8f.takuya.yoshikawa@gmail.com> <20100504220418.083929bc.takuya.yoshikawa@gmail.com> <4BE97F2A.5080900@suse.de> In-Reply-To: <4BE97F2A.5080900@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Alexander Graf Cc: Takuya Yoshikawa , avi@redhat.com, mtosatti@redhat.com, fernando@oss.ntt.co.jp, kvm@vger.kernel.org, kvm-ppc@vger.kernel.org, kvm-ia64@vger.kernel.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org, benh@kernel.crashing.org, paulus@samba.org, linuxppc-dev@ozlabs.org, arnd@arndb.de, linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org >> >> +static inline int set_bit_user_non_atomic(int nr, void __user *addr) >> +{ >> + u8 __user *p; >> + u8 val; >> + >> + p = (u8 __user *)((unsigned long)addr + nr / BITS_PER_BYTE); >> > > Does C do the + or the / first? Either way, I'd like to see brackets here :) OK, I'll change like that! I like brackets style too :) > > > Alex >