virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
* [PULL] vhost: tcm_vhost fixes for 3.9
@ 2013-03-18 11:20 Michael S. Tsirkin
  2013-03-18 17:01 ` David Miller
  0 siblings, 1 reply; 7+ messages in thread
From: Michael S. Tsirkin @ 2013-03-18 11:20 UTC (permalink / raw)
  To: David Miller
  Cc: kvm, virtualization, netdev, linux-kernel, asias, mst, nab,
	target-devel

The following changes since commit 8c6216d7f118a128678270824b6a1286a63863ca:

  Revert "ip_gre: make ipgre_tunnel_xmit() not parse network header as IP unconditionally" (2013-03-16 23:00:41 -0400)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost-net

for you to fetch changes up to deb7cb067dabd3be625eaf5495da8bdc97377fc1:

  tcm_vhost: Flush vhost_work in vhost_scsi_flush() (2013-03-17 13:04:14 +0200)

----------------------------------------------------------------
Asias He (2):
      tcm_vhost: Add missed lock in vhost_scsi_clear_endpoint()
      tcm_vhost: Flush vhost_work in vhost_scsi_flush()

 drivers/vhost/tcm_vhost.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PULL] vhost: tcm_vhost fixes for 3.9
  2013-03-18 11:20 [PULL] vhost: tcm_vhost fixes for 3.9 Michael S. Tsirkin
@ 2013-03-18 17:01 ` David Miller
  2013-03-18 19:54   ` Michael S. Tsirkin
  0 siblings, 1 reply; 7+ messages in thread
From: David Miller @ 2013-03-18 17:01 UTC (permalink / raw)
  To: mst; +Cc: kvm, virtualization, netdev, linux-kernel, asias, nab,
	target-devel

From: "Michael S. Tsirkin" <mst@redhat.com>
Date: Mon, 18 Mar 2013 13:20:03 +0200

> The following changes since commit 8c6216d7f118a128678270824b6a1286a63863ca:
> 
>   Revert "ip_gre: make ipgre_tunnel_xmit() not parse network header as IP unconditionally" (2013-03-16 23:00:41 -0400)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost-net
> 
> for you to fetch changes up to deb7cb067dabd3be625eaf5495da8bdc97377fc1:
> 
>   tcm_vhost: Flush vhost_work in vhost_scsi_flush() (2013-03-17 13:04:14 +0200)

This is a scsi driver, I therefore don't think this pull request is for me.

Please avoid such confusion in the future, and don't use branch names
like "vhost-net" for SCSI driver fixes.

Thanks.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PULL] vhost: tcm_vhost fixes for 3.9
  2013-03-18 17:01 ` David Miller
@ 2013-03-18 19:54   ` Michael S. Tsirkin
  2013-03-18 20:05     ` David Miller
  2013-03-18 21:10     ` Nicholas A. Bellinger
  0 siblings, 2 replies; 7+ messages in thread
From: Michael S. Tsirkin @ 2013-03-18 19:54 UTC (permalink / raw)
  To: David Miller
  Cc: kvm, virtualization, netdev, linux-kernel, asias, nab,
	target-devel

On Mon, Mar 18, 2013 at 01:01:36PM -0400, David Miller wrote:
> From: "Michael S. Tsirkin" <mst@redhat.com>
> Date: Mon, 18 Mar 2013 13:20:03 +0200
> 
> > The following changes since commit 8c6216d7f118a128678270824b6a1286a63863ca:
> > 
> >   Revert "ip_gre: make ipgre_tunnel_xmit() not parse network header as IP unconditionally" (2013-03-16 23:00:41 -0400)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost-net
> > 
> > for you to fetch changes up to deb7cb067dabd3be625eaf5495da8bdc97377fc1:
> > 
> >   tcm_vhost: Flush vhost_work in vhost_scsi_flush() (2013-03-17 13:04:14 +0200)
> 
> This is a scsi driver, I therefore don't think this pull request is for me.
> 
> Please avoid such confusion in the future, and don't use branch names
> like "vhost-net" for SCSI driver fixes.
> 
> Thanks.

We are sharing quite a bit of code, so it's a common tree.
There was a vhost-net patch there too but you picked it up
directly so I dropped that and what's left is vhost-scsi.
Typically vhost-net dominates so I'm just used to merging all vhost
things through you but if you think it's wrong I can send it directly to
Linus.

-- 
MST

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PULL] vhost: tcm_vhost fixes for 3.9
  2013-03-18 19:54   ` Michael S. Tsirkin
@ 2013-03-18 20:05     ` David Miller
  2013-03-18 21:10     ` Nicholas A. Bellinger
  1 sibling, 0 replies; 7+ messages in thread
From: David Miller @ 2013-03-18 20:05 UTC (permalink / raw)
  To: mst; +Cc: kvm, virtualization, netdev, linux-kernel, asias, nab,
	target-devel

From: "Michael S. Tsirkin" <mst@redhat.com>
Date: Mon, 18 Mar 2013 21:54:43 +0200

> On Mon, Mar 18, 2013 at 01:01:36PM -0400, David Miller wrote:
>> From: "Michael S. Tsirkin" <mst@redhat.com>
>> Date: Mon, 18 Mar 2013 13:20:03 +0200
>> 
>> > The following changes since commit 8c6216d7f118a128678270824b6a1286a63863ca:
>> > 
>> >   Revert "ip_gre: make ipgre_tunnel_xmit() not parse network header as IP unconditionally" (2013-03-16 23:00:41 -0400)
>> > 
>> > are available in the git repository at:
>> > 
>> >   git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost-net
>> > 
>> > for you to fetch changes up to deb7cb067dabd3be625eaf5495da8bdc97377fc1:
>> > 
>> >   tcm_vhost: Flush vhost_work in vhost_scsi_flush() (2013-03-17 13:04:14 +0200)
>> 
>> This is a scsi driver, I therefore don't think this pull request is for me.
>> 
>> Please avoid such confusion in the future, and don't use branch names
>> like "vhost-net" for SCSI driver fixes.
>> 
>> Thanks.
> 
> We are sharing quite a bit of code, so it's a common tree.
> There was a vhost-net patch there too but you picked it up
> directly so I dropped that and what's left is vhost-scsi.
> Typically vhost-net dominates so I'm just used to merging all vhost
> things through you but if you think it's wrong I can send it directly to
> Linus.

Just like any other "bus" or "layer" for drivers, you should
really just send the networking bits my way.

Now, in situations where some generic vhost infrastructure is
necessary for a vhost-net specific change, that might be a
situation where we can merge the infrastructure change into
my tree so that the networking bits can go in too.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PULL] vhost: tcm_vhost fixes for 3.9
  2013-03-18 19:54   ` Michael S. Tsirkin
  2013-03-18 20:05     ` David Miller
@ 2013-03-18 21:10     ` Nicholas A. Bellinger
  2013-03-18 21:14       ` Michael S. Tsirkin
  1 sibling, 1 reply; 7+ messages in thread
From: Nicholas A. Bellinger @ 2013-03-18 21:10 UTC (permalink / raw)
  To: Michael S. Tsirkin
  Cc: David Miller, kvm, virtualization, netdev, linux-kernel, asias,
	target-devel

On Mon, 2013-03-18 at 21:54 +0200, Michael S. Tsirkin wrote:
> On Mon, Mar 18, 2013 at 01:01:36PM -0400, David Miller wrote:
> > From: "Michael S. Tsirkin" <mst@redhat.com>
> > Date: Mon, 18 Mar 2013 13:20:03 +0200
> > 
> > > The following changes since commit 8c6216d7f118a128678270824b6a1286a63863ca:
> > > 
> > >   Revert "ip_gre: make ipgre_tunnel_xmit() not parse network header as IP unconditionally" (2013-03-16 23:00:41 -0400)
> > > 
> > > are available in the git repository at:
> > > 
> > >   git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost-net
> > > 
> > > for you to fetch changes up to deb7cb067dabd3be625eaf5495da8bdc97377fc1:
> > > 
> > >   tcm_vhost: Flush vhost_work in vhost_scsi_flush() (2013-03-17 13:04:14 +0200)
> > 
> > This is a scsi driver, I therefore don't think this pull request is for me.
> > 
> > Please avoid such confusion in the future, and don't use branch names
> > like "vhost-net" for SCSI driver fixes.
> > 
> > Thanks.
> 
> We are sharing quite a bit of code, so it's a common tree.
> There was a vhost-net patch there too but you picked it up
> directly so I dropped that and what's left is vhost-scsi.
> Typically vhost-net dominates so I'm just used to merging all vhost
> things through you but if you think it's wrong I can send it directly to
> Linus.
> 

I'll be sending out a target-pending 3.9-rc-fixes pull request later in
the week, and am happy to include this series if you'd like.

--nab

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PULL] vhost: tcm_vhost fixes for 3.9
  2013-03-18 21:10     ` Nicholas A. Bellinger
@ 2013-03-18 21:14       ` Michael S. Tsirkin
  2013-03-18 21:36         ` Nicholas A. Bellinger
  0 siblings, 1 reply; 7+ messages in thread
From: Michael S. Tsirkin @ 2013-03-18 21:14 UTC (permalink / raw)
  To: Nicholas A. Bellinger
  Cc: David Miller, kvm, virtualization, netdev, linux-kernel, asias,
	target-devel

On Mon, Mar 18, 2013 at 02:10:03PM -0700, Nicholas A. Bellinger wrote:
> On Mon, 2013-03-18 at 21:54 +0200, Michael S. Tsirkin wrote:
> > On Mon, Mar 18, 2013 at 01:01:36PM -0400, David Miller wrote:
> > > From: "Michael S. Tsirkin" <mst@redhat.com>
> > > Date: Mon, 18 Mar 2013 13:20:03 +0200
> > > 
> > > > The following changes since commit 8c6216d7f118a128678270824b6a1286a63863ca:
> > > > 
> > > >   Revert "ip_gre: make ipgre_tunnel_xmit() not parse network header as IP unconditionally" (2013-03-16 23:00:41 -0400)
> > > > 
> > > > are available in the git repository at:
> > > > 
> > > >   git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost-net
> > > > 
> > > > for you to fetch changes up to deb7cb067dabd3be625eaf5495da8bdc97377fc1:
> > > > 
> > > >   tcm_vhost: Flush vhost_work in vhost_scsi_flush() (2013-03-17 13:04:14 +0200)
> > > 
> > > This is a scsi driver, I therefore don't think this pull request is for me.
> > > 
> > > Please avoid such confusion in the future, and don't use branch names
> > > like "vhost-net" for SCSI driver fixes.
> > > 
> > > Thanks.
> > 
> > We are sharing quite a bit of code, so it's a common tree.
> > There was a vhost-net patch there too but you picked it up
> > directly so I dropped that and what's left is vhost-scsi.
> > Typically vhost-net dominates so I'm just used to merging all vhost
> > things through you but if you think it's wrong I can send it directly to
> > Linus.
> > 
> 
> I'll be sending out a target-pending 3.9-rc-fixes pull request later in
> the week, and am happy to include this series if you'd like.
> 
> --nab

Sure, go ahead. You can add
Acked-by: Michael S. Tsirkin <mst@redhat.com>
to the patches.

-- 
MST

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PULL] vhost: tcm_vhost fixes for 3.9
  2013-03-18 21:14       ` Michael S. Tsirkin
@ 2013-03-18 21:36         ` Nicholas A. Bellinger
  0 siblings, 0 replies; 7+ messages in thread
From: Nicholas A. Bellinger @ 2013-03-18 21:36 UTC (permalink / raw)
  To: Michael S. Tsirkin
  Cc: David Miller, kvm, virtualization, netdev, linux-kernel, asias,
	target-devel

On Mon, 2013-03-18 at 23:14 +0200, Michael S. Tsirkin wrote:
> On Mon, Mar 18, 2013 at 02:10:03PM -0700, Nicholas A. Bellinger wrote:
> > On Mon, 2013-03-18 at 21:54 +0200, Michael S. Tsirkin wrote:
> > > On Mon, Mar 18, 2013 at 01:01:36PM -0400, David Miller wrote:
> > > > From: "Michael S. Tsirkin" <mst@redhat.com>
> > > > Date: Mon, 18 Mar 2013 13:20:03 +0200
> > > > 
> > > > > The following changes since commit 8c6216d7f118a128678270824b6a1286a63863ca:
> > > > > 
> > > > >   Revert "ip_gre: make ipgre_tunnel_xmit() not parse network header as IP unconditionally" (2013-03-16 23:00:41 -0400)
> > > > > 
> > > > > are available in the git repository at:
> > > > > 
> > > > >   git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost-net
> > > > > 
> > > > > for you to fetch changes up to deb7cb067dabd3be625eaf5495da8bdc97377fc1:
> > > > > 
> > > > >   tcm_vhost: Flush vhost_work in vhost_scsi_flush() (2013-03-17 13:04:14 +0200)
> > > > 
> > > > This is a scsi driver, I therefore don't think this pull request is for me.
> > > > 
> > > > Please avoid such confusion in the future, and don't use branch names
> > > > like "vhost-net" for SCSI driver fixes.
> > > > 
> > > > Thanks.
> > > 
> > > We are sharing quite a bit of code, so it's a common tree.
> > > There was a vhost-net patch there too but you picked it up
> > > directly so I dropped that and what's left is vhost-scsi.
> > > Typically vhost-net dominates so I'm just used to merging all vhost
> > > things through you but if you think it's wrong I can send it directly to
> > > Linus.
> > > 
> > 
> > I'll be sending out a target-pending 3.9-rc-fixes pull request later in
> > the week, and am happy to include this series if you'd like.
> > 
> > --nab
> 
> Sure, go ahead. You can add
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> to the patches.
> 

Applied to target-pending/master.

Thanks MST & Asias!

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2013-03-18 21:36 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-18 11:20 [PULL] vhost: tcm_vhost fixes for 3.9 Michael S. Tsirkin
2013-03-18 17:01 ` David Miller
2013-03-18 19:54   ` Michael S. Tsirkin
2013-03-18 20:05     ` David Miller
2013-03-18 21:10     ` Nicholas A. Bellinger
2013-03-18 21:14       ` Michael S. Tsirkin
2013-03-18 21:36         ` Nicholas A. Bellinger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).