summaryrefslogtreecommitdiffstats
path: root/mtp/MtpStorage.cpp (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Fix erroneous infinite loop in MTP inotify_tEthan Yonker2015-02-091-1/+1
| | | | Change-Id: I60ce40bd08e2ce9b04beb5dc4876b86f3688861b
* Use TWAtomicInt in MTPStorageEthan Yonker2015-01-271-6/+5
| | | | Change-Id: I3450fd2334a441c3d9d40a856128a3cc932cb7f3
* MTP add/remove storage instead of disabling MTPEthan Yonker2014-12-191-12/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement a pipe between TWRP and MTP to allow TWRP to tell MTP to remove storage partitions as they become unavailable (e.g. during a wipe, unmount, etc) instead of disabling MTP completely. This includes some fixes and improvements in destructors to properly remove / delete various items. This also means that we will not be toggling adb off and on quite as often. I do not like that we had to add another thread, but we were unable to use select() on the mtp_usb character device because this device does not support polling. Select always returned indicating that the mtp file descriptor was ready to be read and the resulting read would block. The read block prevented us from being able to include reading of the pipe between TWRP and MTP in the main MTP thread. We might want to add a return pipe letting TWRP know if the removal of the storage device was successful, but I am not sure how we want to implement this. It would invovle timeouts in both TWRP and MTP to ensure that we returned a failure indicator in a timely manner to TWRP and prevent deleting the storage device in the case of a failure. Right now we make no attempt to ensure that an MTP operation is underway like a large file transfer, but we were not doing anything like this in the past. In some respects we have limited control over what happens. If the user installs a zip that unmounts a storage partition, we will not know about the change in storage status anyway. Regular Android does not have these troubles because partitions rarely get unmounted like in recovery. At some point, we have to hold the user accountable for performing actions that may remove a storage partition while they are using MTP anyway. Ideally we do not want to toggle the USB IDs and thus toggle adb off and on during early boot, but I am not sure what the best way to handle that at this time. Change-Id: I9343e5396bf6023d3b994de1bf01ed91d129bc14
* TWRP MTP: make sure inotify_fd is setEthan Yonker2014-11-251-9/+14
| | | | | | | | | | | Originally the inotify_init() call was part of a separate thread which would occasionally allow the readDirs function to run and start trying to add watches before inotify_fd was init'ed properly and result in the add watch failing. This patch set relocates the inotify_init call to happen outside and before starting the separate thread to ensure that inotify_fd is set properly. Change-Id: I8748c23473b60b57887df9d692834f3d2b249802
* mtp: fix log spam: findNode not finding a node is not an errorthat2014-10-141-1/+1
| | | | | | The requested file might be on another storage. Change-Id: Ia71525058b616a28a8ea2b4ab48ebac1c790757e
* mtp: cleanup, fixes and performance improvementsthat2014-10-081-644/+508
| | | | | | | | | | | - use std::map instead of linked list - read directories on demand - fix writing zip files to storage root - fix creating directories - lots of minor fixes - simplify generation of storage IDs and make them spec compliant Change-Id: I2137c27549ddbdc58466f2e3aeda464fac70a3c5
* MTP: Better handling of statEthan Yonker2014-09-041-16/+29
| | | | | | | | | | Do some error checking of lstat calls and better handle those errors. Use lstat instead of dirent to check if an item is a directory because dirent does not return the correct item type when using exfat-fuse. Change-Id: I0d008aff55a0e0b4fbeae24cf8d0c04453b7b77e
* MTP Fix rename in WindowsDees Troy2014-09-041-1/+1
| | | | Change-Id: Ieef2fc16aac1da3d03908cfb285895954b42a14a
* MTP Fix property list for multiple storage devicesDees Troy2014-09-041-1/+2
| | | | Change-Id: Ice23fef31439a6c88d0a12380688eccd8dc3a71c
* Prevent MTP enable on startup if crash detectedEthan Yonker2014-09-041-1/+1
| | | | Change-Id: Ie1388a8acadf714a29721dd90c7f9a8aa38a860d
* Improve MtpStorage handling when more than one storage is presentEthan Yonker2014-09-031-6/+5
| | | | Change-Id: I02f1984d02a85d9fd42450cc3772e88a43d9e9ff
* add mtp responder to TWRP.bigbiff bigbiff2014-09-031-0/+941
Big thanks to Dees_Troy for helping with the implementation. Change-Id: I6c9c522b9c9de5dc139e2ecb0141008182ba07f0