Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Martin Schwidefsky
On Wed, 26 Jul 2017 11:40:14 +0200 Cornelia Huck wrote: > On Wed, 26 Jul 2017 07:40:44 +0200 > Martin Schwidefsky wrote: > > > On Wed, 26 Jul 2017 07:00:33 +0200 > > Heiko Carstens wrote: > > > > > On Tue, Jul 25, 2017

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Martin Schwidefsky
On Wed, 26 Jul 2017 11:40:14 +0200 Cornelia Huck wrote: > On Wed, 26 Jul 2017 07:40:44 +0200 > Martin Schwidefsky wrote: > > > On Wed, 26 Jul 2017 07:00:33 +0200 > > Heiko Carstens wrote: > > > > > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > > > Hi Martin, > > >

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Cornelia Huck
On Wed, 26 Jul 2017 07:40:44 +0200 Martin Schwidefsky wrote: > On Wed, 26 Jul 2017 07:00:33 +0200 > Heiko Carstens wrote: > > > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > > Hi Martin, > > > > > > my s390 qemu tests

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Cornelia Huck
On Wed, 26 Jul 2017 07:40:44 +0200 Martin Schwidefsky wrote: > On Wed, 26 Jul 2017 07:00:33 +0200 > Heiko Carstens wrote: > > > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > > Hi Martin, > > > > > > my s390 qemu tests in linux-next stopped working a few days ago. > > >

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Heiko Carstens
> > > Hmm, I though that NIAI is a NOP on older machines. A runtime check for > > > the facility bit is out of the question as the NIAI-7 gets inlined in > > > the spin_unlock code. So yes, the only available fix is to make the > > > NIAI hinting conditional on zEC12. Which is quite ugly as we

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Heiko Carstens
> > > Hmm, I though that NIAI is a NOP on older machines. A runtime check for > > > the facility bit is out of the question as the NIAI-7 gets inlined in > > > the spin_unlock code. So yes, the only available fix is to make the > > > NIAI hinting conditional on zEC12. Which is quite ugly as we

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Martin Schwidefsky
On Wed, 26 Jul 2017 08:05:27 +0200 Heiko Carstens wrote: > On Wed, Jul 26, 2017 at 07:40:44AM +0200, Martin Schwidefsky wrote: > > On Wed, 26 Jul 2017 07:00:33 +0200 > > Heiko Carstens wrote: > > > > > On Tue, Jul 25, 2017 at 06:14:51PM

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Martin Schwidefsky
On Wed, 26 Jul 2017 08:05:27 +0200 Heiko Carstens wrote: > On Wed, Jul 26, 2017 at 07:40:44AM +0200, Martin Schwidefsky wrote: > > On Wed, 26 Jul 2017 07:00:33 +0200 > > Heiko Carstens wrote: > > > > > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > > > Hi Martin, > > >

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Heiko Carstens
On Wed, Jul 26, 2017 at 07:40:44AM +0200, Martin Schwidefsky wrote: > On Wed, 26 Jul 2017 07:00:33 +0200 > Heiko Carstens wrote: > > > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > > Hi Martin, > > > > > > my s390 qemu tests in linux-next stopped

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-26 Thread Heiko Carstens
On Wed, Jul 26, 2017 at 07:40:44AM +0200, Martin Schwidefsky wrote: > On Wed, 26 Jul 2017 07:00:33 +0200 > Heiko Carstens wrote: > > > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > > Hi Martin, > > > > > > my s390 qemu tests in linux-next stopped working a few days ago. > >

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-25 Thread Martin Schwidefsky
On Wed, 26 Jul 2017 07:00:33 +0200 Heiko Carstens wrote: > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > Hi Martin, > > > > my s390 qemu tests in linux-next stopped working a few days ago. > > Bisect points to commit 's390/spinlock: add niai

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-25 Thread Martin Schwidefsky
On Wed, 26 Jul 2017 07:00:33 +0200 Heiko Carstens wrote: > On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > > Hi Martin, > > > > my s390 qemu tests in linux-next stopped working a few days ago. > > Bisect points to commit 's390/spinlock: add niai spinlock hints'. > > > >

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-25 Thread Heiko Carstens
On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > Hi Martin, > > my s390 qemu tests in linux-next stopped working a few days ago. > Bisect points to commit 's390/spinlock: add niai spinlock hints'. > > Looking at the patch, this isn't really surprising; at least to me it looks >

Re: Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-25 Thread Heiko Carstens
On Tue, Jul 25, 2017 at 06:14:51PM -0700, Guenter Roeck wrote: > Hi Martin, > > my s390 qemu tests in linux-next stopped working a few days ago. > Bisect points to commit 's390/spinlock: add niai spinlock hints'. > > Looking at the patch, this isn't really surprising; at least to me it looks >

Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-25 Thread Guenter Roeck
Hi Martin, my s390 qemu tests in linux-next stopped working a few days ago. Bisect points to commit 's390/spinlock: add niai spinlock hints'. Looking at the patch, this isn't really surprising; at least to me it looks like the patch is making instructions mandatory which are only available in

Qemu problems in -next with 's390/spinlock: add niai spinlock hints'

2017-07-25 Thread Guenter Roeck
Hi Martin, my s390 qemu tests in linux-next stopped working a few days ago. Bisect points to commit 's390/spinlock: add niai spinlock hints'. Looking at the patch, this isn't really surprising; at least to me it looks like the patch is making instructions mandatory which are only available in