lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Fri, 26 Jan 2018 13:31:23 +0100
From:   Christoph Hellwig <hch@....de>
To:     Benjamin LaHaise <ben@...munityfibre.ca>
Cc:     Christoph Hellwig <hch@....de>, viro@...iv.linux.org.uk,
        Avi Kivity <avi@...lladb.com>, linux-aio@...ck.org,
        linux-fsdevel@...r.kernel.org, netdev@...r.kernel.org,
        linux-api@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: aio poll, io_pgetevents and a new in-kernel poll API V4

On Thu, Jan 25, 2018 at 03:10:25PM -0500, Benjamin LaHaise wrote:
> I implemented something similar back in December, but did so without
> changing the in-kernel poll API.  See below for the patch that implements
> it.  Is changing the in-kernel poll API really desirable given how many
> drivers that will touch?

I had various previous versions that did not touch the driver API,
but there are a couple issues with that:

 (1) you cannot make the API race free.  With the existing convoluted
     poll_table_struct-based API you can't check for pending items
     before adding yourself to the waitqueue in a race free manner.
 (2) you cannot make the submit non-blocking without deferring to
     a workqueue or similar and thus incurring another context switch
 (3) you cannot deliver events from the wakeup callback, incurring
     another context switch, this time in the wakeup path that actually
     matters for some applications
 (3) the in-kernel poll API really is broken to start with and needs
     to be fixed anyway.  I'd rather rely on that instead of working
     around decades old cruft that has no reason to exist.

Powered by blists - more mailing lists