All of lore.kernel.org
 help / color / mirror / Atom feed
From: Balbir Singh <bsingharora@gmail.com>
To: Jerome Glisse <jglisse@redhat.com>
Cc: Anshuman Khandual <khandual@linux.vnet.ibm.com>,
	linux-mm@kvack.org, mhocko@suse.com
Subject: Re: [rfc 2/2] smaps: Show zone device memory used
Date: Sat, 21 Oct 2017 12:23:01 +1100	[thread overview]
Message-ID: <1508548981.5662.4.camel@gmail.com> (raw)
In-Reply-To: <20171019170238.GB3044@redhat.com>

On Thu, 2017-10-19 at 13:02 -0400, Jerome Glisse wrote:
> On Thu, Oct 19, 2017 at 06:48:58AM +1100, Balbir Singh wrote:
> > On Wed, 18 Oct 2017 12:40:43 +0530
> > Anshuman Khandual <khandual@linux.vnet.ibm.com> wrote:
> > 
> > > On 10/18/2017 12:01 PM, Balbir Singh wrote:
> > > > With HMM, we can have either public or private zone
> > > > device pages. With private zone device pages, they should
> > > > show up as swapped entities. For public zone device pages  
> > > 
> > > Might be missing something here but why they should show up
> > > as swapped entities ? Could you please elaborate.
> > > 
> > 
> > For migrated entries, my use case is to
> > 
> > 1. malloc()/mmap() memory
> > 2. call migrate_vma()
> > 3. Look at smaps
> > 
> > It's probably not clear in the changelog.
> 
> My only worry is about API, is smaps consider as userspace API ?

Yes, do you think choosing DevicePublicMemory would help?

> My fear here is that maybe we will want to report device memory
> differently in the future and have different category of device

You are right, things will change and we'll probably see more things
in ZONE_DEVICE, but I am not sure how they'd show up in smaps or
can't think of it at the moment. The reason for my patch is was
that I expect only device public memory to have a need to be
visible in smaps as we do migration from regular memory to device
public memory and vice-versa.

> memory. Even thought right now i can only think of wanting to
> differentiate between public and private device memory but right
> now as you pointed out this is reported as swap out.
> 
> Otherwise patches looks good and you got:
> 
> Reviewed-by: JA(C)rA'me Glisse <jglisse@redhat.com>

Thank you.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2017-10-21  1:23 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-18  6:31 [rfc 1/2] mm/hmm: Allow smaps to see zone device public pages Balbir Singh
2017-10-18  6:31 ` [rfc 2/2] smaps: Show zone device memory used Balbir Singh
2017-10-18  7:10   ` Anshuman Khandual
2017-10-18 19:48     ` Balbir Singh
2017-10-19 17:02       ` Jerome Glisse
2017-10-21  1:23         ` Balbir Singh [this message]
2017-10-20 13:08   ` Michal Hocko
2017-10-20 21:55     ` Balbir Singh
2017-10-23  8:49       ` Michal Hocko
2017-10-23  9:32         ` Balbir Singh
2017-10-23  9:36           ` Michal Hocko
2017-10-23  9:37             ` Balbir Singh
2017-10-18  6:56 ` [rfc 1/2] mm/hmm: Allow smaps to see zone device public pages Anshuman Khandual
2017-10-18 19:36   ` Balbir Singh
2017-10-20 13:11 ` Michal Hocko
2017-10-20 21:52   ` Balbir Singh
2017-10-23  8:52     ` Michal Hocko

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=1508548981.5662.4.camel@gmail.com \
    --to=bsingharora@gmail.com \
    --cc=jglisse@redhat.com \
    --cc=khandual@linux.vnet.ibm.com \
    --cc=linux-mm@kvack.org \
    --cc=mhocko@suse.com \
    /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.