[{"id":1766249,"web_url":"http://patchwork.ozlabs.org/comment/1766249/","msgid":"<5d53a24d-53d5-02ed-331c-f5ac7fae70f4@canonical.com>","list_archive_url":null,"date":"2017-09-11T11:45:08","subject":"ACK/cmnt: [SRU][Zesty][PATCH 0/2] Fixes for LP:1715073","submitter":{"id":2898,"url":"http://patchwork.ozlabs.org/api/people/2898/","name":"Stefan Bader","email":"stefan.bader@canonical.com"},"content":"On 08.09.2017 15:38, Joseph Salisbury wrote:\n> BugLink: http://bugs.launchpad.net/bugs/1715073\n> \n> == SRU Justification ==\n> IBM reports that When doing SRIOV if they unload VFs they will see a warning\n> that is coming from the fix for bugzilla: Bug 146479 (LP1625318).\n> \n> Commit 5b0948dfe is a prereq commit for Zesty, but not for Artful, since that commit \n> was included in mainline as of 4.11-rc1.\n> \n> Commit 0fc690a7c3f is not in mainline as of yet, and was a clean pick from linux-next.\n> Commit 0fc690a7c3f is also need in Artful, but will be SRU'd seperatly, since Z needs\n> a prereq commit.\n> \n> == Fixes ==\n> commit 5b0948dfe138f0837699f46f5877f4f81c252dac\n> Author: Emil Tantilov \n> Date: Fri Jan 6 13:59:08 2017 -0800\n> \n> PCI: Lock each enable/disable num_vfs operation in sysfs\n> \n> commit 0fc690a7c3f7053613dcbab6a7613bb6586d8ee2\n> Author: Gavin Shan \n> Date: Fri Aug 11 18:19:33 2017 +1000\n> PCI: Disable VF decoding before pcibios_sriov_disable() updates resources\n> \n> \n> \n> == Regression Potential ==\n> The primary patch is only adding a call to pcibios_sriov_disable(), which \n> disables VF memory decoding before calling pcibios_sriov_disable(). No other major\n> changes are made.\n> \n> == Test Case ==\n> A test kernel was built with these patches and tested by the original bug reporter.\n> The bug reporter states the test kernel resolved the bug.\n> \n> \n> Emil Tantilov (1):\n> PCI: Lock each enable/disable num_vfs operation in sysfs\n> \n> Gavin Shan (1):\n> PCI: Disable VF decoding before pcibios_sriov_disable() updates\n> resources\n> \n> drivers/pci/iov.c | 14 ++++----------\n> drivers/pci/pci-sysfs.c | 23 ++++++++++++++++-------\n> drivers/pci/pci.h | 2 +-\n> 3 files changed, 21 insertions(+), 18 deletions(-)\n> \nAcked-by: Stefan Bader \n\nWe can add the bug link to patch #2 when committing (can use this hopefully as\nreminder).","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrR1r4Tmvz9sxR;\n\tMon, 11 Sep 2017 21:45:20 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1drN9P-0000wN-53; Mon, 11 Sep 2017 11:45:15 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1drN9J-0000tV-KH\n\tfor kernel-team@lists.ubuntu.com; Mon, 11 Sep 2017 11:45:09 +0000","from 1.general.smb.uk.vpn ([10.172.193.28])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1drN9J-0003dK-CX\n\tfor kernel-team@lists.ubuntu.com; Mon, 11 Sep 2017 11:45:09 +0000"],"Subject":"ACK/cmnt: [SRU][Zesty][PATCH 0/2] Fixes for LP:1715073","To":"kernel-team@lists.ubuntu.com","References":"","From":"Stefan Bader ","Message-ID":"<5d53a24d-53d5-02ed-331c-f5ac7fae70f4@canonical.com>","Date":"Mon, 11 Sep 2017 13:45:08 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============8355347648678951848==\"","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" "}},{"id":1766285,"web_url":"http://patchwork.ozlabs.org/comment/1766285/","msgid":"<82cb009d-c64a-00a8-b488-79383dcfd508@canonical.com>","list_archive_url":null,"date":"2017-09-11T12:50:03","subject":"ACK: [SRU][Zesty][PATCH 0/2] Fixes for LP:1715073","submitter":{"id":2900,"url":"http://patchwork.ozlabs.org/api/people/2900/","name":"Colin Ian King","email":"colin.king@canonical.com"},"content":"On 08/09/17 14:38, Joseph Salisbury wrote:\n> BugLink: http://bugs.launchpad.net/bugs/1715073\n> \n> == SRU Justification ==\n> IBM reports that When doing SRIOV if they unload VFs they will see a warning\n> that is coming from the fix for bugzilla: Bug 146479 (LP1625318).\n> \n> Commit 5b0948dfe is a prereq commit for Zesty, but not for Artful, since that commit \n> was included in mainline as of 4.11-rc1.\n> \n> Commit 0fc690a7c3f is not in mainline as of yet, and was a clean pick from linux-next.\n> Commit 0fc690a7c3f is also need in Artful, but will be SRU'd seperatly, since Z needs\n> a prereq commit.\n> \n> == Fixes ==\n> commit 5b0948dfe138f0837699f46f5877f4f81c252dac\n> Author: Emil Tantilov \n> Date: Fri Jan 6 13:59:08 2017 -0800\n> \n> PCI: Lock each enable/disable num_vfs operation in sysfs\n> \n> commit 0fc690a7c3f7053613dcbab6a7613bb6586d8ee2\n> Author: Gavin Shan \n> Date: Fri Aug 11 18:19:33 2017 +1000\n> PCI: Disable VF decoding before pcibios_sriov_disable() updates resources\n> \n> \n> \n> == Regression Potential ==\n> The primary patch is only adding a call to pcibios_sriov_disable(), which \n> disables VF memory decoding before calling pcibios_sriov_disable(). No other major\n> changes are made.\n> \n> == Test Case ==\n> A test kernel was built with these patches and tested by the original bug reporter.\n> The bug reporter states the test kernel resolved the bug.\n> \n> \n> Emil Tantilov (1):\n> PCI: Lock each enable/disable num_vfs operation in sysfs\n> \n> Gavin Shan (1):\n> PCI: Disable VF decoding before pcibios_sriov_disable() updates\n> resources\n> \n> drivers/pci/iov.c | 14 ++++----------\n> drivers/pci/pci-sysfs.c | 23 ++++++++++++++++-------\n> drivers/pci/pci.h | 2 +-\n> 3 files changed, 21 insertions(+), 18 deletions(-)\n> \nPositive testing results.\n\nAcked-by: Colin Ian King ","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrSSf3VStz9s4s;\n\tMon, 11 Sep 2017 22:50:10 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1drOAA-0002Bd-5R; Mon, 11 Sep 2017 12:50:06 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1drOA8-0002Ag-Bc\n\tfor kernel-team@lists.ubuntu.com; Mon, 11 Sep 2017 12:50:04 +0000","from 1.general.cking.uk.vpn ([10.172.193.212])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1drOA7-00076m-Pq; Mon, 11 Sep 2017 12:50:03 +0000"],"Subject":"ACK: [SRU][Zesty][PATCH 0/2] Fixes for LP:1715073","To":"kernel-team@lists.ubuntu.com","References":"","From":"Colin Ian King ","Message-ID":"<82cb009d-c64a-00a8-b488-79383dcfd508@canonical.com>","Date":"Mon, 11 Sep 2017 13:50:03 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101\n\tThunderbird/45.8.0","MIME-Version":"1.0","In-Reply-To":"","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" "}},{"id":1769198,"web_url":"http://patchwork.ozlabs.org/comment/1769198/","msgid":"<877693e1-132d-5d7c-d357-ac2e10866284@canonical.com>","list_archive_url":null,"date":"2017-09-15T13:43:36","subject":"APPLIED Z: [SRU][Zesty][PATCH 0/2] Fixes for LP:1715073","submitter":{"id":2898,"url":"http://patchwork.ozlabs.org/api/people/2898/","name":"Stefan Bader","email":"stefan.bader@canonical.com"},"content":"On 08.09.2017 15:38, Joseph Salisbury wrote:\n> BugLink: http://bugs.launchpad.net/bugs/1715073\n> \n> == SRU Justification ==\n> IBM reports that When doing SRIOV if they unload VFs they will see a warning\n> that is coming from the fix for bugzilla: Bug 146479 (LP1625318).\n> \n> Commit 5b0948dfe is a prereq commit for Zesty, but not for Artful, since that commit \n> was included in mainline as of 4.11-rc1.\n> \n> Commit 0fc690a7c3f is not in mainline as of yet, and was a clean pick from linux-next.\n> Commit 0fc690a7c3f is also need in Artful, but will be SRU'd seperatly, since Z needs\n> a prereq commit.\n> \n> == Fixes ==\n> commit 5b0948dfe138f0837699f46f5877f4f81c252dac\n> Author: Emil Tantilov \n> Date: Fri Jan 6 13:59:08 2017 -0800\n> \n> PCI: Lock each enable/disable num_vfs operation in sysfs\n> \n> commit 0fc690a7c3f7053613dcbab6a7613bb6586d8ee2\n> Author: Gavin Shan \n> Date: Fri Aug 11 18:19:33 2017 +1000\n> PCI: Disable VF decoding before pcibios_sriov_disable() updates resources\n> \n> \n> \n> == Regression Potential ==\n> The primary patch is only adding a call to pcibios_sriov_disable(), which \n> disables VF memory decoding before calling pcibios_sriov_disable(). No other major\n> changes are made.\n> \n> == Test Case ==\n> A test kernel was built with these patches and tested by the original bug reporter.\n> The bug reporter states the test kernel resolved the bug.\n> \n> \n> Emil Tantilov (1):\n> PCI: Lock each enable/disable num_vfs operation in sysfs\n> \n> Gavin Shan (1):\n> PCI: Disable VF decoding before pcibios_sriov_disable() updates\n> resources\n> \n> drivers/pci/iov.c | 14 ++++----------\n> drivers/pci/pci-sysfs.c | 23 ++++++++++++++++-------\n> drivers/pci/pci.h | 2 +-\n> 3 files changed, 21 insertions(+), 18 deletions(-)\n> \nApplied to Zesty master-next. Also added BugLink to patch #2 and removed\nlinux-next reference as this is now upstream with same sha1.","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=kernel-team-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xtxSc6drDz9sRm;\n\tFri, 15 Sep 2017 23:43:44 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1dsquB-0001sm-6K; Fri, 15 Sep 2017 13:43:39 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1dsqu8-0001sC-Ut\n\tfor kernel-team@lists.ubuntu.com; Fri, 15 Sep 2017 13:43:36 +0000","from 1.general.smb.uk.vpn ([10.172.193.28])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1dsqu8-0005OH-Ms\n\tfor kernel-team@lists.ubuntu.com; Fri, 15 Sep 2017 13:43:36 +0000"],"Subject":"APPLIED Z: [SRU][Zesty][PATCH 0/2] Fixes for LP:1715073","To":"kernel-team@lists.ubuntu.com","References":"","From":"Stefan Bader ","Message-ID":"<877693e1-132d-5d7c-d357-ac2e10866284@canonical.com>","Date":"Fri, 15 Sep 2017 15:43:36 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"","X-BeenThere":"kernel-team@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Kernel team discussions ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============2121066680984633051==\"","Errors-To":"kernel-team-bounces@lists.ubuntu.com","Sender":"\"kernel-team\" "}}]