<html><head><style>pre,code,address {
margin: 0px;
}
h1,h2,h3,h4,h5,h6 {
margin-top: 0.2em;
margin-bottom: 0.2em;
}
ol,ul {
margin-top: 0em;
margin-bottom: 0em;
}
blockquote {
margin-top: 0em;
margin-bottom: 0em;
}
</style></head><body><div>On Tue, 2024-12-10 at 13:37 +0100, Greg Kroah-Hartman wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>t's a clue nonetheless.<br></div></blockquote><div><br></div><div>'git bisect' would be nice to run if possible...<br></div></blockquote><div><br></div><div>Done:</div><div><br></div><div>a) One (hopefully) small caveat :</div><div> the very first bisect hit the 6.12.2 bug of missing commit b23decf8ac91</div><div> ("sched: Initialize idle tasks only once").</div><div> I applied the commit and continued.</div><div><br></div><div>b) Result :</div><div><br></div><div>6ac269abab9ca5ae910deb2d3ca54351c3467e99 is the first bad commit</div><div>commit 6ac269abab9ca5ae910deb2d3ca54351c3467e99 (HEAD)</div><div>Author: Bingbu Cao <<a href="mailto:bingbu.cao@intel.com">bingbu.cao@intel.com</a>></div><div>Date: Wed Oct 16 15:53:01 2024 +0800</div><div><br></div><div> media: ipu6: not override the dma_ops of device in driver</div><div><br></div><div> [ Upstream commit daabc5c64703432c4a8798421a3588c2c142c51b ]</div><div><br></div><div> DMA ops are a helper for architectures and not for drivers to override the</div><div> DMA implementation. Driver should not override the DMA implementation.</div><div><br></div><div> This patch removes the dma_ops override from auxiliary device and adds</div><div> driver-internal helpers that use the actual DMA mapping APIs.</div><div><br></div><div> Fixes: 9163d83573e4 ("media: intel/ipu6: add IPU6 DMA mapping API and MMU table")</div><div> Signed-off-by: Bingbu Cao <<a href="mailto:bingbu.cao@intel.com">bingbu.cao@intel.com</a>></div><div> Reviewed-by: Christoph Hellwig <<a href="mailto:hch@lst.de">hch@lst.de</a>></div><div> [Sakari Ailus: Fix the commit message a little.]</div><div> Signed-off-by: Sakari Ailus <<a href="mailto:sakari.ailus@linux.intel.com">sakari.ailus@linux.intel.com</a>></div><div> Signed-off-by: Hans Verkuil <<a href="mailto:hverkuil@xs4all.nl">hverkuil@xs4all.nl</a>></div><div> Signed-off-by: Sasha Levin <<a href="mailto:sashal@kernel.org">sashal@kernel.org</a>></div><div><br></div><div> drivers/media/pci/intel/ipu6/ipu6-bus.c | 6 --</div><div> drivers/media/pci/intel/ipu6/ipu6-buttress.c | 21 +++++--</div><div> drivers/media/pci/intel/ipu6/ipu6-cpd.c | 18 +++---</div><div> drivers/media/pci/intel/ipu6/ipu6-dma.c | 195 +++++++++++++++++++++++++++++---------------------------------</div><div> drivers/media/pci/intel/ipu6/ipu6-dma.h | 34 ++++++++++-</div><div> drivers/media/pci/intel/ipu6/ipu6-fw-com.c | 14 ++---</div><div> 6 files changed, 156 insertions(+), 132 deletions(-)</div><div><br></div><div>c) git bisect log</div><div><br></div><div>git bisect start</div><div># status: waiting for both good and bad commits</div><div># good: [d390303b28dabbb91b2d32016a4f72da478733b9] Linux 6.12.1</div><div>git bisect good d390303b28dabbb91b2d32016a4f72da478733b9</div><div># status: waiting for bad commit, 1 good commit known</div><div># bad: [61baee2dc5341c936e7fa7b1ca33c5607868de69] Linux 6.12.4</div><div>git bisect bad 61baee2dc5341c936e7fa7b1ca33c5607868de69</div><div># bad: [2bc07714dc955a91d2923a440ea02c3cb3376b10] virtiofs: use pages instead of pointer for kernel direct IO</div><div>git bisect bad 2bc07714dc955a91d2923a440ea02c3cb3376b10</div><div># bad: [6f2648838ce1a9f2c432e84e38dec5d38412a551] wifi: wilc1000: Set MAC after operation mode</div><div>git bisect bad 6f2648838ce1a9f2c432e84e38dec5d38412a551</div><div># good: [e20117a105d4174c2daf4d6da17b92d757487e60] media: i2c: max96717: clean up on error in max96717_subdev_init()</div><div>git bisect good e20117a105d4174c2daf4d6da17b92d757487e60</div><div># bad: [74f21be9990a42dc2357bcf87a13e16c6998b90e] drm/vc4: hdmi: Avoid hang with debug registers when suspended</div><div>git bisect bad 74f21be9990a42dc2357bcf87a13e16c6998b90e</div><div># good: [dfdd0e8ff65dfc4183b622c3ffd8d8c5d572ca34] dt-bindings: cache: qcom,llcc: Fix X1E80100 reg entries</div><div>git bisect good dfdd0e8ff65dfc4183b622c3ffd8d8c5d572ca34</div><div># good: [b0cd515202cb31c89d7de46b103a7ff2de9de02b] media: ipu6: Fix DMA and physical address debugging messages for 32-bit</div><div>git bisect good b0cd515202cb31c89d7de46b103a7ff2de9de02b</div><div># bad: [3d48d0fbaaa74a04fb9092780a3f83dc4f3f8160] HID: hyperv: streamline driver probe to avoid devres issues</div><div>git bisect bad 3d48d0fbaaa74a04fb9092780a3f83dc4f3f8160</div><div># bad: [e3692460675bfe07f677711ead09a7a43934aa2f] pwm: Assume a disabled PWM to emit a constant inactive output</div><div>git bisect bad e3692460675bfe07f677711ead09a7a43934aa2f</div><div># bad: [11b0543efe54357b94dccb88dac2099c59358cd4] media: ipu6: remove architecture DMA ops dependency in Kconfig</div><div>git bisect bad 11b0543efe54357b94dccb88dac2099c59358cd4</div><div># bad: [6ac269abab9ca5ae910deb2d3ca54351c3467e99] media: ipu6: not override the dma_ops of device in driver</div><div>git bisect bad 6ac269abab9ca5ae910deb2d3ca54351c3467e99</div><div># first bad commit: [6ac269abab9ca5ae910deb2d3ca54351c3467e99] media: ipu6: not override the dma_ops of device in driver</div><div><br></div><div><br></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"></blockquote><div><br></div><div><span><pre>-- <br></pre><div><span style="background-color: inherit;">Gene</span></div><div><br></div></span></div></body></html>