{"comment":{"url":"https://api.github.com/repos/mikehollinger/linux/comments/10702617","html_url":"https://github.com/mikehollinger/linux/commit/ea724b3a3ae60f079965b94c993cc0fc1ba3417f#commitcomment-10702617","id":10702617,"user":{"login":"mrochs","id":10732706,"avatar_url":"https://avatars.githubusercontent.com/u/10732706?v=3","gravatar_id":"","url":"https://api.github.com/users/mrochs","html_url":"https://github.com/mrochs","followers_url":"https://api.github.com/users/mrochs/followers","following_url":"https://api.github.com/users/mrochs/following{/other_user}","gists_url":"https://api.github.com/users/mrochs/gists{/gist_id}","starred_url":"https://api.github.com/users/mrochs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrochs/subscriptions","organizations_url":"https://api.github.com/users/mrochs/orgs","repos_url":"https://api.github.com/users/mrochs/repos","events_url":"https://api.github.com/users/mrochs/events{/privacy}","received_events_url":"https://api.github.com/users/mrochs/received_events","type":"User","site_admin":false},"position":92,"line":1387,"path":"drivers/scsi/cflash/cflash_mc.c","commit_id":"ea724b3a3ae60f079965b94c993cc0fc1ba3417f","created_at":"2015-04-13T22:46:19Z","updated_at":"2015-04-13T22:46:19Z","body":"This is fine for now, but we'll need to give this more thought later when revamping the global lun info support.\r\n\r\n- Need to add the LUN table setup in the qc=1 discovery path. Probably once we get some feedback from community.\r\n\r\n- For virtual luns we're limited on translation support for only 256 physical LUNs.\r\n\r\n- What happens when we remove a LUN? We'll need some type of mechanism to cap the 'last_lun_index' and manage which ones are free or not. We should probably also limit this to only LUNs that are operating in the virtual mode."}}