Re: [PATCH 1/1] s390/pci: Log new handle in clp_disable_fh()

2020-06-02 Thread Niklas Schnelle
Hi Petr, sorry for not reacting sooner, I was on holiday, saw your message but figured it wasn't super critical. Thanks for finding this issue and thank you Pierre and Vasily for stepping in. Best, Niklas Schnelle On 5/28/20 11:08 AM, Petr Tesarik wrote: > Hi all, > > just a gentle ping. > >

Re: [PATCH 1/1] s390/pci: Log new handle in clp_disable_fh()

2020-05-28 Thread Vasily Gorbik
On Thu, May 28, 2020 at 12:01:45PM +0200, Pierre Morel wrote: > > On 2020-05-28 11:08, Petr Tesarik wrote: > > Hi all, > > > > just a gentle ping. > > > > If the current behaviour (logging the original handle) was intended, > > then it was worth mentioning in the commit message for 17cdec960cf77

Re: [PATCH 1/1] s390/pci: Log new handle in clp_disable_fh()

2020-05-28 Thread Pierre Morel
On 2020-05-28 11:08, Petr Tesarik wrote: Hi all, just a gentle ping. If the current behaviour (logging the original handle) was intended, then it was worth mentioning in the commit message for 17cdec960cf77, which made the change, but since that's no longer an option, I'd be happy with an exp

Re: [PATCH 1/1] s390/pci: Log new handle in clp_disable_fh()

2020-05-28 Thread Petr Tesarik
Hi all, just a gentle ping. If the current behaviour (logging the original handle) was intended, then it was worth mentioning in the commit message for 17cdec960cf77, which made the change, but since that's no longer an option, I'd be happy with an explanation in email. Petr T On Fri, 22 May 20

[PATCH 1/1] s390/pci: Log new handle in clp_disable_fh()

2020-05-22 Thread Petr Tesarik
After disabling a function, the original handle is logged instead of the disabled handle. Fixes: 17cdec960cf77 (s390/pci: Recover handle in clp_set_pci_fn()) Signed-off-by: Petr Tesarik --- arch/s390/pci/pci_clp.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/s390/pc