This message generated a parse failure. Raw output follows here. Please use 'back' to navigate. From devnull@lkml.org Thu Apr 25 01:13:12 2024 Delivery-date: Fri, 12 May 2006 13:42:44 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751241AbWELLmO (ORCPT ); Fri, 12 May 2006 07:42:14 -0400 Received: from spirit.analogic.com ([204.178.40.4]:6663 "EHLO spirit.analogic.com") by vger.kernel.org with ESMTP id S1751249AbWELLmN convert rfc822-to-8bit (ORCPT ); Fri, 12 May 2006 07:42:13 -0400 Received: from chaos.analogic.com ([10.112.50.11]) by phoenix.analogic.com with Microsoft SMTPSVC(6.0.3790.211); Fri, 12 May 2006 07:42:12 -0400 Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Received: from chaos.analogic.com (localhost [127.0.0.1]) by chaos.analogic.com (8.12.11/8.12.11) with ESMTP id k4CBgCLx008725; Fri, 12 May 2006 07:42:12 -0400 X-MimeOLE: Produced By Microsoft Exchange V6.5.7226.0 Received: (from linux-os@localhost) by chaos.analogic.com (8.12.11/8.12.11/Submit) id k4CBg76P008724; Fri, 12 May 2006 07:42:07 -0400 X-OriginalArrivalTime: 12 May 2006 11:42:12.0159 (UTC) FILETIME=[1BBFC4F0:01C675B9] content-class: urn:content-classes:message Subject: Re: Linux poll() again Date: Fri, 12 May 2006 07:42:07 -0400 Message-Id: In-Reply-To: <20060511211615.GA8485@us.ibm.com> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: Linux poll() again Thread-Index: AcZ1uRvJO98qLENCRoSJvOLUc7BzsA== References: <20060511204741.GG22741@us.ibm.com> <20060511211615.GA8485@us.ibm.com> From: "linux-os \(Dick Johnson\)" To: "Nishanth Aravamudan" Cc: "Linux kernel" , Reply-To: "linux-os \(Dick Johnson\)" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 11 May 2006, Nishanth Aravamudan wrote: > On 11.05.2006 [17:04:46 -0400], linux-os (Dick Johnson) wrote: >> >> On Thu, 11 May 2006, Nishanth Aravamudan wrote: >> >>> On 11.05.2006 [10:25:29 -0400], linux-os (Dick Johnson) wrote: >>>> >>>> >>>> Hello, >>>> >>>> I'm trying to fix a long-standing bug which has a >>>> work-around that has been working for a year or >>>> so. >>> >>> >>> >>>> Here is relevent code: >>>> >>>> for(;;) { >>>> mem->pfd.fd = fd; >>>> mem->pfd.events = POLLIN|POLLERR|POLLHUP|POLLNVAL; >>>> mem->pfd.revents = 0x00; >>> >>> Hrm, in looking at the craziness that is sys_poll() for a bit, I think >>> it's the underlying f_ops that are responsible for not setting POLLHUP, >>> that is: >>> >>> if (file != NULL) { >>> mask = DEFAULT_POLLMASK; >>> if (file->f_op && file->f_op->poll) >>> mask = file->f_op->poll(file, *pwait); >>> mask &= fdp->events | POLLERR | POLLHUP; >>> fput_light(file, fput_needed); >>> } >>> >>> and file->f_op->poll(file, *pwait) is not setting POLLHUP on the >>> disconnect. What filesystem is this? >> >> I think that's the problem. A socket isn't a file-system and the >> code won't set either bits if it isn't. Perhaps, the kernel code >> needs to consider a socket as a virtual file of some kind? Surely >> one needs to use poll() on sockets, no? > > Duh, I'm not reading well today -- for sockets, we do > > file->f_op->poll() -> (socket_file_ops) sock_poll() -> sock->ops->poll() > > So, now I need to know what kind of socket is this to go from there .. > > Thanks, > Nish A stream socket can be "connected". Anything that can be connected needs to know when the connection is broken. socket(AF_INET, SOCK_STREAM, IPPROTO_IP); ip_sock.sin_family = AF_INET; Such a socket is bound to an address and port using bind(), listen() is established, the accept() is called to accept connections. Accept returns a socket (fd) of the connected host. It's this fd that needs to "know" if/when the host disconnects. Cheers, Dick Johnson Penguin : Linux version 2.6.16.4 on an i686 machine (5592.89 BogoMips). New book: http://www.lymanschool.com _  **************************************************************** The information transmitted in this message is confidential and may be privileged. Any review, retransmission, dissemination, or other use of this information by persons or entities other than the intended recipient is prohibited. If you are not the intended recipient, please notify Analogic Corporation immediately - by replying to this message or by sending an email to DeliveryErrors@analogic.com - and destroy all copies of this information, including any attachments, without reading or disclosing them. Thank you. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/