From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Moyer Subject: Re: [PATCH 2/6] move _body_io_syscall to the generic syscall.h Date: Fri, 05 Jan 2018 17:49:59 -0500 Message-ID: References: <20180104080325.14716-1-hch@lst.de> <20180104080325.14716-3-hch@lst.de> <20180105164031.GK27850@kvack.org> Mime-Version: 1.0 Content-Type: text/plain Cc: Christoph Hellwig , linux-aio@kvack.org, Avi Kivity , linux-fsdevel@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Benjamin LaHaise Return-path: In-Reply-To: <20180105164031.GK27850@kvack.org> (Benjamin LaHaise's message of "Fri, 5 Jan 2018 11:40:31 -0500") Sender: owner-linux-aio@kvack.org List-Id: netdev.vger.kernel.org Hi, Ben, Thanks for the quick reply. Benjamin LaHaise writes: > On Fri, Jan 05, 2018 at 11:25:17AM -0500, Jeff Moyer wrote: >> Christoph Hellwig writes: >> >> > This way it can be used for the fallback 6-argument version on >> > all architectures. >> > >> > Signed-off-by: Christoph Hellwig >> >> This is a strange way to do things. However, I was never really sold on >> libaio having to implement its own system call wrappers. That decision >> definitely resulted in some maintenance overhead. >> >> Ben, what was your reasoning for not just using syscall? > > The main issue was that glibc's pthreads implementation really sucked back > during initial development and there was a use-case for having the io_XXX > functions usable directly from clone()ed threads that didn't have all the > glibc pthread state setup for per-cpu areas to handle per-thread errno. > That made sense back then, but is rather silly today. Thanks for the background info. > Technically, I'm not sure the generic syscall wrapper is safe to use. The > io_XXX arch wrappers don't modify errno, while it appears the generic one > does. That said, nobody has ever noticed... Good point. Common architectures don't use the generic syscall wrapper, so I'm not sure we can conclude that it won't break anything. At the same time, I'm not sure I want to write and test the io_syscall6 assembly for all of the supported arches. I could save and restore errno. That sounds ugly, but less painful than the other options. Does anyone have any strong preferences? -Jeff -- To unsubscribe, send a message with 'unsubscribe linux-aio' in the body to majordomo@kvack.org. For more info on Linux AIO, see: http://www.kvack.org/aio/ Don't email: aart@kvack.org