{"action":"created","issue":{"url":"https://api.github.com/repos/zfsonlinux/zfs/issues/5925","repository_url":"https://api.github.com/repos/zfsonlinux/zfs","labels_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/5925/labels{/name}","comments_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/5925/comments","events_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/5925/events","html_url":"https://github.com/zfsonlinux/zfs/pull/5925","id":216979093,"node_id":"MDExOlB1bGxSZXF1ZXN0MTEyNTg5MDUz","number":5925,"title":"OpenZFS - 6363 Add UNMAP/TRIM functionality","user":{"login":"dweeezil","id":1432005,"node_id":"MDQ6VXNlcjE0MzIwMDU=","avatar_url":"https://avatars1.githubusercontent.com/u/1432005?v=4","gravatar_id":"","url":"https://api.github.com/users/dweeezil","html_url":"https://github.com/dweeezil","followers_url":"https://api.github.com/users/dweeezil/followers","following_url":"https://api.github.com/users/dweeezil/following{/other_user}","gists_url":"https://api.github.com/users/dweeezil/gists{/gist_id}","starred_url":"https://api.github.com/users/dweeezil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dweeezil/subscriptions","organizations_url":"https://api.github.com/users/dweeezil/orgs","repos_url":"https://api.github.com/users/dweeezil/repos","events_url":"https://api.github.com/users/dweeezil/events{/privacy}","received_events_url":"https://api.github.com/users/dweeezil/received_events","type":"User","site_admin":false},"labels":[{"id":40358,"node_id":"MDU6TGFiZWw0MDM1OA==","url":"https://api.github.com/repos/zfsonlinux/zfs/labels/Feature","name":"Feature","color":"02d7e1","default":false},{"id":453022836,"node_id":"MDU6TGFiZWw0NTMwMjI4MzY=","url":"https://api.github.com/repos/zfsonlinux/zfs/labels/Work%20in%20Progress","name":"Work in Progress","color":"b60205","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/zfsonlinux/zfs/milestones/12","html_url":"https://github.com/zfsonlinux/zfs/milestone/12","labels_url":"https://api.github.com/repos/zfsonlinux/zfs/milestones/12/labels","id":7014,"node_id":"MDk6TWlsZXN0b25lNzAxNA==","number":12,"title":"0.8.0","description":"ZFS on Linux v0.8.0 - ZFS+SPL Consolidation\r\n","creator":{"login":"behlendorf","id":148917,"node_id":"MDQ6VXNlcjE0ODkxNw==","avatar_url":"https://avatars2.githubusercontent.com/u/148917?v=4","gravatar_id":"","url":"https://api.github.com/users/behlendorf","html_url":"https://github.com/behlendorf","followers_url":"https://api.github.com/users/behlendorf/followers","following_url":"https://api.github.com/users/behlendorf/following{/other_user}","gists_url":"https://api.github.com/users/behlendorf/gists{/gist_id}","starred_url":"https://api.github.com/users/behlendorf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/behlendorf/subscriptions","organizations_url":"https://api.github.com/users/behlendorf/orgs","repos_url":"https://api.github.com/users/behlendorf/repos","events_url":"https://api.github.com/users/behlendorf/events{/privacy}","received_events_url":"https://api.github.com/users/behlendorf/received_events","type":"User","site_admin":false},"open_issues":59,"closed_issues":121,"state":"open","created_at":"2011-05-03T17:39:03Z","updated_at":"2018-06-20T17:12:18Z","due_on":null,"closed_at":null},"comments":97,"created_at":"2017-03-25T14:52:10Z","updated_at":"2018-07-07T16:48:58Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/zfsonlinux/zfs/pulls/5925","html_url":"https://github.com/zfsonlinux/zfs/pull/5925","diff_url":"https://github.com/zfsonlinux/zfs/pull/5925.diff","patch_url":"https://github.com/zfsonlinux/zfs/pull/5925.patch"},"body":"<!--- Provide a general summary of your changes in the Title above -->\r\n\r\n### Description\r\nAdd TRIM support.  Replacement for #3656.\r\n\r\n### Motivation and Context\r\n<!--- Why is this change required? What problem does it solve? -->\r\n<!--- If it fixes an open issue, please link to the issue here. -->\r\n\r\n### How Has This Been Tested?\r\n<!--- Please describe in detail how you tested your changes. -->\r\n<!--- Include details of your testing environment, and the tests you ran to -->\r\n<!--- see how your change affects other areas of the code, etc. -->\r\n<!--- If your change is a performance enhancement, please provide benchmarks here. -->\r\nVarious stress testing with an assortment of vdev types.\r\n\r\n### Types of changes\r\n<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->\r\n- [ ] Bug fix (non-breaking change which fixes an issue)\r\n- [ ] New feature (non-breaking change which adds functionality)\r\n- [x] Performance enhancement (non-breaking change which improves efficiency)\r\n- [ ] Code cleanup (non-breaking change which makes code smaller or more readable)\r\n- [ ] Breaking change (fix or feature that would cause existing functionality to change)\r\n\r\n### Checklist:\r\n<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->\r\n<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->\r\n- [ ] My code follows the ZFS on Linux code style requirements.\r\n- [ ] I have updated the documentation accordingly.\r\n- [ ] I have read the **CONTRIBUTING** document.\r\n- [ ] I have added tests to cover my changes.\r\n- [ ] All new and existing tests passed.\r\n- [ ] Change has been approved by a ZFS on Linux member.\r\n\r\nThis PR should integrate all the recent changes in the upstream patch set.  The stack also includes the separate fixes which were in #3656.  It seems stable so far during some fairly abusive testing on SSDs with various types of vdevs.  It does _not_ include the \"partial\" trim support of the previous PR."},"comment":{"url":"https://api.github.com/repos/zfsonlinux/zfs/issues/comments/403228630","html_url":"https://github.com/zfsonlinux/zfs/pull/5925#issuecomment-403228630","issue_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/5925","id":403228630,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMzIyODYzMA==","user":{"login":"dweeezil","id":1432005,"node_id":"MDQ6VXNlcjE0MzIwMDU=","avatar_url":"https://avatars1.githubusercontent.com/u/1432005?v=4","gravatar_id":"","url":"https://api.github.com/users/dweeezil","html_url":"https://github.com/dweeezil","followers_url":"https://api.github.com/users/dweeezil/followers","following_url":"https://api.github.com/users/dweeezil/following{/other_user}","gists_url":"https://api.github.com/users/dweeezil/gists{/gist_id}","starred_url":"https://api.github.com/users/dweeezil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dweeezil/subscriptions","organizations_url":"https://api.github.com/users/dweeezil/orgs","repos_url":"https://api.github.com/users/dweeezil/repos","events_url":"https://api.github.com/users/dweeezil/events{/privacy}","received_events_url":"https://api.github.com/users/dweeezil/received_events","type":"User","site_admin":false},"created_at":"2018-07-07T16:48:58Z","updated_at":"2018-07-07T16:48:58Z","author_association":"MEMBER","body":"I'm going to try to get this rebased to master over the weekend, however, the next Big Thing I want to bring into master is openzfs 9102 (f.k.a. \"eager zero\") which will likely be another big sticking point with TRIM given that they're somewhat analogous features.  I'm hoping that TRIM will be able to be refactored to use some of the new facilities (the per-vdev \"xlat\" method, in particular)."}}
{"actor":{"display_login":"dweeezil"}}