As a developer, I want to replace Android Parcels by a more general (non-android) IPC mechanism in NFC module, so that NFC module will be less dependent on android. (Hence, it enhances the portability for non-android platform, regarding NFC module).
Summary: Remove/reduce Android dependencies in gecko/gonk → [NFC] Remove/reduce Android dependencies in gecko/gonk
Currently NFC is using Android Binder Parcels internally which should be possible to replace in both Gecko and Gonk
> > 2) |UnixSocketBuffer| is possible but not optimal, because it returns a > sequence of characters. Since you're going to break the protocol, you should > instead build upon the code in hal/daemon. It implements sockets on top of > SOCK_SEQPACKET, which gives you individual messages one by one. So no more > searching for EOMs in the byte stream. I believe that Nfc ipc is using SOCK_SEQPACKET to communicate with nfcd (saw it in ipc/nfc/NfcConnector.cpp). I think it does similarly to daemonsocket
Removed ni, discussed with Thomas internally
Attachment #8709311 - Attachment description: Gecko patch v1 → Gecko patch WIP
Created attachment 8709320 [details] [diff] [review] Gonk patch WIP
Attachment #8709312 - Attachment is obsolete: true
Hi Naoki I am trying to remove Android Parcel in Nfc gecko and nfcd daemon. There're some paddings in Android Parcel implementation, so removing usage of Parcel will break the interface between Gecko and nfcd. If the change is accepted and landed, nfc feature will stop working in all devices. User/developer will need FOTA to get nfc working again. Is it possible to have FOTA on all devices after landing this bug? Thanks
To prevent the smoke tests from failing completely, yes, I guess we can do a FOTA for Aries for internal users and Flame. I would need to take the patches and push it into the FOTA build. Is that correct? For Nexus builds, I am working on making it so that users can FOTA these builds. 1) The only group that will miss out is the mozfest contributor groups that we cannot FOTA the devices for. I do not know how many of those people use NFC. 2) Also, I would have to wait for code review before I start this if we are to go ahead with this plan.
Removing parcel shouldn't affect IPC between Gecko and NFCD. Please stop discussing FOTA solution here.
(In reply to Yoshi Huang[:allstars.chh] from comment #9) > Removing parcel shouldn't affect IPC between Gecko and NFCD. > Please stop discussing FOTA solution here. As discussed in offline meetings, we agree to implement this bug with minimum impact to the current NFC architecture. That is, we keep Parcel in NFC daemon and only remove it from Gecko part of NFC. So that we don't have to concern FOTA. Yoshi or Dimi could elaborate the implementation detail. p.s. Rename the bug title to make its purpose clearer.
Assignee: nobody → tnguyen
Status: NEW → ASSIGNED
Summary: [NFC] Remove/reduce Android dependencies in gecko/gonk → [NFC] Remove Android Parcel from Gecko
Assignee: tnguyen → nobody
Status: ASSIGNED → NEW
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.