Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1622290 > unrolled thread

Re: [PATCH] mm,hugetlb: compute page_size_log properly

Started byDavidlohr Bueso <dave@stgolabs.net>
First post2017-04-12 18:20 +0200
Last post2017-04-12 19:30 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH] mm,hugetlb: compute page_size_log properly Davidlohr Bueso <dave@stgolabs.net> - 2017-04-12 18:20 +0200
    Re: [PATCH] mm,hugetlb: compute page_size_log properly Andi Kleen <ak@linux.intel.com> - 2017-04-12 18:40 +0200
    Re: [PATCH] mm,hugetlb: compute page_size_log properly Matthew Wilcox <willy@infradead.org> - 2017-04-12 19:30 +0200

#1622290 — Re: [PATCH] mm,hugetlb: compute page_size_log properly

FromDavidlohr Bueso <dave@stgolabs.net>
Date2017-04-12 18:20 +0200
SubjectRe: [PATCH] mm,hugetlb: compute page_size_log properly
Message-ID<tvt9U-6Fh-25@gated-at.bofh.it>
On Thu, 30 Mar 2017, Michal Hocko wrote:

>On Wed 29-03-17 10:45:14, Andi Kleen wrote:
>> On Wed, Mar 29, 2017 at 10:06:25AM +0200, Michal Hocko wrote:
>> >
>> > Do we actually have any users?
>>
>> Yes this feature is widely used.
>
>Considering that none of SHM_HUGE* has been exported to the userspace
>headers all the users would have to use the this flag by the value and I
>am quite skeptical that application actually do that. Could you point me
>to some projects that use this?

Hmm Andrew, if there's not one example, could you please pick up this patch?

Thanks,
Davidlohr

[toc] | [next] | [standalone]


#1622312

FromAndi Kleen <ak@linux.intel.com>
Date2017-04-12 18:40 +0200
Message-ID<tvttg-6Mv-13@gated-at.bofh.it>
In reply to#1622290
On Wed, Apr 12, 2017 at 09:18:29AM -0700, Davidlohr Bueso wrote:
> On Thu, 30 Mar 2017, Michal Hocko wrote:
> 
> > On Wed 29-03-17 10:45:14, Andi Kleen wrote:
> > > On Wed, Mar 29, 2017 at 10:06:25AM +0200, Michal Hocko wrote:
> > > >
> > > > Do we actually have any users?
> > > 
> > > Yes this feature is widely used.
> > 
> > Considering that none of SHM_HUGE* has been exported to the userspace
> > headers all the users would have to use the this flag by the value and I
> > am quite skeptical that application actually do that. Could you point me
> > to some projects that use this?
> 
> Hmm Andrew, if there's not one example, could you please pick up this patch?

?!? We just don't break user ABIs this way in Linux!

Just because you don't like something you cannot simply remove it.

I don't know if there are open source users, but there are known closed
source users for which this was originally implemented.

-Andi

[toc] | [prev] | [next] | [standalone]


#1622404

FromMatthew Wilcox <willy@infradead.org>
Date2017-04-12 19:30 +0200
Message-ID<tvufE-7kr-21@gated-at.bofh.it>
In reply to#1622290
On Wed, Apr 12, 2017 at 09:18:29AM -0700, Davidlohr Bueso wrote:
> On Thu, 30 Mar 2017, Michal Hocko wrote:
> 
> > On Wed 29-03-17 10:45:14, Andi Kleen wrote:
> > > On Wed, Mar 29, 2017 at 10:06:25AM +0200, Michal Hocko wrote:
> > > >
> > > > Do we actually have any users?
> > > 
> > > Yes this feature is widely used.
> > 
> > Considering that none of SHM_HUGE* has been exported to the userspace
> > headers all the users would have to use the this flag by the value and I
> > am quite skeptical that application actually do that. Could you point me
> > to some projects that use this?
> 
> Hmm Andrew, if there's not one example, could you please pick up this patch?

Please comment on the replacement patch I suggested.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web