All of lore.kernel.org
 help / color / mirror / Atom feed
* Use of VM_IOREMAP in xenbus
@ 2020-04-09  6:18 ` Christoph Hellwig
  0 siblings, 0 replies; 6+ messages in thread
From: Christoph Hellwig @ 2020-04-09  6:18 UTC (permalink / raw)
  To: Wei Liu
  Cc: Bob Liu, Paul Durrant, Konrad Wilk, Boris Ostrovsky,
	David Vrabel, Juergen Gross, Stefano Stabellini, xen-devel,
	linux-kernel

Hi Wei,

commit ccc9d90a9a8b5 ("xenbus_client: Extend interface to support
multi-page ring") addes a use of vmap in what is now
xenbus_map_ring_valloc_hvm, and uses the VM_IOREMAP flag that is
only really intended for implementing ioremap.  Do you remember
what the reason is that this flag was used?


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

* Use of VM_IOREMAP in xenbus
@ 2020-04-09  6:18 ` Christoph Hellwig
  0 siblings, 0 replies; 6+ messages in thread
From: Christoph Hellwig @ 2020-04-09  6:18 UTC (permalink / raw)
  To: Wei Liu
  Cc: Juergen Gross, Stefano Stabellini, Konrad Wilk, linux-kernel,
	Bob Liu, Paul Durrant, David Vrabel, xen-devel, Boris Ostrovsky

Hi Wei,

commit ccc9d90a9a8b5 ("xenbus_client: Extend interface to support
multi-page ring") addes a use of vmap in what is now
xenbus_map_ring_valloc_hvm, and uses the VM_IOREMAP flag that is
only really intended for implementing ioremap.  Do you remember
what the reason is that this flag was used?



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

* Re: Use of VM_IOREMAP in xenbus
  2020-04-09  6:18 ` Christoph Hellwig
@ 2020-04-09  6:37   ` Jürgen Groß
  -1 siblings, 0 replies; 6+ messages in thread
From: Jürgen Groß @ 2020-04-09  6:37 UTC (permalink / raw)
  To: Christoph Hellwig, Wei Liu
  Cc: Bob Liu, Durrant, Paul, Konrad Wilk, Boris Ostrovsky,
	Stefano Stabellini, xen-devel, linux-kernel

Adjusting recipient list (dropping David, new mail addresses for Wei and
Paul).

On 09.04.20 08:18, Christoph Hellwig wrote:
> Hi Wei,
> 
> commit ccc9d90a9a8b5 ("xenbus_client: Extend interface to support
> multi-page ring") addes a use of vmap in what is now
> xenbus_map_ring_valloc_hvm, and uses the VM_IOREMAP flag that is
> only really intended for implementing ioremap.  Do you remember
> what the reason is that this flag was used?
> 

Juergen

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

* Re: Use of VM_IOREMAP in xenbus
@ 2020-04-09  6:37   ` Jürgen Groß
  0 siblings, 0 replies; 6+ messages in thread
From: Jürgen Groß @ 2020-04-09  6:37 UTC (permalink / raw)
  To: Christoph Hellwig, Wei Liu
  Cc: Stefano Stabellini, Konrad Wilk, Durrant, Paul, linux-kernel,
	Bob Liu, xen-devel, Boris Ostrovsky

Adjusting recipient list (dropping David, new mail addresses for Wei and
Paul).

On 09.04.20 08:18, Christoph Hellwig wrote:
> Hi Wei,
> 
> commit ccc9d90a9a8b5 ("xenbus_client: Extend interface to support
> multi-page ring") addes a use of vmap in what is now
> xenbus_map_ring_valloc_hvm, and uses the VM_IOREMAP flag that is
> only really intended for implementing ioremap.  Do you remember
> what the reason is that this flag was used?
> 

Juergen


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

* Re: Use of VM_IOREMAP in xenbus
  2020-04-09  6:37   ` Jürgen Groß
@ 2020-04-09 14:28     ` Wei Liu
  -1 siblings, 0 replies; 6+ messages in thread
From: Wei Liu @ 2020-04-09 14:28 UTC (permalink / raw)
  To: Jürgen Groß
  Cc: Christoph Hellwig, Wei Liu, Bob Liu, Durrant, Paul, Konrad Wilk,
	Boris Ostrovsky, Stefano Stabellini, xen-devel, linux-kernel

Hi Christoph

On Thu, Apr 09, 2020 at 08:37:48AM +0200, Jürgen Groß wrote:
> Adjusting recipient list (dropping David, new mail addresses for Wei and
> Paul).
> 
> On 09.04.20 08:18, Christoph Hellwig wrote:
> > Hi Wei,
> > 
> > commit ccc9d90a9a8b5 ("xenbus_client: Extend interface to support
> > multi-page ring") addes a use of vmap in what is now
> > xenbus_map_ring_valloc_hvm, and uses the VM_IOREMAP flag that is
> > only really intended for implementing ioremap.  Do you remember
> > what the reason is that this flag was used?
> > 

I don't remember the reason. I can that flag can be dropped per your
reasoning.

Wei.

> 
> Juergen

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

* Re: Use of VM_IOREMAP in xenbus
@ 2020-04-09 14:28     ` Wei Liu
  0 siblings, 0 replies; 6+ messages in thread
From: Wei Liu @ 2020-04-09 14:28 UTC (permalink / raw)
  To: Jürgen Groß
  Cc: Stefano Stabellini, Wei Liu, Konrad Wilk, Durrant, Paul,
	linux-kernel, Bob Liu, xen-devel, Boris Ostrovsky,
	Christoph Hellwig

Hi Christoph

On Thu, Apr 09, 2020 at 08:37:48AM +0200, Jürgen Groß wrote:
> Adjusting recipient list (dropping David, new mail addresses for Wei and
> Paul).
> 
> On 09.04.20 08:18, Christoph Hellwig wrote:
> > Hi Wei,
> > 
> > commit ccc9d90a9a8b5 ("xenbus_client: Extend interface to support
> > multi-page ring") addes a use of vmap in what is now
> > xenbus_map_ring_valloc_hvm, and uses the VM_IOREMAP flag that is
> > only really intended for implementing ioremap.  Do you remember
> > what the reason is that this flag was used?
> > 

I don't remember the reason. I can that flag can be dropped per your
reasoning.

Wei.

> 
> Juergen


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

end of thread, other threads:[~2020-04-09 14:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-09  6:18 Use of VM_IOREMAP in xenbus Christoph Hellwig
2020-04-09  6:18 ` Christoph Hellwig
2020-04-09  6:37 ` Jürgen Groß
2020-04-09  6:37   ` Jürgen Groß
2020-04-09 14:28   ` Wei Liu
2020-04-09 14:28     ` Wei Liu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.