)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"9e17b947799fadddd39cf900baf93a3e8070b825","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Allow mapping of multiple Eswtich (PFs) to a PCI device"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"SR-IOV agent maps PCI device to one SR-IOV PF"},{"line_number":10,"context_line":"we need to support multiple PF devices to on SR-IOV PCI device."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I70f78fc2d404a922f0ae445e13e230a7a02adfdf"},{"line_number":13,"context_line":"Closes-Bug: #1649080"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a71b18c_b3f0ddfe","line":10,"range":{"start_line":10,"start_character":42,"end_line":10,"end_character":44},"updated":"2016-12-11 14:03:02.000000000","message":"s/on/one","commit_id":"27e523d8fc95021042291ce7a99b7465948db532"},{"author":{"_account_id":21683,"name":"edan david","email":"edand@mellanox.com","username":"edand"},"change_message_id":"de7988cb5ea1cb50396b8adb1c4004c2f7d80f6e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Allow mapping of multiple Eswtich (PFs) to a PCI device"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"SR-IOV agent maps PCI device to one SR-IOV PF"},{"line_number":10,"context_line":"we need to support multiple PF devices to on SR-IOV PCI device."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I70f78fc2d404a922f0ae445e13e230a7a02adfdf"},{"line_number":13,"context_line":"Closes-Bug: #1649080"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a71b18c_93d6814d","line":10,"range":{"start_line":10,"start_character":42,"end_line":10,"end_character":44},"in_reply_to":"3a71b18c_b3f0ddfe","updated":"2016-12-11 14:05:41.000000000","message":"Done","commit_id":"27e523d8fc95021042291ce7a99b7465948db532"}],"neutron/plugins/ml2/drivers/mech_sriov/agent/eswitch_manager.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"71baa7256b35330fc1b0131d9f95d9b7714a68cf","unresolved":false,"context_lines":[{"line_number":369,"context_line":"        embedded_switch \u003d EmbSwitch(phys_net, dev_name, exclude_devices)"},{"line_number":370,"context_line":"        self.emb_switches_map.setdefault(phys_net, []).append(embedded_switch)"},{"line_number":371,"context_line":"        for pci_slot in embedded_switch.get_pci_slot_list():"},{"line_number":372,"context_line":"            self.pci_slot_map.setdefault(pci_slot, []).append(embedded_switch)"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    def _get_emb_eswitch(self, device_mac, pci_slot):"},{"line_number":375,"context_line":"        \"\"\"Get embedded switch."}],"source_content_type":"text/x-python","patch_set":4,"id":"3a461143_e55f4e70","line":372,"range":{"start_line":372,"start_character":12,"end_line":372,"end_character":78},"updated":"2017-01-26 10:51:13.000000000","message":"Please, correct me if I\u0027m wrong.\n\nThe whole patch relies on the fact that a pci_slot [1] could have several embedded switches attached to it.\n\nAn embedded switch contains a set of VFs and their PCI addresses. All this addresses are unique. Depending on the SR-IOV NIC manufacturer, the PCI address asignation will be diferent, but always unique.\n\nThat means, self.pci_slot_map will be always a dict of pci_addr:emb_switch, not a dict of pci_addr:[list of emb_switches]\n\n[1] domain:bus:device.function \u003c-- unique","commit_id":"da6a37116b0c305caacf3482a2860fe1f65c7fc3"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"91ebe3598c8135c4a688d12c91c1536454d3b966","unresolved":false,"context_lines":[{"line_number":427,"context_line":"            #other port before resetting the rate."},{"line_number":428,"context_line":"            if embedded_switch.get_pci_device(pci_slot) is None:"},{"line_number":429,"context_line":"                embedded_switch.set_device_rate(pci_slot, rate_type, 0)"},{"line_number":430,"context_line":"            else:"},{"line_number":431,"context_line":"                LOG.warning(_LW(\"VF with PCI slot %(pci_slot)s is already \""},{"line_number":432,"context_line":"                                \"assigned; skipping reset for \u0027%(rate_type)s\u0027 \""},{"line_number":433,"context_line":"                                \"device configuration parameter\"),"}],"source_content_type":"text/x-python","patch_set":4,"id":"dfeb2761_bf2b77e9","line":430,"updated":"2017-04-02 08:38:15.000000000","message":"Is this warning logic still correct? We can now have a case where one switch is cleared and the other isn\u0027t, right?","commit_id":"da6a37116b0c305caacf3482a2860fe1f65c7fc3"}]}