From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon-590EEB7GvNiWaY/ihj7yzEB+6BGkLq7r@public.gmane.org Subject: [Bug 75431] posix_fadvise(2): wrong param name? Date: Sat, 03 May 2014 19:18:42 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-man-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-man@vger.kernel.org https://bugzilla.kernel.org/show_bug.cgi?id=75431 Michael Kerrisk changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED CC| |mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org Resolution|--- |CODE_FIX --- Comment #1 from Michael Kerrisk --- (In reply to Eric Siegerman from comment #0) > In posix_fadvise(2), under "Architecture-specific variants", it says: > [Due to register-alignment issues,] > ... the call signature of posix_fadvise() shown in > the SYNOPSIS would force a register to be wasted as padding between > the fd and len arguments. > > Should it say "offset" instead of "len" in the last line of that quoted bit? > > Rationale: the SYNOPSIS shows a call signature of: > int posix_fadvise(int fd, off_t offset, off_t len, int advice); > in which: > - fd and len aren't adjacent > - it's between a shorter value followed by a longer one that I'd expect > alignment padding to be needed, and the only pair where that occurs is (fd, > offset) The change sounds right to me. I've made it. > CAVEAT: I'm not familiar with either the syscall or the specific alignment > issues in question, so my guess about this is just that -- a deduced guess, > NOT specific knowledge. So don't make the change on my say-so; please > verify it first! I'm going to trust you on this one ;-). I was the one who injected the error in the first place. -- You are receiving this mail because: You are watching the assignee of the bug. -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html