Re: [PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-09 Thread Michael S. Tsirkin
On Mon, Sep 09, 2019 at 03:18:01PM +0800, Jason Wang wrote:
> 
> On 2019/9/6 下午9:15, David Miller wrote:
> > From: Jason Wang 
> > Date: Fri, 6 Sep 2019 18:02:35 +0800
> > 
> > > On 2019/9/5 下午9:59, Jason Gunthorpe wrote:
> > > > I think you should apply the revert this cycle and rebase the other
> > > > patch for next..
> > > > 
> > > > Jason
> > > Yes, the plan is to revert in this release cycle.
> > Then you should reset patch #1 all by itself targetting 'net'.
> 
> 
> Thanks for the reminding. I want the patch to go through Michael's vhost
> tree, that's why I don't put 'net' prefix. For next time, maybe I can use
> "vhost" as a prefix for classification?

That's fine by me.

-- 
MST
___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Re: [PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-09 Thread Jason Wang



On 2019/9/6 下午9:15, David Miller wrote:

From: Jason Wang 
Date: Fri, 6 Sep 2019 18:02:35 +0800


On 2019/9/5 下午9:59, Jason Gunthorpe wrote:

I think you should apply the revert this cycle and rebase the other
patch for next..

Jason

Yes, the plan is to revert in this release cycle.

Then you should reset patch #1 all by itself targetting 'net'.



Thanks for the reminding. I want the patch to go through Michael's vhost  
tree, that's why I don't put 'net' prefix. For next time, maybe I can  
use "vhost" as a prefix for classification?


___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization


Re: [PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-08 Thread Jason Wang


On 2019/9/7 下午11:03, Jason Gunthorpe wrote:

On Fri, Sep 06, 2019 at 06:02:35PM +0800, Jason Wang wrote:

On 2019/9/5 下午9:59, Jason Gunthorpe wrote:

On Thu, Sep 05, 2019 at 08:27:34PM +0800, Jason Wang wrote:

Hi:

Per request from Michael and Jason, the metadata accelreation is
reverted in this version and rework in next version.

Please review.

Thanks

Jason Wang (2):
Revert "vhost: access vq metadata through kernel virtual address"
vhost: re-introducing metadata acceleration through kernel virtual
  address

There are a bunch of patches in the queue already that will help
vhost, and I a working on one for next cycle that will help alot more
too.


I will check those patches, but if you can give me some pointers or keywords
it would be much appreciated.

You can look here:

https://github.com/jgunthorpe/linux/commits/mmu_notifier

The first parts, the get/put are in the hmm tree, and the last part,
the interval tree in the last commit is still a WIP, but it would
remove alot of that code from vhost as well.

Jason



Thanks a lot, will have a look at these and come back if I met any issues.

___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Re: [PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-07 Thread Jason Gunthorpe
On Fri, Sep 06, 2019 at 06:02:35PM +0800, Jason Wang wrote:
> 
> On 2019/9/5 下午9:59, Jason Gunthorpe wrote:
> > On Thu, Sep 05, 2019 at 08:27:34PM +0800, Jason Wang wrote:
> > > Hi:
> > > 
> > > Per request from Michael and Jason, the metadata accelreation is
> > > reverted in this version and rework in next version.
> > > 
> > > Please review.
> > > 
> > > Thanks
> > > 
> > > Jason Wang (2):
> > >Revert "vhost: access vq metadata through kernel virtual address"
> > >vhost: re-introducing metadata acceleration through kernel virtual
> > >  address
> > There are a bunch of patches in the queue already that will help
> > vhost, and I a working on one for next cycle that will help alot more
> > too.
> 
> 
> I will check those patches, but if you can give me some pointers or keywords
> it would be much appreciated.

You can look here:

https://github.com/jgunthorpe/linux/commits/mmu_notifier

The first parts, the get/put are in the hmm tree, and the last part,
the interval tree in the last commit is still a WIP, but it would
remove alot of that code from vhost as well.

Jason
___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Re: [PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-06 Thread David Miller
From: Jason Wang 
Date: Fri, 6 Sep 2019 18:02:35 +0800

> On 2019/9/5 下午9:59, Jason Gunthorpe wrote:
>> I think you should apply the revert this cycle and rebase the other
>> patch for next..
>>
>> Jason
> 
> Yes, the plan is to revert in this release cycle.

Then you should reset patch #1 all by itself targetting 'net'.
___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization


Re: [PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-06 Thread Jason Wang


On 2019/9/5 下午9:59, Jason Gunthorpe wrote:

On Thu, Sep 05, 2019 at 08:27:34PM +0800, Jason Wang wrote:

Hi:

Per request from Michael and Jason, the metadata accelreation is
reverted in this version and rework in next version.

Please review.

Thanks

Jason Wang (2):
   Revert "vhost: access vq metadata through kernel virtual address"
   vhost: re-introducing metadata acceleration through kernel virtual
 address

There are a bunch of patches in the queue already that will help
vhost, and I a working on one for next cycle that will help alot more
too.



I will check those patches, but if you can give me some pointers or 
keywords it would be much appreciated.





I think you should apply the revert this cycle and rebase the other
patch for next..

Jason



Yes, the plan is to revert in this release cycle.

Thanks

___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Re: [PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-05 Thread Jason Gunthorpe
On Thu, Sep 05, 2019 at 08:27:34PM +0800, Jason Wang wrote:
> Hi:
> 
> Per request from Michael and Jason, the metadata accelreation is
> reverted in this version and rework in next version.
> 
> Please review.
> 
> Thanks
> 
> Jason Wang (2):
>   Revert "vhost: access vq metadata through kernel virtual address"
>   vhost: re-introducing metadata acceleration through kernel virtual
> address

There are a bunch of patches in the queue already that will help
vhost, and I a working on one for next cycle that will help alot more
too.

I think you should apply the revert this cycle and rebase the other
patch for next..

Jason
___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization


[PATCH 0/2] Revert and rework on the metadata accelreation

2019-09-05 Thread Jason Wang
Hi:

Per request from Michael and Jason, the metadata accelreation is
reverted in this version and rework in next version.

Please review.

Thanks

Jason Wang (2):
  Revert "vhost: access vq metadata through kernel virtual address"
  vhost: re-introducing metadata acceleration through kernel virtual
address

 drivers/vhost/vhost.c | 202 +-
 drivers/vhost/vhost.h |   8 +-
 2 files changed, 123 insertions(+), 87 deletions(-)

-- 
2.19.1

___
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization