On Sat, Feb 25, 2017 at 10:38 PM, Logan Gunthorpe <log...@deltatee.com> wrote:
> From: Dan Williams <dan.j.willi...@intel.com>
>
> If device_add() fails, cleanup the cdev. Otherwise, we leak a kobj_map()
> with a stale device number.
>
> As Jason points out, there is a small possibility that userspace has
> opened and mapped the device in the time between cdev_add() and the
> device_add() failure. We need a new kill_dax_dev() helper to invalidate
> any established mappings.
>
> Fixes: ba09c01d2fa8 ("dax: convert to the cdev api")
> Cc: <sta...@vger.kernel.org>
> Cc: Logan Gunthorpe <log...@deltatee.com>
> Reported-by: Jason Gunthorpe <jguntho...@obsidianresearch.com>
> Signed-off-by: Dan Williams <dan.j.willi...@intel.com>

Since you are forwarding this patch you should add

"Signed-off-by: Logan Gunthorpe <log...@deltatee.com>"
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

Reply via email to