The message buffer received by gh_msgq_recv is now not directly used by
any upper layers and always copied to a "buff" in gh_rm_connection, so
the same buffer can be re-used in the gh_msgq_recv loop.

Change-Id: I92988711b84b6f1e6b653c7d8cb5b66abc7a322e
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

No need to allocate a message queue buffer for each message queue
fragment, re-use the same one.

Change-Id: I5743a704ec08732a2e405724ba795d2d8cea90e3
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

Gracefully handle non-standard sequences of messages such as
continuation messages without receiving a reply/notification message
first, or a new reply/notification message without having finished the
last message sequence.

Change-Id: I52922fc9ccefb1f7dbb40fbffd2ad6b7fd1162b7
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

Consolidate the handling of complete replies or notifications into one
common function that is called after each message is received. This
reduces redundant lines of code.

Change-Id: Iaa52c9d07d86327498f6aa2b60fa8aaa6bfe2c7f
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

Rename "recv_buff" and "recv_buff_size" to "payload" and "size",
respectively. gh_rm_connection on tracks received buffers, so it is
redundant to add "recv". Rename "reply_err_code" to "rm_error" to make
it clear that this field is used for the resource manager's return error
code, and not some other value. While here, re-org and document the
gh_rm_connection struct.

Change-Id: I101262dd907a0e678d8d2c5232a17745f057edaf
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

current_recv_buffer tracks the current end of an ongoing message from
resource manager. It can be easily tracked by "conection->recv_buff +
connection->recv_buff_size". Do that instead to make code clearer.

Change-Id: I191dbdbf39c802868e1c6a262ef5420c02f127b2
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

Currently, there is an optimized path to directly use the message queue
buffer when a received notification fits within a single message and
there aren't any fragments. This greatly complicates buffer management
and might not provide any measurable benefit.

Suppose Linux receives a notification with one fragment then a
notification with 3 fragments. In current flow, there is 1 alloc for the
first notification and 4 allocations for the 2nd notification (1 for
each message + 1 for overall combined buffer). In the new flow, we could
optimize this to 0 allocations for each message because we can re-use
the same message buffer for each fragment and then a single alloc for
the entire payload.

There is, however, a slightly increased cost for single-message
notifications in the form of an extra memcpy from the message buffer to
notification buffer. Under the code optimizations and simplifications
that come with this cost, it is a worthy cost.

Change-Id: Ib52bd332f59b108d7dab9121c0beefed0aa66994
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

Preparation for cleaning up the resource manager core.

Notification size validation needs to only be performed on the payload
size. The RM RPC header will always be present, else the RM driver would
not have been able to determine that the message is a notification.
Thus, considering the RPC header is not required.

Change-Id: Id6a561a77d693e4f8f307d9d0d96b8d173940c60
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>

Enable QCOM_QFPROM_SYS config for anorak.

Change-Id: I7831f23e2a2622e6ae9f937e877e8ad07d75ca42
Signed-off-by: Kamati Srinivas <quic_kamasrin@quicinc.com>

This change adds missing device destroy for module remove case
and probe failure case.

Change-Id: I8da4ccce8ef07f2d52d94c5f0b2812e80326ede4
Signed-off-by: Maulik Shah <quic_mkshah@quicinc.com>

Enable crash on error during boot.

Change-Id: I406c3c96bf20433426aa939e769051d171a18edb
Signed-off-by: Ram Prakash Gupta <quic_rampraka@quicinc.com>

Protect maps lists while map find operation

Change-Id: Id91ef41360adc30ea1fae9a1fe0523cfbe5f6101
Signed-off-by: Vamsi Krishna Gattupalli <quic_vgattupa@quicinc.com>

Added ICE(Inline Crypto Engine) and FBE(File Based Encryption)
related drivers in anorak target first stage module list.

Test: Tested build compilation.

Change-Id: I5a08c929a806c9308e743d9734bb502885c0b91d
Signed-off-by: Rahul L <quic_rahull@quicinc.com>

 Enable support for secure subsytem's access control restrictions.

Change-Id: I0f793ce4e734a5c5b6c7a8921ab5bdea746f8817
Signed-off-by: Mariam Elia <quic_melia@quicinc.com>