devm_request_irq should be paired with devm_free_irq.
Shared state should only be modified AFTER freeing the IRQ, since
free_irq services any pending IRQs first before actually freeing.

Bug 200331923

Change-Id: Id2af45dd364da5a8c8f49dd6d333908f059c5280
Signed-off-by: Faisal Zaghloul <fzaghloul@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1701854
Reviewed-by: Peter Newman <pnewman@nvidia.com>
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Vladislav Buzov <vbuzov@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

The ivc cdev driver incorrectly uses the hypervisor ivc queue
hwirq number when registering its interrupt handler (vs.
the associated hv device logical irq line number).  Due to this
incorrect registration the device does not receive any interrupts
resulting in it not being able to establish the ivc channel.
This change modifies the code to use the hv device logical
irq number associated with the ivc channel during irq handler
registration.

Bug 200308411
JIRA VFND-4214

Change-Id: I400863eb285046cae3c068c58d8fbf1aedb0bac9
Signed-off-by: John Moser <jmoser@nvidia.com>
Reviewed-on: http://git-master/r/1496247
Reviewed-on: http://git-master/r/1498656
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Dennis Kou <dkou@nvidia.com>
Reviewed-by: Vladislav Buzov <vbuzov@nvidia.com>

linux/sched.h should be included explicitly

Bug 200187990
ESSL-1335

Change-Id: I964a1272ce2abd95219d5080eb6d3414cd4cfa5f
Signed-off-by: dmitry pervushin <dpervushin@nvidia.com>
Reviewed-on: http://git-master/r/1128275
Reviewed-on: http://git-master/r/1198262
Reviewed-on: http://git-master/r/1314825
Reviewed-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
Tested-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
Reviewed-by: Bahadir Balban <bbalban@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Timo Alho <talho@nvidia.com>

Remove the second assignment of the struct ivc pointer in the IVC
character device polling function, which, depending on which assignment
the compiler keeps, could potentially circumvent the BUG_ON() verifying
that the device pointer is valid.

Bug 200116059
CID 20417

Change-Id: I397ca6efe4ce4ab6f6b97a9bac67d713f893910a
Signed-off-by: Peter Newman <pnewman@nvidia.com>
Reviewed-on: http://git-master/r/1009995
Reviewed-on: http://git-master/r/1198255
Reviewed-on: http://git-master/r/1314813
Reviewed-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
Tested-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Timo Alho <talho@nvidia.com>

Fix memory corruption issue when IVC queues have gaps in
their numbering: don't use queue number to index into ivc_dev_array

VFND-557

Change-Id: Ib962fc74fab39a882a7822bca226db29ba0fb9c8
Signed-off-by: Peter Daifuku <pdaifuku@nvidia.com>
(cherry picked from commit 7f45a5842c3dfec5f897e8619023f32c4e38f639)
Reviewed-on: http://git-master/r/789689
Reviewed-on: http://git-master/r/1198246
Reviewed-on: http://git-master/r/1314810
Reviewed-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
Tested-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Timo Alho <talho@nvidia.com>

tegra_hv_get_ivc_info() returns an ERR_PTR, not NULL when it fails.

VFND-557

Change-Id: Iaf38a59fc033560c9cab31fdabc7f9bc474e0763
Signed-off-by: Peter Newman <pnewman@nvidia.com>
Reviewed-on: http://git-master/r/785650
Reviewed-on: http://git-master/r/1198244
Reviewed-on: http://git-master/r/1314808
Reviewed-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
Tested-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Timo Alho <talho@nvidia.com>

To support users of the in-kernel IVC API that run before platform
devices are initialized, move initialization of IVC to the coreinit
phase. Split the IVC character device interface into a separate driver
that initializes much later.

 - Remove the option for building the tegra_hv driver as a loadable
   module, as this functionality is never tested, and IVC is needed far
   too early on Tegra platforms for this ever to be an option.
 - Add a tegra_hv_ivc_convert_cookie(), which is a temporary API for
   converting the cookie pointer returned by tegra_hv_ivc_reserve() to a
   struct ivc *, which is needed for writing code that is compatible
   with multiple IVC backends.

Bug VFND-557

Change-Id: I74eae3d495eff1173d08b2d11a313c9f0abde4f7
Signed-off-by: Peter Newman <pnewman@nvidia.com>
Signed-off-by: Arnab Basu <abasu@nvidia.com>
Reviewed-on: http://git-master/r/776251
Reviewed-on: http://git-master/r/1198243
Reviewed-on: http://git-master/r/1314807
Reviewed-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
Tested-by: Aniruddha Banerjee <aniruddhab@nvidia.com>
Reviewed-by: Bahadir Balban <bbalban@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Timo Alho <talho@nvidia.com>