Re: [RESEND 2/3] powerpc/memcpy: Add memcpy_mcsafe for pmem

2018-04-05 Thread Nicholas Piggin
On Thu, 05 Apr 2018 16:40:26 -0400 Jeff Moyer <jmo...@redhat.com> wrote: > Nicholas Piggin <npig...@gmail.com> writes: > > > On Thu, 5 Apr 2018 15:53:07 +1000 > > Balbir Singh <bsinghar...@gmail.com> wrote: > >> I'm thinking about it, I wonde

Re: [RESEND 2/3] powerpc/memcpy: Add memcpy_mcsafe for pmem

2018-04-05 Thread Nicholas Piggin
On Thu, 5 Apr 2018 15:53:07 +1000 Balbir Singh <bsinghar...@gmail.com> wrote: > On Thu, 5 Apr 2018 15:04:05 +1000 > Nicholas Piggin <npig...@gmail.com> wrote: > > > On Wed, 4 Apr 2018 20:00:52 -0700 > > Dan Williams <dan.j.willi...@intel.com> wrote: &

Re: [RESEND 2/3] powerpc/memcpy: Add memcpy_mcsafe for pmem

2018-04-04 Thread Nicholas Piggin
t makes it simple. Would be nice if there was an easy way to share this with the regular memcpy code... that's probably for another day though, probably better to let this settle down first. I didn't review exact instructions, but the approach looks right to me. Acked-by: Nicholas Piggin <npig...@gmail.co

Re: [RESEND 1/3] powerpc/mce: Bug fixes for MCE handling in kernel space

2018-04-04 Thread Nicholas Piggin
On Thu, 5 Apr 2018 09:19:41 +1000 Balbir Singh wrote: > The code currently assumes PAGE_SHIFT as the shift value of > the pfn, this works correctly (mostly) for user space pages, > but the correct thing to do is It would be good to actually explain the problem in the

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Nicholas Piggin
On Fri, 16 Sep 2016 08:33:50 +1000 Dave Chinner <da...@fromorbit.com> wrote: > On Thu, Sep 15, 2016 at 09:42:22PM +1000, Nicholas Piggin wrote: > > On Thu, 15 Sep 2016 20:32:10 +1000 > > Dave Chinner <da...@fromorbit.com> wrote: > > > > > > You stil

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Nicholas Piggin
On Thu, 15 Sep 2016 20:32:10 +1000 Dave Chinner <da...@fromorbit.com> wrote: > On Thu, Sep 15, 2016 at 01:49:45PM +1000, Nicholas Piggin wrote: > > On Thu, 15 Sep 2016 12:31:33 +1000 > > Dave Chinner <da...@fromorbit.com> wrote: > > > > > On Wed,

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-14 Thread Nicholas Piggin
On Thu, 15 Sep 2016 12:31:33 +1000 Dave Chinner <da...@fromorbit.com> wrote: > On Wed, Sep 14, 2016 at 08:19:36PM +1000, Nicholas Piggin wrote: > > On Wed, 14 Sep 2016 17:39:02 +1000 > > Dave Chinner <da...@fromorbit.com> wrote: > > > Ok, look

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-14 Thread Nicholas Piggin
On Wed, 14 Sep 2016 17:39:02 +1000 Dave Chinner <da...@fromorbit.com> wrote: > On Tue, Sep 13, 2016 at 11:53:11AM +1000, Nicholas Piggin wrote: > > On Tue, 13 Sep 2016 07:34:36 +1000 > > Dave Chinner <da...@fromorbit.com> wrote: > > But let me under

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-12 Thread Nicholas Piggin
On Mon, 12 Sep 2016 00:51:28 -0700 Christoph Hellwig wrote: > On Mon, Sep 12, 2016 at 05:25:15PM +1000, Oliver O'Halloran wrote: > > What are the problems here? Is this a matter of existing filesystems > > being unable/unwilling to support this or is it just fundamentally > >