):ocfs2_check_if_ancestor:1091 max
lookup times reached, filesystem may have nested directories, src inode: 18007,
dest inode: 17940.
...
Signed-off-by: Mauricio Faria de Oliveira
---
fs/ocfs2/namei.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/ocfs2/namei.c b/fs/ocfs2/namei.c
index
On 04/09/2018 11:51 PM, Michael Ellerman wrote:
Thanks for picking this one up.
I hate to be a pain ... but before we merge this and proliferate these
names, I'd like to change the names of some of these early asm
functions. They're terribly named due to historical reasons.
Indeed :) No
On 04/09/2018 11:51 PM, Michael Ellerman wrote:
Thanks for picking this one up.
I hate to be a pain ... but before we merge this and proliferate these
names, I'd like to change the names of some of these early asm
functions. They're terribly named due to historical reasons.
Indeed :) No
rence from the
variable start_here_common to the function .init.text:start_kernel()
The function start_here_common() references
the function __init start_kernel().
This is often because start_here_common lacks a __init
annotation or the annotation of start_kernel is wrong.
Signed-off-by: Maur
rence from the
variable start_here_common to the function .init.text:start_kernel()
The function start_here_common() references
the function __init start_kernel().
This is often because start_here_common lacks a __init
annotation or the annotation of start_kernel is wrong.
Signed-off-by: Maur
: Nicholas Piggin
Date: Fri Dec 23 00:14:19 AEDT 2016
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
v2: fix missing close parenthesis in conditional (wrong patch file, sorry)
scripts/mod/modpost.c | 22 --
1 file changed, 20 insertions
: Nicholas Piggin
Date: Fri Dec 23 00:14:19 AEDT 2016
Signed-off-by: Mauricio Faria de Oliveira
---
v2: fix missing close parenthesis in conditional (wrong patch file, sorry)
scripts/mod/modpost.c | 22 --
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/s
: Nicholas Piggin
Date: Fri Dec 23 00:14:19 AEDT 2016
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
scripts/mod/modpost.c | 22 --
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost
: Nicholas Piggin
Date: Fri Dec 23 00:14:19 AEDT 2016
Signed-off-by: Mauricio Faria de Oliveira
---
scripts/mod/modpost.c | 22 --
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
index 9917f92..c65d5e2 100644
---
On 09/26/2017 05:40 AM, Stewart Smith wrote:
The simple fix is to bump the length of the array to 32 which "should be
enough for everyone(TM)".
Tested-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
# uname -r
4.14.0-rc3
# dmesg
[0.00] opal: OPAL detected
On 09/26/2017 05:40 AM, Stewart Smith wrote:
The simple fix is to bump the length of the array to 32 which "should be
enough for everyone(TM)".
Tested-by: Mauricio Faria de Oliveira
# uname -r
4.14.0-rc3
# dmesg
[0.00] opal: OPAL detected !
[0.00] crashkernel: me
On 07/11/2017 12:32 PM, Mauricio Faria de Oliveira wrote:
Also, it seems the Unavailable/Standby states would not be logged
without a recheck from alua_check_sense(), since the only callers
of alua_rtpg_queue() are alua_activate() and alua_check[_sense]()
Well, actually it does get logged
On 07/11/2017 12:32 PM, Mauricio Faria de Oliveira wrote:
Also, it seems the Unavailable/Standby states would not be logged
without a recheck from alua_check_sense(), since the only callers
of alua_rtpg_queue() are alua_activate() and alua_check[_sense]()
Well, actually it does get logged
go through that function.
(and it occurred to me that the state-change check of patch 3 can
be done there, simpler.)
cheers,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
go through that function.
(and it occurred to me that the state-change check of patch 3 can
be done there, simpler.)
cheers,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
On 07/10/2017 07:47 PM, Mauricio Faria de Oliveira wrote:
This patchset addresses that problem, and adds a few improvements
to the logging of PG state changes.
Here are some kernel log snippets with the patchset, if that helps.
The 2 port groups temporarily gone into unavailable state
On 07/10/2017 07:47 PM, Mauricio Faria de Oliveira wrote:
This patchset addresses that problem, and adds a few improvements
to the logging of PG state changes.
Here are some kernel log snippets with the patchset, if that helps.
The 2 port groups temporarily gone into unavailable state
ated on path activation (alua_activate(),
as it schedules a recheck), thus I/O requests are no longer failed.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
Reported-by: Naresh Bannoth <nbann...@in.ibm.com>
---
v2:
- also add support for standby state
ated on path activation (alua_activate(),
as it schedules a recheck), thus I/O requests are no longer failed.
Signed-off-by: Mauricio Faria de Oliveira
Reported-by: Naresh Bannoth
---
v2:
- also add support for standby state to alua_check_sense(), alua_prep_fn()
(Bart Van Assche )
driv
for the current PG.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
v2:
- use lockdep_assert_held() instead of documenting locking conventions
(Bart Van Assche <bart.vanass...@sandisk.com>)
- define two functions (with/without supported states information)
for the current PG.
Signed-off-by: Mauricio Faria de Oliveira
---
v2:
- use lockdep_assert_held() instead of documenting locking conventions
(Bart Van Assche )
- define two functions (with/without supported states information)
(Bart Van Assche )
- simplify which device is used for printing
Insert sdev_dbg() calls in the function path which may queue
alua_rtpg_work() past initialization, for debugging purposes:
- alua_activate()
- alua_check_sense()
- alua_rtpg_queue()
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
drivers/scsi/device_h
Insert sdev_dbg() calls in the function path which may queue
alua_rtpg_work() past initialization, for debugging purposes:
- alua_activate()
- alua_check_sense()
- alua_rtpg_queue()
Signed-off-by: Mauricio Faria de Oliveira
---
drivers/scsi/device_handler/scsi_dh_alua.c | 14 --
1
scheduled in alua_check_sense() to update PG state.
So, do not to print such message if unavailable/standby state remains
(i.e., the PG did not transition to/from such states). All other cases
continue to be printed.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
scheduled in alua_check_sense() to update PG state.
So, do not to print such message if unavailable/standby state remains
(i.e., the PG did not transition to/from such states). All other cases
continue to be printed.
Signed-off-by: Mauricio Faria de Oliveira
---
v2:
- changed v1's alua_state_remains
in unavailable/standby
are not logged - only changes are.
Patch 4 adds few sdev_dbg() calls to track the path to alua_rtpg_work()
Tested on v4.12+ (commit b4b8cbf679c4).
Mauricio Faria de Oliveira (4):
scsi: scsi_dh_alua: allow I/O in target port unavailable and standby
states
scsi
in unavailable/standby
are not logged - only changes are.
Patch 4 adds few sdev_dbg() calls to track the path to alua_rtpg_work()
Tested on v4.12+ (commit b4b8cbf679c4).
Mauricio Faria de Oliveira (4):
scsi: scsi_dh_alua: allow I/O in target port unavailable and standby
states
scsi
SED.
test cases/13.t completed PASSED.
test cases/14.t completed PASSED.
test cases/15.t completed PASSED.
test cases/16.t completed PASSED.
test cases/10.t completed PASSED.
test cases/8.t completed PASSED.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ib
SED.
test cases/13.t completed PASSED.
test cases/14.t completed PASSED.
test cases/15.t completed PASSED.
test cases/16.t completed PASSED.
test cases/10.t completed PASSED.
test cases/8.t completed PASSED.
Signed-off-by: Mauricio Faria de Oliveira
Reported-by: Lekshmi C
f how the I/O is
being broken up into frames at the transport level and at which offset
the transfer was interrupted.
Christoph, Hannes, Martin,
Thank you all for your comments and pointers to the documentation/spec.
I'll carry it on with the HBA and storage folks.
cheers,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
f how the I/O is
being broken up into frames at the transport level and at which offset
the transfer was interrupted.
Christoph, Hannes, Martin,
Thank you all for your comments and pointers to the documentation/spec.
I'll carry it on with the HBA and storage folks.
cheers,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
[sda] tag#0 8 sectors total, 4096 bytes done.
[...] sd 0:0:0:0: tag#0 0 sectors total, 0 bytes done.
Apologies for the ridiculously long commit message with description and
test-cases, but this problem has been relatively difficult to reproduce
and understand, so I thought the documentation/instr
[sda] tag#0 8 sectors total, 4096 bytes done.
[...] sd 0:0:0:0: tag#0 0 sectors total, 0 bytes done.
Apologies for the ridiculously long commit message with description and
test-cases, but this problem has been relatively difficult to reproduce
and understand, so I thought the documentation/instru
or 17096824
Links:
[1]
http://git.qemu.org/?p=qemu.git;a=commit;h=336a6915bc7089fb20fea4ba99972ad9a97c5f52
[2] https://libvirt.org/formatdomain.html#elementsDisks (see 'disk' -> 'device')
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
Signed-off-by: Brahadamba
or 17096824
Links:
[1]
http://git.qemu.org/?p=qemu.git;a=commit;h=336a6915bc7089fb20fea4ba99972ad9a97c5f52
[2] https://libvirt.org/formatdomain.html#elementsDisks (see 'disk' -> 'device')
Signed-off-by: Mauricio Faria de Oliveira
Signed-off-by: Brahadambal Srinivasan
Reported-by: Manjunatha
or 17096824
Links:
[1]
http://git.qemu.org/?p=qemu.git;a=commit;h=336a6915bc7089fb20fea4ba99972ad9a97c5f52
[2] https://libvirt.org/formatdomain.html#elementsDisks (see 'disk' -> 'device')
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
Signed-off-by: Brahadamba
or 17096824
Links:
[1]
http://git.qemu.org/?p=qemu.git;a=commit;h=336a6915bc7089fb20fea4ba99972ad9a97c5f52
[2] https://libvirt.org/formatdomain.html#elementsDisks (see 'disk' -> 'device')
Signed-off-by: Mauricio Faria de Oliveira
Signed-off-by: Brahadambal Srinivasan
Reported-by: Manjunatha
On 11/23/2016 12:12 PM, Johannes Thumshirn wrote:
Looks good and sorry for the bug,
Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de>
Thanks for the quick review. Not a problem!
This problem turned out to be a good learning exercise. :)
--
Mauricio Faria de Oliveira
IBM Linux Tech
On 11/23/2016 12:12 PM, Johannes Thumshirn wrote:
Looks good and sorry for the bug,
Reviewed-by: Johannes Thumshirn
Thanks for the quick review. Not a problem!
This problem turned out to be a good learning exercise. :)
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
Due credit; an oversight.
On 11/23/2016 10:33 AM, Mauricio Faria de Oliveira wrote:
Reported-by: Harsha Thyagaraja <hathy...@in.ibm.com>
Cc: sta...@vger.kernel.org # v4.8
Fixes: 22466da5b4b7 ("lpfc: Fix possible NULL pointer dereference")
Signed-off-by: Mauricio Faria de
Due credit; an oversight.
On 11/23/2016 10:33 AM, Mauricio Faria de Oliveira wrote:
Reported-by: Harsha Thyagaraja
Cc: sta...@vger.kernel.org # v4.8
Fixes: 22466da5b4b7 ("lpfc: Fix possible NULL pointer dereference")
Signed-off-by: Mauricio Faria de Oliveira
--
Maurici
et_from_kernel_thread+0x5c/0xbc
<...>
Cc: sta...@vger.kernel.org # v4.8
Fixes: 22466da5b4b7 ("lpfc: Fix possible NULL pointer dereference")
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
drivers/scsi/lpfc/lpfc_sli.c | 14 --
1 file
et_from_kernel_thread+0x5c/0xbc
<...>
Cc: sta...@vger.kernel.org # v4.8
Fixes: 22466da5b4b7 ("lpfc: Fix possible NULL pointer dereference")
Signed-off-by: Mauricio Faria de Oliveira
---
drivers/scsi/lpfc/lpfc_sli.c | 14 --
1 file changed, 8 insertions(+), 6 deletio
orry for this oversight.)
With it applied, both PCI device remove and EEH recovery works fine.
Fixes: 1535aa75a3d8 ("scsi: qla2xxx: fix invalid DMA access after
command aborts in PCI device remove")
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
orry for this oversight.)
With it applied, both PCI device remove and EEH recovery works fine.
Fixes: 1535aa75a3d8 ("scsi: qla2xxx: fix invalid DMA access after
command aborts in PCI device remove")
Signed-off-by: Mauricio Faria de Oliveira
---
drivers/scsi/qla2xxx/qla_os.c | 21
and handle responses.
Reported-by: Naresh Bannoth <nbann...@in.ibm.com>
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
drivers/scsi/qla2xxx/qla_os.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2
700a10] qla2xxx_queuecommand+0x50/0x3f0 [qla2xxx]
So, fail commands in qla2xxx_queuecommand() if the UNLOADING bit is set.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
drivers/scsi/qla2xxx/qla_os.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/
and handle responses.
Reported-by: Naresh Bannoth
Signed-off-by: Mauricio Faria de Oliveira
---
drivers/scsi/qla2xxx/qla_os.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c
index fdb135b..c50dd22 100644
--- a/drivers/scsi
700a10] qla2xxx_queuecommand+0x50/0x3f0 [qla2xxx]
So, fail commands in qla2xxx_queuecommand() if the UNLOADING bit is set.
Signed-off-by: Mauricio Faria de Oliveira
---
drivers/scsi/qla2xxx/qla_os.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/driver
xxx [001d:70:00.0]-801c:1: Abort command issued nexus=1:0:0 -- 1 2003.
<...>
qla2xxx [001d:70:00.1]-801c:2: Abort command issued nexus=2:3:0 -- 1 2003.
<...>
(command does return; adapter can be re-added correctly)
Mauricio Faria de Oliveira (2):
qla2xxx: do no
xxx [001d:70:00.0]-801c:1: Abort command issued nexus=1:0:0 -- 1 2003.
<...>
qla2xxx [001d:70:00.1]-801c:2: Abort command issued nexus=2:3:0 -- 1 2003.
<...>
(command does return; adapter can be re-added correctly)
Mauricio Faria de Oliveira (2):
qla2xxx: do no
,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
systems.
Regards,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
systems.
Regards,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
cpus() * 4", and what is the scenario
where the math is based on? I've been thinking and assuming values for
a while now, and didn't figure out the point where / how it occurs.
Thanks for your support,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
cpus() * 4", and what is the scenario
where the math is based on? I've been thinking and assuming values for
a while now, and didn't figure out the point where / how it occurs.
Thanks for your support,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
e actually requested by userspace?) or the number
of times io_setup(N, ) returned successfully (say, io contexts),
regardless of the total/sum of their nr_events?
- any other comments/suggestions are appreciated.
Thanks in advance,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
e actually requested by userspace?) or the number
of times io_setup(N, ) returned successfully (say, io contexts),
regardless of the total/sum of their nr_events?
- any other comments/suggestions are appreciated.
Thanks in advance,
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
On 09/28/2016 05:45 AM, Andrew Donnellan wrote:
2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl
case. Mauricio - any objections from the pseries/dlpar side?
No objections.
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
On 09/28/2016 05:45 AM, Andrew Donnellan wrote:
2dd9c11b9d4 applies cleanly on top of 4.4 and works for me in the cxl
case. Mauricio - any objections from the pseries/dlpar side?
No objections.
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
that could help with it, I'd be happy to
write something simpler/more discreet.
Thanks for the suggestions and discussions.
[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-June/144196.html
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
that could help with it, I'd be happy to
write something simpler/more discreet.
Thanks for the suggestions and discussions.
[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-June/144196.html
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
to), but it wasn't accepted [1]; so
I guess it apparently is, in some cases.
[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-June/144196.html
thanks
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
to), but it wasn't accepted [1]; so
I guess it apparently is, in some cases.
[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-June/144196.html
thanks
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
y can return either
BLK_MQ_RQ_QUEUE_BUSY or BLK_MQ_RQ_QUEUE_ERROR - so the former is OK.)
Does that make sense?
Thanks for the review.
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
y can return either
BLK_MQ_RQ_QUEUE_BUSY or BLK_MQ_RQ_QUEUE_ERROR - so the former is OK.)
Does that make sense?
Thanks for the review.
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
Documentation/DMA-attributes.txt | 17 +
include/linux/dma-mapping.h | 5 +
2 files changed, 22 insertions(+)
diff
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
arch/powerpc/kernel/iommu.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/kernel/iommu.c b/arch/p
Use the DMA_ATTR_NO_WARN attribute for the dma_map_sg() call of the nvme
driver that returns BLK_MQ_RQ_QUEUE_BUSY (not for BLK_MQ_RQ_QUEUE_ERROR).
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
Reviewed-by: Gabriel Krisman Bertazi <kris...@linux.vnet.ibm.com>
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira
---
Documentation/DMA-attributes.txt | 17 +
include/linux/dma-mapping.h | 5 +
2 files changed, 22 insertions(+)
diff --git a/Documentation/DMA-attributes.txt b
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira
---
arch/powerpc/kernel/iommu.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/kernel/iommu.c b/arch/powerpc/kernel/iommu.c
index 37d6e74
Use the DMA_ATTR_NO_WARN attribute for the dma_map_sg() call of the nvme
driver that returns BLK_MQ_RQ_QUEUE_BUSY (not for BLK_MQ_RQ_QUEUE_ERROR).
Signed-off-by: Mauricio Faria de Oliveira
Reviewed-by: Gabriel Krisman Bertazi
---
drivers/nvme/host/pci.c | 3 ++-
1 file changed, 2 insertions
ss warnings from checkpatch.pl (line wrapping and typos)
Tested on next-20160801.
Mauricio Faria de Oliveira (3):
dma-mapping: introduce the DMA_ATTR_NO_WARN attribute
powerpc: implement the DMA_ATTR_NO_WARN attribute
nvme: use the DMA_ATTR_NO_WARN attribute
Documentation/DMA-att
e wrapping and typos)
Tested on next-20160801.
Mauricio Faria de Oliveira (3):
dma-mapping: introduce the DMA_ATTR_NO_WARN attribute
powerpc: implement the DMA_ATTR_NO_WARN attribute
nvme: use the DMA_ATTR_NO_WARN attribute
Documentation/DMA-attributes.txt | 17 +
arch/powe
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
Changelog:
v3:
- powerpc: none
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
arch/powerpc/kernel/iommu
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
Changelog:
v3:
- dma: none.
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
Documentation/DMA-attributes.tx
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira
---
Changelog:
v3:
- powerpc: none
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
arch/powerpc/kernel/iommu.c | 6 --
1 file changed, 4 insertions
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira
---
Changelog:
v3:
- dma: none.
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
Documentation/DMA-attributes.txt | 17 +
include/linux/dma-attrs.h
Use the DMA_ATTR_NO_WARN attribute on dma_map_sg() calls of nvme driver.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
Reviewed-by: Gabriel Krisman Bertazi <kris...@linux.vnet.ibm.com>
---
Changelog:
v3:
- nvme: use DMA_ATTR_NO_WARN when ret = BLK_MQ_RQ_QU
Use the DMA_ATTR_NO_WARN attribute on dma_map_sg() calls of nvme driver.
Signed-off-by: Mauricio Faria de Oliveira
Reviewed-by: Gabriel Krisman Bertazi
---
Changelog:
v3:
- nvme: use DMA_ATTR_NO_WARN when ret = BLK_MQ_RQ_QUEUE_BUSY (io will be
requeued) but not when ret
nvme: use DMA_ATTR_NO_WARN when ret = BLK_MQ_RQ_QUEUE_BUSY (io will be
requeued) but not when ret = BLK_MQ_RQ_QUEUE_ERROR (io will be failed).
thanks: Masayoshi Mizuma <m.miz...@jp.fujitsu.com>
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
Mauricio Faria de Oli
nvme: use DMA_ATTR_NO_WARN when ret = BLK_MQ_RQ_QUEUE_BUSY (io will be
requeued) but not when ret = BLK_MQ_RQ_QUEUE_ERROR (io will be failed).
thanks: Masayoshi Mizuma
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
Mauricio Faria de Oliveira (3):
dma:
On 07/08/2016 04:54 AM, Masayoshi Mizuma wrote:
Here, I think the error messages should not be suppressed because
the return value of nvme_map_data() is BLK_MQ_RQ_QUEUE_ERROR, so
the IO returns as -EIO.
Agree; good point. fixed in v3.
Thanks for reviewing.
--
Mauricio Faria de Oliveira
IBM
On 07/08/2016 04:54 AM, Masayoshi Mizuma wrote:
Here, I think the error messages should not be suppressed because
the return value of nvme_map_data() is BLK_MQ_RQ_QUEUE_ERROR, so
the IO returns as -EIO.
Agree; good point. fixed in v3.
Thanks for reviewing.
--
Mauricio Faria de Oliveira
IBM
On 07/06/2016 09:41 PM, Gabriel Krisman Bertazi wrote:
checkpatch.pl complains about line wrapping. Other than that, this
looks good to me.
I'll submit a v2 w/ that and typos fixed.
Thanks for reviewing.
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
On 07/06/2016 09:41 PM, Gabriel Krisman Bertazi wrote:
checkpatch.pl complains about line wrapping. Other than that, this
looks good to me.
I'll submit a v2 w/ that and typos fixed.
Thanks for reviewing.
--
Mauricio Faria de Oliveira
IBM Linux Technology Center
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
Changelog:
v2:
- address warnings from checkpatch.pl (line wrapping and typos)
Documentation/DMA-attributes.txt | 17 +
include/lin
s warnings from checkpatch.pl (line wrapping and typos)
Mauricio Faria de Oliveira (3):
dma: introduce DMA_ATTR_NO_WARN
nvme: implement DMA_ATTR_NO_WARN
powerpc: implement DMA_ATTR_NO_WARN
Documentation/DMA-attributes.txt | 17 +
arch/powerpc/kernel/iommu.c | 6 -
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
Changelog:
v2:
- address warnings from checkpatch.pl (line wrapping and typos)
arch/powerpc/kernel/iommu.c | 6 --
1 file chan
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira
---
Changelog:
v2:
- address warnings from checkpatch.pl (line wrapping and typos)
Documentation/DMA-attributes.txt | 17 +
include/linux/dma-attrs.h| 1 +
2 files
s warnings from checkpatch.pl (line wrapping and typos)
Mauricio Faria de Oliveira (3):
dma: introduce DMA_ATTR_NO_WARN
nvme: implement DMA_ATTR_NO_WARN
powerpc: implement DMA_ATTR_NO_WARN
Documentation/DMA-attributes.txt | 17 +
arch/powerpc/kernel/iommu.c | 6 -
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira
---
Changelog:
v2:
- address warnings from checkpatch.pl (line wrapping and typos)
arch/powerpc/kernel/iommu.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff
Use the DMA_ATTR_NO_WARN attribute on dma_map_sg() calls of nvme driver.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
Reviewed-by: Gabriel Krisman Bertazi <kris...@linux.vnet.ibm.com>
---
Changelog:
v2:
- address warnings from checkpatch.pl (line wrappi
Use the DMA_ATTR_NO_WARN attribute on dma_map_sg() calls of nvme driver.
Signed-off-by: Mauricio Faria de Oliveira
Reviewed-by: Gabriel Krisman Bertazi
---
Changelog:
v2:
- address warnings from checkpatch.pl (line wrapping and typos)
drivers/nvme/host/pci.c | 12 ++--
1 file
with error messages at least on powerpc: "iommu_alloc failed")
- Patch 3/3 implements support for it on powerpc arch (where this problem
was observed. It's possible to extend support for more archs
if the patchset is welcome).
Mauricio Faria de Oliveira
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
Documentation/DMA-attributes.txt | 17 +
include/linux/dma-attrs.h| 1 +
2 files changed, 18 insertions(+)
diff --git a/Documen
with error messages at least on powerpc: "iommu_alloc failed")
- Patch 3/3 implements support for it on powerpc arch (where this problem
was observed. It's possible to extend support for more archs
if the patchset is welcome).
Mauricio Faria de Oliveira
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira
---
Documentation/DMA-attributes.txt | 17 +
include/linux/dma-attrs.h| 1 +
2 files changed, 18 insertions(+)
diff --git a/Documentation/DMA-attributes.txt b
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
---
arch/powerpc/kernel/iommu.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/kernel/iommu.c b/arch/powerpc/
1 - 100 of 105 matches
Mail list logo