Re: [PATCH v5 15/17] device-dax: add an 'align' attribute

From: Dan Williams
Date: Fri Sep 25 2020 - 23:32:21 EST


On Fri, Sep 25, 2020 at 7:22 PM Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
>
> On Fri, 25 Sep 2020 12:13:04 -0700 Dan Williams <dan.j.williams@xxxxxxxxx> wrote:
>
> > Introduce a device align attribute. While doing so, rename the region
> > align attribute to be more explicitly named as so, but keep it named as
> > @align to retain the API for tools like daxctl.
> >
> > Changes on align may not always be valid, when say certain mappings were
> > created with 2M and then we switch to 1G. So, we validate all ranges
> > against the new value being attempted, post resizing.
> >
> > Link: https://lkml.kernel.org/r/159643105944.4062302.3131761052969132784.stgit@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
> > Link: https://lore.kernel.org/r/20200716172913.19658-3-joao.m.martins@xxxxxxxxxx
> > Signed-off-by: Joao Martins <joao.m.martins@xxxxxxxxxx>
> > Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx>
> >
>
> The signoff chain implies that this was From:Joao. Please clarify?

Yes, sorry, my script to squash the fix rewrote the author and I
failed to catch it. This indeed should be:

From: Joao Martins <joao.m.martins@xxxxxxxxxx>

I double-checked, and it looks like this was the only one in the
series with that problem.