* [PATCH] dax: ->direct_access does not sleep anymore
@ 2018-02-26 16:50 ` Boaz Harrosh
0 siblings, 0 replies; 4+ messages in thread
From: Boaz Harrosh @ 2018-02-26 16:50 UTC (permalink / raw)
To: Dan Williams, linux-fsdevel, linux-nvdimm
In Patch:
[7a862fb] brd: remove dax support
Dan Williams has removed the only might_sleep
implementation of ->direct_access.
So we no longer need to check for it.
CC: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Boaz Harrosh <boazh@netapp.com>
---
drivers/dax/super.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/dax/super.c b/drivers/dax/super.c
index 3ec8046..0a75550 100644
--- a/drivers/dax/super.c
+++ b/drivers/dax/super.c
@@ -236,12 +236,6 @@ long dax_direct_access(struct dax_device *dax_dev, pgoff_t pgoff, long nr_pages,
{
long avail;
- /*
- * The device driver is allowed to sleep, in order to make the
- * memory directly accessible.
- */
- might_sleep();
-
if (!dax_dev)
return -EOPNOTSUPP;
--
2.5.5
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] dax: ->direct_access does not sleep anymore
@ 2018-02-26 16:50 ` Boaz Harrosh
0 siblings, 0 replies; 4+ messages in thread
From: Boaz Harrosh @ 2018-02-26 16:50 UTC (permalink / raw)
To: Dan Williams, linux-fsdevel, linux-nvdimm
In Patch:
[7a862fb] brd: remove dax support
Dan Williams has removed the only might_sleep
implementation of ->direct_access.
So we no longer need to check for it.
CC: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Boaz Harrosh <boazh@netapp.com>
---
drivers/dax/super.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/dax/super.c b/drivers/dax/super.c
index 3ec8046..0a75550 100644
--- a/drivers/dax/super.c
+++ b/drivers/dax/super.c
@@ -236,12 +236,6 @@ long dax_direct_access(struct dax_device *dax_dev, pgoff_t pgoff, long nr_pages,
{
long avail;
- /*
- * The device driver is allowed to sleep, in order to make the
- * memory directly accessible.
- */
- might_sleep();
-
if (!dax_dev)
return -EOPNOTSUPP;
--
2.5.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] dax: ->direct_access does not sleep anymore
2018-02-26 16:50 ` Boaz Harrosh
@ 2018-02-26 20:41 ` Dan Williams
-1 siblings, 0 replies; 4+ messages in thread
From: Dan Williams @ 2018-02-26 20:41 UTC (permalink / raw)
To: Boaz Harrosh; +Cc: linux-fsdevel, linux-nvdimm
On Mon, Feb 26, 2018 at 8:50 AM, Boaz Harrosh <boazh@netapp.com> wrote:
>
> In Patch:
> [7a862fb] brd: remove dax support
>
> Dan Williams has removed the only might_sleep
> implementation of ->direct_access.
> So we no longer need to check for it.
>
> CC: Dan Williams <dan.j.williams@intel.com>
> Signed-off-by: Boaz Harrosh <boazh@netapp.com>
Looks good, applied.
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dax: ->direct_access does not sleep anymore
@ 2018-02-26 20:41 ` Dan Williams
0 siblings, 0 replies; 4+ messages in thread
From: Dan Williams @ 2018-02-26 20:41 UTC (permalink / raw)
To: Boaz Harrosh; +Cc: linux-fsdevel, linux-nvdimm
On Mon, Feb 26, 2018 at 8:50 AM, Boaz Harrosh <boazh@netapp.com> wrote:
>
> In Patch:
> [7a862fb] brd: remove dax support
>
> Dan Williams has removed the only might_sleep
> implementation of ->direct_access.
> So we no longer need to check for it.
>
> CC: Dan Williams <dan.j.williams@intel.com>
> Signed-off-by: Boaz Harrosh <boazh@netapp.com>
Looks good, applied.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-02-26 20:41 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-26 16:50 [PATCH] dax: ->direct_access does not sleep anymore Boaz Harrosh
2018-02-26 16:50 ` Boaz Harrosh
2018-02-26 20:41 ` Dan Williams
2018-02-26 20:41 ` Dan Williams
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.