UrfArbitrator: allow the arbitrator to continue if /dev/rfkill is unavailable

[cyphermox]

Signed-off-by: Mathieu Trudel-Lapierre mathieu.trudel-lapierre@canonical.com

[lcp]

I don't get it. The patch seems not changing anything...

[cyphermox]

Indeed, looking more into it it's missing pieces.

[lcp]

Sorry for my late reply. I didn't notice the new commits.
The QEMU profile looks fine.
I have concern about the UrfArbitrator patch. Since the rest of the code are based on the assumption that priv-fd is valid, the changes in urf_arbitrator_startup() seems not enough. For example, set_block() and finalize() will access priv->fd and this may cause problems. Could you handle those cases if necessary?

[cyphermox]

On Mon, Nov 10, 2014 at 10:53 PM, Gary Ching-Pang Lin <
notifications@github.com> wrote:

Sorry for my late reply. I didn't notice the new commits.
The QEMU profile looks fine.
I have concern about the UrfArbitrator patch. Since the rest of the code
are based on the assumption that priv-fd is valid, the changes in
urf_arbitrator_startup() seems not enough. For example, set_block() and
finalize() will access priv->fd and this may cause problems. Could you
handle those cases if necessary?

I'll actually resubmit this after a bit of rework, there are some changes
and bug fixes coming in from other collaborators.

Mathieu Trudel-Lapierre mathieu.tl@gmail.com
Freenode: cyphermox, Jabber: mathieu.tl@gmail.com
4096R/EE018C93 1967 8F7D 03A1 8F38 732E FF82 C126 33E1 EE01 8C93