Re: [PATCH] KVM: s390: fix build failure
- Original Message - > From: "Stephen Rothwell" <s...@canb.auug.org.au> > To: "Heiko Carstens" <heiko.carst...@de.ibm.com> > Cc: "Christian Borntraeger" <borntrae...@de.ibm.com>, "Sudip Mukherjee" > <sudipm.mukher...@gmail.com>, "Cornelia Huck" > <cornelia.h...@de.ibm.com>, "Paolo Bonzini" <pbonz...@redhat.com>, "Radim > Krčmář" <rkrc...@redhat.com>, "Martin > Schwidefsky" <schwidef...@de.ibm.com>, linux-kernel@vger.kernel.org, > linux-s...@vger.kernel.org, k...@vger.kernel.org > Sent: Wednesday, June 8, 2016 10:09:17 AM > Subject: Re: [PATCH] KVM: s390: fix build failure > > Hi Heiko, > > On Wed, 8 Jun 2016 07:31:29 +0200 Heiko Carstens <heiko.carst...@de.ibm.com> > wrote: > > > > On Wed, Jun 08, 2016 at 07:17:35AM +0200, Christian Borntraeger wrote: > > > On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > > > > etr_ptff definitions are moved and renamed but we missed updating them > > > > here and as a result s390 defconfig and allmodconfig was failing with > > > > the error: > > > > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > > > > > > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > > > > Signed-off-by: Sudip Mukherjee <sudip.mukher...@codethink.co.uk> > > > > > > Thank you for the report and patch. > > > > > > This is linux-next only. Its a conflict between my kvms390 queue and > > > Martins s390 queue. We cannot apply this directly as it would break > > > the build of my tree when not merged in next. (and it does not apply > > > on Martins tree). > > > > > > I will have a look how to fix that up. > > > > We could ask Stephen Rothwell to apply the patch only to linux-next? ;) > > Martin sent me the same patch earlier and I applied it to the merge of > the kvms390 tree today. Someone needs to remember to tell Linus when > the time comes. I can take care of that. On your side, the resolution will have to migrate to the KVM tree as soon as Christian sends me a pull request for 4.8. Thanks, Paolo
Re: [PATCH] KVM: s390: fix build failure
- Original Message - > From: "Stephen Rothwell" > To: "Heiko Carstens" > Cc: "Christian Borntraeger" , "Sudip Mukherjee" > , "Cornelia Huck" > , "Paolo Bonzini" , "Radim > Krčmář" , "Martin > Schwidefsky" , linux-kernel@vger.kernel.org, > linux-s...@vger.kernel.org, k...@vger.kernel.org > Sent: Wednesday, June 8, 2016 10:09:17 AM > Subject: Re: [PATCH] KVM: s390: fix build failure > > Hi Heiko, > > On Wed, 8 Jun 2016 07:31:29 +0200 Heiko Carstens > wrote: > > > > On Wed, Jun 08, 2016 at 07:17:35AM +0200, Christian Borntraeger wrote: > > > On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > > > > etr_ptff definitions are moved and renamed but we missed updating them > > > > here and as a result s390 defconfig and allmodconfig was failing with > > > > the error: > > > > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > > > > > > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > > > > Signed-off-by: Sudip Mukherjee > > > > > > Thank you for the report and patch. > > > > > > This is linux-next only. Its a conflict between my kvms390 queue and > > > Martins s390 queue. We cannot apply this directly as it would break > > > the build of my tree when not merged in next. (and it does not apply > > > on Martins tree). > > > > > > I will have a look how to fix that up. > > > > We could ask Stephen Rothwell to apply the patch only to linux-next? ;) > > Martin sent me the same patch earlier and I applied it to the merge of > the kvms390 tree today. Someone needs to remember to tell Linus when > the time comes. I can take care of that. On your side, the resolution will have to migrate to the KVM tree as soon as Christian sends me a pull request for 4.8. Thanks, Paolo
Re: [PATCH] KVM: s390: fix build failure
Hi Heiko, On Wed, 8 Jun 2016 07:31:29 +0200 Heiko Carstenswrote: > > On Wed, Jun 08, 2016 at 07:17:35AM +0200, Christian Borntraeger wrote: > > On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > > > etr_ptff definitions are moved and renamed but we missed updating them > > > here and as a result s390 defconfig and allmodconfig was failing with > > > the error: > > > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > > > > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > > > Signed-off-by: Sudip Mukherjee > > > > Thank you for the report and patch. > > > > This is linux-next only. Its a conflict between my kvms390 queue and > > Martins s390 queue. We cannot apply this directly as it would break > > the build of my tree when not merged in next. (and it does not apply > > on Martins tree). > > > > I will have a look how to fix that up. > > We could ask Stephen Rothwell to apply the patch only to linux-next? ;) Martin sent me the same patch earlier and I applied it to the merge of the kvms390 tree today. Someone needs to remember to tell Linus when the time comes. -- Cheers, Stephen Rothwell
Re: [PATCH] KVM: s390: fix build failure
Hi Heiko, On Wed, 8 Jun 2016 07:31:29 +0200 Heiko Carstens wrote: > > On Wed, Jun 08, 2016 at 07:17:35AM +0200, Christian Borntraeger wrote: > > On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > > > etr_ptff definitions are moved and renamed but we missed updating them > > > here and as a result s390 defconfig and allmodconfig was failing with > > > the error: > > > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > > > > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > > > Signed-off-by: Sudip Mukherjee > > > > Thank you for the report and patch. > > > > This is linux-next only. Its a conflict between my kvms390 queue and > > Martins s390 queue. We cannot apply this directly as it would break > > the build of my tree when not merged in next. (and it does not apply > > on Martins tree). > > > > I will have a look how to fix that up. > > We could ask Stephen Rothwell to apply the patch only to linux-next? ;) Martin sent me the same patch earlier and I applied it to the merge of the kvms390 tree today. Someone needs to remember to tell Linus when the time comes. -- Cheers, Stephen Rothwell
Re: [PATCH] KVM: s390: fix build failure
On Tue, 7 Jun 2016 22:49:30 +0100 Sudip Mukherjeewrote: > etr_ptff definitions are moved and renamed but we missed updating them > here and as a result s390 defconfig and allmodconfig was failing with > the error: > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > Signed-off-by: Sudip Mukherjee > --- > > s390 defconfig build log is at: > https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 > > arch/s390/kvm/kvm-s390.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c > index fa51aef..3039eaf 100644 > --- a/arch/s390/kvm/kvm-s390.c > +++ b/arch/s390/kvm/kvm-s390.c > @@ -29,7 +29,7 @@ > #include > #include > #include > -#include > +#include > #include > #include > #include > @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) > } > > if (test_facility(28)) /* TOD-clock steering */ > - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); > + ptff(kvm_s390_available_subfunc.ptff, > + sizeof(kvm_s390_available_subfunc.ptff), > + PTFF_QAF); > > if (test_facility(17)) { /* MSA */ > __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); Already send the exact same patch to Stephen yesterday. But thanks anyway. -- blue skies, Martin. "Reality continues to ruin my life." - Calvin.
Re: [PATCH] KVM: s390: fix build failure
On Tue, 7 Jun 2016 22:49:30 +0100 Sudip Mukherjee wrote: > etr_ptff definitions are moved and renamed but we missed updating them > here and as a result s390 defconfig and allmodconfig was failing with > the error: > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > Signed-off-by: Sudip Mukherjee > --- > > s390 defconfig build log is at: > https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 > > arch/s390/kvm/kvm-s390.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c > index fa51aef..3039eaf 100644 > --- a/arch/s390/kvm/kvm-s390.c > +++ b/arch/s390/kvm/kvm-s390.c > @@ -29,7 +29,7 @@ > #include > #include > #include > -#include > +#include > #include > #include > #include > @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) > } > > if (test_facility(28)) /* TOD-clock steering */ > - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); > + ptff(kvm_s390_available_subfunc.ptff, > + sizeof(kvm_s390_available_subfunc.ptff), > + PTFF_QAF); > > if (test_facility(17)) { /* MSA */ > __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); Already send the exact same patch to Stephen yesterday. But thanks anyway. -- blue skies, Martin. "Reality continues to ruin my life." - Calvin.
Re: [PATCH] KVM: s390: fix build failure
On Wed, Jun 08, 2016 at 07:17:35AM +0200, Christian Borntraeger wrote: > On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > > etr_ptff definitions are moved and renamed but we missed updating them > > here and as a result s390 defconfig and allmodconfig was failing with > > the error: > > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > > Signed-off-by: Sudip Mukherjee> > Thank you for the report and patch. > > This is linux-next only. Its a conflict between my kvms390 queue and > Martins s390 queue. We cannot apply this directly as it would break > the build of my tree when not merged in next. (and it does not apply > on Martins tree). > > I will have a look how to fix that up. We could ask Stephen Rothwell to apply the patch only to linux-next? ;) > > --- > > > > s390 defconfig build log is at: > > https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 > > > > arch/s390/kvm/kvm-s390.c | 6 -- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c > > index fa51aef..3039eaf 100644 > > --- a/arch/s390/kvm/kvm-s390.c > > +++ b/arch/s390/kvm/kvm-s390.c > > @@ -29,7 +29,7 @@ > > #include > > #include > > #include > > -#include > > +#include > > #include > > #include > > #include > > @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) > > } > > > > if (test_facility(28)) /* TOD-clock steering */ > > - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); > > + ptff(kvm_s390_available_subfunc.ptff, > > +sizeof(kvm_s390_available_subfunc.ptff), > > +PTFF_QAF); > > > > if (test_facility(17)) { /* MSA */ > > __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); > > >
Re: [PATCH] KVM: s390: fix build failure
On Wed, Jun 08, 2016 at 07:17:35AM +0200, Christian Borntraeger wrote: > On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > > etr_ptff definitions are moved and renamed but we missed updating them > > here and as a result s390 defconfig and allmodconfig was failing with > > the error: > > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > > Signed-off-by: Sudip Mukherjee > > Thank you for the report and patch. > > This is linux-next only. Its a conflict between my kvms390 queue and > Martins s390 queue. We cannot apply this directly as it would break > the build of my tree when not merged in next. (and it does not apply > on Martins tree). > > I will have a look how to fix that up. We could ask Stephen Rothwell to apply the patch only to linux-next? ;) > > --- > > > > s390 defconfig build log is at: > > https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 > > > > arch/s390/kvm/kvm-s390.c | 6 -- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c > > index fa51aef..3039eaf 100644 > > --- a/arch/s390/kvm/kvm-s390.c > > +++ b/arch/s390/kvm/kvm-s390.c > > @@ -29,7 +29,7 @@ > > #include > > #include > > #include > > -#include > > +#include > > #include > > #include > > #include > > @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) > > } > > > > if (test_facility(28)) /* TOD-clock steering */ > > - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); > > + ptff(kvm_s390_available_subfunc.ptff, > > +sizeof(kvm_s390_available_subfunc.ptff), > > +PTFF_QAF); > > > > if (test_facility(17)) { /* MSA */ > > __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); > > >
Re: [PATCH] KVM: s390: fix build failure
On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > etr_ptff definitions are moved and renamed but we missed updating them > here and as a result s390 defconfig and allmodconfig was failing with > the error: > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > Signed-off-by: Sudip MukherjeeThank you for the report and patch. This is linux-next only. Its a conflict between my kvms390 queue and Martins s390 queue. We cannot apply this directly as it would break the build of my tree when not merged in next. (and it does not apply on Martins tree). I will have a look how to fix that up. > --- > > s390 defconfig build log is at: > https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 > > arch/s390/kvm/kvm-s390.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c > index fa51aef..3039eaf 100644 > --- a/arch/s390/kvm/kvm-s390.c > +++ b/arch/s390/kvm/kvm-s390.c > @@ -29,7 +29,7 @@ > #include > #include > #include > -#include > +#include > #include > #include > #include > @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) > } > > if (test_facility(28)) /* TOD-clock steering */ > - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); > + ptff(kvm_s390_available_subfunc.ptff, > + sizeof(kvm_s390_available_subfunc.ptff), > + PTFF_QAF); > > if (test_facility(17)) { /* MSA */ > __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); >
Re: [PATCH] KVM: s390: fix build failure
On 06/07/2016 11:49 PM, Sudip Mukherjee wrote: > etr_ptff definitions are moved and renamed but we missed updating them > here and as a result s390 defconfig and allmodconfig was failing with > the error: > arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared > > Fixes: cc8f94656487 ("s390/time: move PTFF definitions") > Signed-off-by: Sudip Mukherjee Thank you for the report and patch. This is linux-next only. Its a conflict between my kvms390 queue and Martins s390 queue. We cannot apply this directly as it would break the build of my tree when not merged in next. (and it does not apply on Martins tree). I will have a look how to fix that up. > --- > > s390 defconfig build log is at: > https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 > > arch/s390/kvm/kvm-s390.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c > index fa51aef..3039eaf 100644 > --- a/arch/s390/kvm/kvm-s390.c > +++ b/arch/s390/kvm/kvm-s390.c > @@ -29,7 +29,7 @@ > #include > #include > #include > -#include > +#include > #include > #include > #include > @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) > } > > if (test_facility(28)) /* TOD-clock steering */ > - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); > + ptff(kvm_s390_available_subfunc.ptff, > + sizeof(kvm_s390_available_subfunc.ptff), > + PTFF_QAF); > > if (test_facility(17)) { /* MSA */ > __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); >
[PATCH] KVM: s390: fix build failure
etr_ptff definitions are moved and renamed but we missed updating them here and as a result s390 defconfig and allmodconfig was failing with the error: arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared Fixes: cc8f94656487 ("s390/time: move PTFF definitions") Signed-off-by: Sudip Mukherjee--- s390 defconfig build log is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 arch/s390/kvm/kvm-s390.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index fa51aef..3039eaf 100644 --- a/arch/s390/kvm/kvm-s390.c +++ b/arch/s390/kvm/kvm-s390.c @@ -29,7 +29,7 @@ #include #include #include -#include +#include #include #include #include @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) } if (test_facility(28)) /* TOD-clock steering */ - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); + ptff(kvm_s390_available_subfunc.ptff, +sizeof(kvm_s390_available_subfunc.ptff), +PTFF_QAF); if (test_facility(17)) { /* MSA */ __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); -- 1.9.1
[PATCH] KVM: s390: fix build failure
etr_ptff definitions are moved and renamed but we missed updating them here and as a result s390 defconfig and allmodconfig was failing with the error: arch/s390/kvm/kvm-s390.c:230:45: error: 'ETR_PTFF_QAF' undeclared Fixes: cc8f94656487 ("s390/time: move PTFF definitions") Signed-off-by: Sudip Mukherjee --- s390 defconfig build log is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/135776067 arch/s390/kvm/kvm-s390.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index fa51aef..3039eaf 100644 --- a/arch/s390/kvm/kvm-s390.c +++ b/arch/s390/kvm/kvm-s390.c @@ -29,7 +29,7 @@ #include #include #include -#include +#include #include #include #include @@ -227,7 +227,9 @@ static void kvm_s390_cpu_feat_init(void) } if (test_facility(28)) /* TOD-clock steering */ - etr_ptff(kvm_s390_available_subfunc.ptff, ETR_PTFF_QAF); + ptff(kvm_s390_available_subfunc.ptff, +sizeof(kvm_s390_available_subfunc.ptff), +PTFF_QAF); if (test_facility(17)) { /* MSA */ __cpacf_query(CPACF_KMAC, kvm_s390_available_subfunc.kmac); -- 1.9.1