Project

General

Profile

Actions

Bug #8635

closed

epoll should not emit POLLNVAL

Added by Patrick Mooney over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Category:
kernel
Start date:
2017-09-07
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

This replicates downstream bug OS-5884

While checking the poll() handlers in various drivers across the system for proper POLLET behavior per OS-5882, it became clear that a subset of drivers can emit POLLNVAL via revents. While this is valid behavior for poll(2) and friends, it is not a recognized event value in the epoll API. There should be translation logic added to the devpoll output code.

Actions #1

Updated by Electric Monk over 4 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit 80d5689f5d4588adc071138e25e9d0d5252d9b55

commit  80d5689f5d4588adc071138e25e9d0d5252d9b55
Author: Patrick Mooney <pmooney@pfmooney.com>
Date:   2017-10-19T02:47:16.000Z

    8634 epoll fails to wake on certain edge-triggered conditions
    8635 epoll should not emit POLLNVAL
    8636 recursive epoll should emit EPOLLRDNORM
    Reviewed by: Jerry Jelinek <jerry.jelinek@joyent.com>
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Reviewed by: Igor Kozhukhov <igor@dilos.org>
    Approved by: Dan McDonald <danmcd@joyent.com>

Actions

Also available in: Atom PDF