Re: linux-next: manual merge of the block tree with the jc_docs tree

[Christoph Hellwig <hch@lst.de>]

Next time it would be really helpful if fixups to nvme documentation
went through the nvme/block tree..

On Tue, Jul 09, 2019 at 10:08:30AM +1000, Stephen Rothwell wrote:
> Hi all,
> 
> On Tue, 25 Jun 2019 14:42:26 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > Today's linux-next merge of the block tree got a conflict in:
> > 
> >   Documentation/fault-injection/nvme-fault-injection.txt
> > 
> > between commit:
> > 
> >   10ffebbed550 ("docs: fault-injection: convert docs to ReST and rename to *.rst")
> > 
> > from the jc_docs tree and commit:
> > 
> >   7e31d8215fd8 ("Documentation: nvme: add an example for nvme fault injection")
> > 
> > from the block tree.
> > 
> > I fixed it up (I removed the file and applied the following patch) and
> > can carry the fix as necessary. This is now fixed as far as linux-next
> > is concerned, but any non trivial conflicts should be mentioned to your
> > upstream maintainer when your tree is submitted for merging.  You may
> > also want to consider cooperating with the maintainer of the conflicting
> > tree to minimise any particularly complex conflicts.
> > 
> > From: Stephen Rothwell <sfr@canb.auug.org.au>
> > Date: Tue, 25 Jun 2019 14:39:46 +1000
> > Subject: [PATCH] Documentation: nvme: fix for change to rst
> > 
> > Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> > ---
> >  .../fault-injection/nvme-fault-injection.rst  | 58 +++++++++++++++++++
> >  1 file changed, 58 insertions(+)
> > 
> > diff --git a/Documentation/fault-injection/nvme-fault-injection.rst b/Documentation/fault-injection/nvme-fault-injection.rst
> > index bbb1bf3e8650..cdb2e829228e 100644
> > --- a/Documentation/fault-injection/nvme-fault-injection.rst
> > +++ b/Documentation/fault-injection/nvme-fault-injection.rst
> > @@ -118,3 +118,61 @@ Message from dmesg::
> >      cpu_startup_entry+0x6f/0x80
> >      start_secondary+0x187/0x1e0
> >      secondary_startup_64+0xa5/0xb0
> > +
> > +Example 3: Inject an error into the 10th admin command
> > +------------------------------------------------------
> > +
> > +::
> > +
> > +  echo 100 > /sys/kernel/debug/nvme0/fault_inject/probability
> > +  echo 10 > /sys/kernel/debug/nvme0/fault_inject/space
> > +  echo 1 > /sys/kernel/debug/nvme0/fault_inject/times
> > +  nvme reset /dev/nvme0
> > +
> > +Expected Result::
> > +
> > +  After NVMe controller reset, the reinitialization may or may not succeed.
> > +  It depends on which admin command is actually forced to fail.
> > +
> > +Message from dmesg::
> > +
> > +  nvme nvme0: resetting controller
> > +  FAULT_INJECTION: forcing a failure.
> > +  name fault_inject, interval 1, probability 100, space 1, times 1
> > +  CPU: 0 PID: 0 Comm: swapper/0 Not tainted 5.2.0-rc2+ #2
> > +  Hardware name: MSI MS-7A45/B150M MORTAR ARCTIC (MS-7A45), BIOS 1.50 04/25/2017
> > +  Call Trace:
> > +   <IRQ>
> > +   dump_stack+0x63/0x85
> > +   should_fail+0x14a/0x170
> > +   nvme_should_fail+0x38/0x80 [nvme_core]
> > +   nvme_irq+0x129/0x280 [nvme]
> > +   ? blk_mq_end_request+0xb3/0x120
> > +   __handle_irq_event_percpu+0x84/0x1a0
> > +   handle_irq_event_percpu+0x32/0x80
> > +   handle_irq_event+0x3b/0x60
> > +   handle_edge_irq+0x7f/0x1a0
> > +   handle_irq+0x20/0x30
> > +   do_IRQ+0x4e/0xe0
> > +   common_interrupt+0xf/0xf
> > +   </IRQ>
> > +  RIP: 0010:cpuidle_enter_state+0xc5/0x460
> > +  Code: ff e8 8f 5f 86 ff 80 7d c7 00 74 17 9c 58 0f 1f 44 00 00 f6 c4 02 0f 85 69 03 00 00 31 ff e8 62 aa 8c ff fb 66 0f 1f 44 00 00 <45> 85 ed 0f 88 37 03 00 00 4c 8b 45 d0 4c 2b 45 b8 48 ba cf f7 53
> > +  RSP: 0018:ffffffff88c03dd0 EFLAGS: 00000246 ORIG_RAX: ffffffffffffffdc
> > +  RAX: ffff9dac25a2ac80 RBX: ffffffff88d53760 RCX: 000000000000001f
> > +  RDX: 0000000000000000 RSI: 000000002d958403 RDI: 0000000000000000
> > +  RBP: ffffffff88c03e18 R08: fffffff75e35ffb7 R09: 00000a49a56c0b48
> > +  R10: ffffffff88c03da0 R11: 0000000000001b0c R12: ffff9dac25a34d00
> > +  R13: 0000000000000006 R14: 0000000000000006 R15: ffffffff88d53760
> > +   cpuidle_enter+0x2e/0x40
> > +   call_cpuidle+0x23/0x40
> > +   do_idle+0x201/0x280
> > +   cpu_startup_entry+0x1d/0x20
> > +   rest_init+0xaa/0xb0
> > +   arch_call_rest_init+0xe/0x1b
> > +   start_kernel+0x51c/0x53b
> > +   x86_64_start_reservations+0x24/0x26
> > +   x86_64_start_kernel+0x74/0x77
> > +   secondary_startup_64+0xa4/0xb0
> > +  nvme nvme0: Could not set queue count (16385)
> > +  nvme nvme0: IO queues not created
> > -- 
> > 2.20.1
> 
> I am still getting this conflict (the commit ids may have changed).
> Just a reminder in case you think Linus may need to know.
> 
> -- 
> Cheers,
> Stephen Rothwell


---end quoted text---