mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "prakash.sangappa" <prakash.sangappa@oracle.com>
To: Michal Hocko <mhocko@kernel.org>
Cc: Mike Rapoport <rppt@linux.vnet.ibm.com>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	Andrea Arcangeli <aarcange@redhat.com>,
	Mike Kravetz <mike.kravetz@oracle.com>,
	Dave Hansen <dave.hansen@intel.com>,
	Christoph Hellwig <hch@infradead.org>,
	linux-api@vger.kernel.org
Subject: Re: [RFC PATCH] userfaultfd: Add feature to request for a signal delivery
Date: Thu, 29 Jun 2017 14:41:22 -0700	[thread overview]
Message-ID: <936bde7b-1913-5589-22f4-9bbfdb6a8dd5@oracle.com> (raw)
In-Reply-To: <20170629080910.GC31603@dhcp22.suse.cz>



On 06/29/2017 01:09 AM, Michal Hocko wrote:
> On Wed 28-06-17 11:23:32, Prakash Sangappa wrote:
>>
>> On 6/28/17 6:18 AM, Mike Rapoport wrote:
> [...]
>>> I've just been thinking that maybe it would be possible to use
>>> UFFD_EVENT_REMOVE for this case. We anyway need to implement the generation
>>> of UFFD_EVENT_REMOVE for the case of hole punching in hugetlbfs for
>>> non-cooperative userfaultfd. It could be that it will solve your issue as
>>> well.
>>>
>> Will this result in a signal delivery?
>>
>> In the use case described, the database application does not need any event
>> for  hole punching. Basically, just a signal for any invalid access to
>> mapped area over holes in the file.
> OK, but it would be better to think that through for other potential
> usecases so that this doesn't end up as a single hugetlb feature. E.g.
> what should happen if a regular anonymous memory gets swapped out?
> Should we deliver signal as well? How does userspace tell whether this
> was a no backing page from unavailable backing page?

This may not be useful in all cases. Potential, it could be used
with use of mlock() on anonymous memory to ensure any access
to memory that is not locked is caught, again for robustness
purpose.

  reply	other threads:[~2017-06-29 21:40 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-26 19:46 Prakash Sangappa
2017-06-27  7:06 ` Michal Hocko
2017-06-27 15:35   ` Mike Rapoport
2017-06-27 16:01     ` Prakash Sangappa
2017-06-28 13:18       ` Mike Rapoport
2017-06-28 18:23         ` Prakash Sangappa
2017-06-29  8:09           ` Michal Hocko
2017-06-29 21:41             ` prakash.sangappa [this message]
2017-06-30  9:47               ` Michal Hocko
2017-06-30 13:08                 ` Andrea Arcangeli
2017-07-01  0:55                   ` prakash sangappa
2017-07-04 16:40                     ` Andrea Arcangeli
2017-07-05 22:24                       ` prakash.sangappa
2017-07-05 18:41                 ` John Stultz
2017-06-29 10:46           ` Mike Rapoport
2017-06-29 21:49             ` prakash.sangappa
2017-06-27 15:47   ` Prakash Sangappa

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=936bde7b-1913-5589-22f4-9bbfdb6a8dd5@oracle.com \
    --to=prakash.sangappa@oracle.com \
    --cc=aarcange@redhat.com \
    --cc=dave.hansen@intel.com \
    --cc=hch@infradead.org \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mhocko@kernel.org \
    --cc=mike.kravetz@oracle.com \
    --cc=rppt@linux.vnet.ibm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®