All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Liu <wl@xen.org>
To: "Jürgen Groß" <jgross@suse.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>, Wei Liu <wl@xen.org>,
	Konrad Wilk <konrad.wilk@oracle.com>,
	"Durrant, Paul" <pdurrant@amazon.com>,
	linux-kernel@vger.kernel.org, Bob Liu <bob.liu@oracle.com>,
	xen-devel@lists.xenproject.org,
	Boris Ostrovsky <boris.ostrovsky@oracle.com>,
	Christoph Hellwig <hch@lst.de>
Subject: Re: Use of VM_IOREMAP in xenbus
Date: Thu, 9 Apr 2020 15:28:13 +0100	[thread overview]
Message-ID: <20200409142813.lbxembj7b63b2wmi@debian> (raw)
In-Reply-To: <8074b77d-d784-95ee-8d47-069827855876@suse.com>

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


WARNING: multiple messages have this Message-ID (diff)
From: Wei Liu <wl@xen.org>
To: "Jürgen Groß" <jgross@suse.com>
Cc: Christoph Hellwig <hch@lst.de>, Wei Liu <wl@xen.org>,
	Bob Liu <bob.liu@oracle.com>,
	"Durrant, Paul" <pdurrant@amazon.com>,
	Konrad Wilk <konrad.wilk@oracle.com>,
	Boris Ostrovsky <boris.ostrovsky@oracle.com>,
	Stefano Stabellini <sstabellini@kernel.org>,
	xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org
Subject: Re: Use of VM_IOREMAP in xenbus
Date: Thu, 9 Apr 2020 15:28:13 +0100	[thread overview]
Message-ID: <20200409142813.lbxembj7b63b2wmi@debian> (raw)
In-Reply-To: <8074b77d-d784-95ee-8d47-069827855876@suse.com>

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

  reply	other threads:[~2020-04-09 14:28 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2020-04-09 14:28     ` Wei Liu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200409142813.lbxembj7b63b2wmi@debian \
    --to=wl@xen.org \
    --cc=bob.liu@oracle.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=hch@lst.de \
    --cc=jgross@suse.com \
    --cc=konrad.wilk@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pdurrant@amazon.com \
    --cc=sstabellini@kernel.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.