Re: [PATCH 2/2] device-dax: kill NR_DEV_DAX

[Geert Uytterhoeven]

On Mon, May 8, 2017 at 11:03 PM, Dan Williams <dan.j.williams@xxxxxxxxx> wrote:
> There is no point to ask how many device-dax instances the kernel should
> support. Since we are already using a dynamic major number, just allow
> the max number of minors by default and be done. This also fixes the
> fact that the proposed max for the NR_DEV_DAX range was larger than what
> could be supported by alloc_chrdev_region().
>
> Fixes: ba09c01d2fa8 ("dax: convert to the cdev api")
> Reported-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx>

Tested-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds