Date: Thu, 12 May 2011 05:15:38 +0200 From: Attilio Rao <attilio@freebsd.org> To: src-committers@freebsd.org, svn-src-projects@freebsd.org, Bruce Evans <brde@optusnet.com.au>, Warner Losh <imp@freebsd.org>, Artem Belevich <art@freebsd.org>, Oleksandr Tymoshenko <gonzo@freebsd.org> Subject: Re: svn commit: r221614 - projects/largeSMP/sys/powerpc/include Message-ID: <BANLkTi=e7GtBM-PTq9yJHSLRoaOWh62AeA@mail.gmail.com> In-Reply-To: <201105080039.p480doiZ021493@svn.freebsd.org> References: <201105080039.p480doiZ021493@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
2011/5/8 Attilio Rao <attilio@freebsd.org>: > Author: attilio > Date: Sun May =C2=A08 00:39:49 2011 > New Revision: 221614 > URL: http://svn.freebsd.org/changeset/base/221614 > > Log: > =C2=A0All architectures define the size-bounded types (uint32_t, uint64_t= , etc.) > =C2=A0starting from base C types (int, long, etc). mips seems having the same issue, so here is my patch: http://www.freebsd.org/~attilio/largeSMP/mips-atomic.diff Please note that mips defines the 16/8 bits functions within support.S but it really hadn't do (I didn't rewrite as inline functions because I wasn't confident enough with modifying mips assembly). In this patch I removed the 8-bits operations like they aren't used anywhere and actually they weren't complete at all. I tried almost all the MIPS config file with all modules and they compile fine with this patch. I have cpumask_t -> cpuset_t conversion ready and I'd commit to this tree just after someone has tested this atomic alone. Thanks, Attilio --=20 Peace can only be achieved by understanding - A. Einstein
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?BANLkTi=e7GtBM-PTq9yJHSLRoaOWh62AeA>