mbox

[PULL,0/3] vfio-ccw: some fixes

Message ID 20210520113450.267893-1-cohuck@redhat.com (mailing list archive)
State New, archived
Headers show

Pull-request

https://git.kernel.org/pub/scm/linux/kernel/git/kvms390/vfio-ccw.git tags/vfio-ccw-20210520

Message

Cornelia Huck May 20, 2021, 11:34 a.m. UTC
The following changes since commit 6efb943b8616ec53a5e444193dccf1af9ad627b5:

  Linux 5.13-rc1 (2021-05-09 14:17:44 -0700)

are available in the Git repository at:

  https://git.kernel.org/pub/scm/linux/kernel/git/kvms390/vfio-ccw.git tags/vfio-ccw-20210520

for you to fetch changes up to 2af7a834a435460d546f0cf0a8b8e4d259f1d910:

  vfio-ccw: Serialize FSM IDLE state with I/O completion (2021-05-12 12:59:50 +0200)

----------------------------------------------------------------
Avoid some races in vfio-ccw request handling.

----------------------------------------------------------------

Eric Farman (3):
  vfio-ccw: Check initialized flag in cp_init()
  vfio-ccw: Reset FSM state to IDLE inside FSM
  vfio-ccw: Serialize FSM IDLE state with I/O completion

 drivers/s390/cio/vfio_ccw_cp.c  |  4 ++++
 drivers/s390/cio/vfio_ccw_drv.c | 12 ++++++++++--
 drivers/s390/cio/vfio_ccw_fsm.c |  1 +
 drivers/s390/cio/vfio_ccw_ops.c |  2 --
 4 files changed, 15 insertions(+), 4 deletions(-)

Comments

Vasily Gorbik May 26, 2021, 10:33 p.m. UTC | #1
On Thu, May 20, 2021 at 01:34:47PM +0200, Cornelia Huck wrote:
> The following changes since commit 6efb943b8616ec53a5e444193dccf1af9ad627b5:
> 
>   Linux 5.13-rc1 (2021-05-09 14:17:44 -0700)
> 
> are available in the Git repository at:
> 
>   https://git.kernel.org/pub/scm/linux/kernel/git/kvms390/vfio-ccw.git tags/vfio-ccw-20210520
> 
> for you to fetch changes up to 2af7a834a435460d546f0cf0a8b8e4d259f1d910:
> 
>   vfio-ccw: Serialize FSM IDLE state with I/O completion (2021-05-12 12:59:50 +0200)
> 
> ----------------------------------------------------------------
> Avoid some races in vfio-ccw request handling.
> 
> ----------------------------------------------------------------
> 
> Eric Farman (3):
>   vfio-ccw: Check initialized flag in cp_init()
>   vfio-ccw: Reset FSM state to IDLE inside FSM
>   vfio-ccw: Serialize FSM IDLE state with I/O completion

Pulled into fixes, thanks.

BTW, linux-s390@vger.kernel.org is now archived on lore and we started
using b4 (https://git.kernel.org/pub/scm/utils/b4/b4.git) to pick up
changes. Besides all other features, it can convert Message-Id: to Link:

Hm, and b4 also now complains:
✗ BADSIG: DKIM/ibm.com
have to look into that...
Cornelia Huck May 27, 2021, 6:33 a.m. UTC | #2
On Thu, 27 May 2021 00:33:00 +0200
Vasily Gorbik <gor@linux.ibm.com> wrote:

> On Thu, May 20, 2021 at 01:34:47PM +0200, Cornelia Huck wrote:
> > The following changes since commit 6efb943b8616ec53a5e444193dccf1af9ad627b5:
> > 
> >   Linux 5.13-rc1 (2021-05-09 14:17:44 -0700)
> > 
> > are available in the Git repository at:
> > 
> >   https://git.kernel.org/pub/scm/linux/kernel/git/kvms390/vfio-ccw.git tags/vfio-ccw-20210520
> > 
> > for you to fetch changes up to 2af7a834a435460d546f0cf0a8b8e4d259f1d910:
> > 
> >   vfio-ccw: Serialize FSM IDLE state with I/O completion (2021-05-12 12:59:50 +0200)
> > 
> > ----------------------------------------------------------------
> > Avoid some races in vfio-ccw request handling.
> > 
> > ----------------------------------------------------------------
> > 
> > Eric Farman (3):
> >   vfio-ccw: Check initialized flag in cp_init()
> >   vfio-ccw: Reset FSM state to IDLE inside FSM
> >   vfio-ccw: Serialize FSM IDLE state with I/O completion  
> 
> Pulled into fixes, thanks.
> 
> BTW, linux-s390@vger.kernel.org is now archived on lore and we started

Oh, nice.

> using b4 (https://git.kernel.org/pub/scm/utils/b4/b4.git) to pick up
> changes. Besides all other features, it can convert Message-Id: to Link:

I've been using b4 to pick patches (Linux and especially QEMU) for
quite some time now, but never felt the need to convert Message-Id: to
Link:. If you prefer the Link: format, I can certainly start using that
for kernel patches.

> 
> Hm, and b4 also now complains:
> ✗ BADSIG: DKIM/ibm.com
> have to look into that...
>
Vasily Gorbik May 27, 2021, 3:25 p.m. UTC | #3
> > BTW, linux-s390@vger.kernel.org is now archived on lore and we started
> 
> Oh, nice.
> 
> > using b4 (https://git.kernel.org/pub/scm/utils/b4/b4.git) to pick up
> > changes. Besides all other features, it can convert Message-Id: to Link:
> 
> I've been using b4 to pick patches (Linux and especially QEMU) for
> quite some time now, but never felt the need to convert Message-Id: to
> Link:. If you prefer the Link: format, I can certainly start using that
> for kernel patches.

It's up to you, whatever you prefer. Just wanted to point out that this is
possible now. I personally just follow what seems to be a more common practice.

$ git log --grep=Link:.*lore --oneline linux/master | wc -l
53522
$ git log --grep=Message-Id: --oneline linux/master | wc -l
1666


> > 
> > Hm, and b4 also now complains:
> > ✗ BADSIG: DKIM/ibm.com
> > have to look into that...

Ok, figured out that this is only failing with the company vpn. Internal
dns servers handle ibm.com domain, but fail to serve public key via
"pp1._domainkey.ibm.com" request, which is required to verify DKIM.
All works fine with external dns.
Cornelia Huck May 27, 2021, 4 p.m. UTC | #4
On Thu, 27 May 2021 17:25:50 +0200
Vasily Gorbik <gor@linux.ibm.com> wrote:

> > > BTW, linux-s390@vger.kernel.org is now archived on lore and we started  
> > 
> > Oh, nice.
> >   
> > > using b4 (https://git.kernel.org/pub/scm/utils/b4/b4.git) to pick up
> > > changes. Besides all other features, it can convert Message-Id: to Link:  
> > 
> > I've been using b4 to pick patches (Linux and especially QEMU) for
> > quite some time now, but never felt the need to convert Message-Id: to
> > Link:. If you prefer the Link: format, I can certainly start using that
> > for kernel patches.  
> 
> It's up to you, whatever you prefer. Just wanted to point out that this is
> possible now. I personally just follow what seems to be a more common practice.
> 
> $ git log --grep=Link:.*lore --oneline linux/master | wc -l
> 53522
> $ git log --grep=Message-Id: --oneline linux/master | wc -l
> 1666

For QEMU, it's the other way around :)

But I'll try to use Link: for kernel patches, unless I forget.