New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 618911 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

[serial] FIrst read will never complete on some posix platforms

Project Member Reported by aschulman@chromium.org, Jun 10 2016

Issue description

OS: OSX

(1) Open a serial connection to a device that already has data ready to read.
(2) Read.
(3) Wait forever and the read will never notify that it has completed.
 
Status: Assigned (was: Untriaged)
Status: Fixed (was: Assigned)
This was fixed with https://codereview.chromium.org/2057973002/, right?

(If not, please reopen the bug.)
Status: Assigned (was: Fixed)
That patch was never submitted because it has an outstanding comment.
Yep, got sidetracked with other pressing issues. Will come back to this one soon.
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b09276a67a58e6aa0c23d8a052f0b7bd4c9ca668

commit b09276a67a58e6aa0c23d8a052f0b7bd4c9ca668
Author: aschulman <aschulman@chromium.org>
Date: Thu Jul 28 20:38:29 2016

[serial] When a read is requested on posix platforms, try the read immediately.

On some posix platforms (e.g., OSX) the first read after a serial device is
opened may never complete. The problem is on these platforms the message loop
will not notify that a file descriptor is ready to read if it was in that state
when it was opened.

This fixed this problem by trying to perform a read immediately after it is requested.

BUG= 618911 

Review-Url: https://codereview.chromium.org/2057973002
Cr-Commit-Position: refs/heads/master@{#408467}

[modify] https://crrev.com/b09276a67a58e6aa0c23d8a052f0b7bd4c9ca668/device/serial/serial_io_handler_posix.cc
[modify] https://crrev.com/b09276a67a58e6aa0c23d8a052f0b7bd4c9ca668/device/serial/serial_io_handler_posix.h

Status: Fixed (was: Assigned)

Sign in to add a comment