From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753970AbbK0EXM (ORCPT ); Thu, 26 Nov 2015 23:23:12 -0500 Received: from zeniv.linux.org.uk ([195.92.253.2]:52676 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753883AbbK0EWo (ORCPT ); Thu, 26 Nov 2015 23:22:44 -0500 Date: Fri, 27 Nov 2015 04:22:39 +0000 From: Al Viro To: Tomas Winkler Cc: Greg Kroah-Hartman , linux-kernel@vger.kernel.org Subject: [bug] mei_fasync() bogosity Message-ID: <20151127042237.GW22011@ZenIV.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org in "mei: implement fasync for event notification" a ->fasync() instance is added as static int mei_fasync(int fd, struct file *file, int band) { struct mei_cl *cl = file->private_data; if (!mei_cl_is_connected(cl)) return POLLERR; return fasync_helper(fd, file, band, &cl->ev_async); } Valid return values for ->fasync() are 0, 1 and -E...; 8 (aka POLLERR) is not. What had been intended there? -ENODEV? Some other error? As it is, the thing is broken.