From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/13359 Path: news.gmane.org!.POSTED!not-for-mail From: Rich Felker Newsgroups: gmane.linux.lib.musl.general Subject: Re: riscv port for review Date: Thu, 11 Oct 2018 11:49:27 -0400 Message-ID: <20181011154927.GB5150@brightrain.aerifal.cx> References: <20180928022404.GQ17995@brightrain.aerifal.cx> Reply-To: musl@lists.openwall.com NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: blaine.gmane.org 1539272856 3240 195.159.176.226 (11 Oct 2018 15:47:36 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Thu, 11 Oct 2018 15:47:36 +0000 (UTC) User-Agent: Mutt/1.5.21 (2010-09-15) To: musl@lists.openwall.com Original-X-From: musl-return-13375-gllmg-musl=m.gmane.org@lists.openwall.com Thu Oct 11 17:47:32 2018 Return-path: Envelope-to: gllmg-musl@m.gmane.org Original-Received: from mother.openwall.net ([195.42.179.200]) by blaine.gmane.org with smtp (Exim 4.84_2) (envelope-from ) id 1gAdBU-0000lP-6Q for gllmg-musl@m.gmane.org; Thu, 11 Oct 2018 17:47:32 +0200 Original-Received: (qmail 1740 invoked by uid 550); 11 Oct 2018 15:49:40 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Original-Received: (qmail 1721 invoked from network); 11 Oct 2018 15:49:40 -0000 Content-Disposition: inline In-Reply-To: Original-Sender: Rich Felker Xref: news.gmane.org gmane.linux.lib.musl.general:13359 Archived-At: On Thu, Oct 11, 2018 at 12:34:40AM -0700, Michael Forney wrote: > On 2018-09-27, Rich Felker wrote: > > Pulled from here: > > https://github.com/riscv/riscv-musl/commit/6a4f4a9c774608add4b02f95322518bd2f5f51ee > > > > Attached for review. > > I noticed that some fcntl.h constants are incorrect (O_DIRECTORY, > O_NOFOLLOW, O_DIRECT, O_LARGEFILE, and O_TMPFILE). Linux doesn't seem > to have a riscv-specific fcntl.h, so I think they just come from > asm-generic. If that's the case the riscv32/riscv64 bits/fcntl.h files should just be removed and the generic one will be used. Rich