Hi, I sometimes download patch series off of public inbox hosted servers to apply with git-am. Occasionally I have found that these do not apply cleanly because the thread is not sorted in patch order.
For an example, see https://lore.kernel.org/lkml/[email protected]/ This thread shows up and the links are all in order on the HTML view, but if I download the t.mbox and split it using git mailsplit, I get the following: $ git mailsplit t.mbox -osplit $ cd split $ for f in *; do echo $f ; grep '^Subject:' $f; done > 0001 > Subject: [PATCH v2 0/7] vfio: Interrupt eventfd hardening > 0002 > Subject: [PATCH v2 2/7] vfio/pci: Lock external INTx masking ops > 0003 > Subject: [PATCH v2 3/7] vfio: Introduce interface to flush virqfd inject > workqueue > 0004 > Subject: [PATCH v2 1/7] vfio/pci: Disable auto-enable of exclusive INTx IRQ > 0005 > Subject: [PATCH v2 4/7] vfio/pci: Create persistent INTx handler > 0006 > Subject: [PATCH v2 6/7] vfio/platform: Create persistent IRQ handlers > 0007 > Subject: [PATCH v2 5/7] vfio/platform: Disable virqfds on cleanup > 0008 > Subject: [PATCH v2 7/7] vfio/fsl-mc: Block calling interrupt handler without > trigger > 0009 > Subject: RE: [PATCH v2 5/7] vfio/platform: Disable virqfds on cleanup > 0010 > Subject: RE: [PATCH v2 6/7] vfio/platform: Create persistent IRQ handlers > 0011 > Subject: RE: [PATCH v2 7/7] vfio/fsl-mc: Block calling interrupt handler > 0012 > Subject: Re: [PATCH v2 1/7] vfio/pci: Disable auto-enable of exclusive INTx > 0013 > Subject: Re: [PATCH v2 2/7] vfio/pci: Lock external INTx masking ops > 0014 > Subject: Re: [PATCH v2 3/7] vfio: Introduce interface to flush virqfd inject > 0015 > Subject: Re: [PATCH v2 4/7] vfio/pci: Create persistent INTx handler > 0016 > Subject: Re: [PATCH v2 5/7] vfio/platform: Disable virqfds on cleanup > 0017 > Subject: Re: [PATCH v2 6/7] vfio/platform: Create persistent IRQ handlers > 0018 > Subject: Re: [PATCH v2 7/7] vfio/fsl-mc: Block calling interrupt handler > 0019 > Subject: Re: [PATCH v2 1/7] vfio/pci: Disable auto-enable of exclusive INTx > 0020 > Subject: Re: [PATCH v2 6/7] vfio/platform: Create persistent IRQ handlers > I'm not really sure why messages appear in this order in the t.mbox file. The messages appear sorted properly when viewing the thread via the next and previous links and in the summary of links at the HTML display.. Any ideas? It is rather frustrating to have to realize that an mbox is not sorted and I have to split and sort it manually... I tried looking through the the public-inbox to see if I could figure it out but its not clear to me. Maybe its because of how thread_html uses walk_thread to go in a particular order where was the mbox code seems to just iterate over messages in the thread and go in essentially a random order? Thanks, Jake
