On Fri, Feb 10, 2023 at 09:53:35AM -0800, Dan Williams wrote: > Dan Williams wrote: > > Changes since v1: [1] > > - Add a fix for memdev removal racing port removal (found by unit tests) > > - Add a fix to unwind region target list updates on error in > > cxl_region_attach() (Jonathan) > > - Move the passthrough decoder fix for submission for v6.2-final (Greg) > > - Fix wrong initcall for cxl_core (Gregory and Davidlohr) > > - Add an endpoint decoder state (CXL_DECODER_STATE_AUTO) to replace > > the flag CXL_DECODER_F_AUTO (Jonathan) > > - Reflow cmp_decode_pos() to reduce levels of indentation (Jonathan) > > - Fix a leaked reference count in cxl_add_to_region() (Jonathan) > > - Make cxl_add_to_region() return an error (Jonathan) > > - Fix several spurious whitespace changes (Jonathan) > > - Cleanup some spurious changes from the tools/testing/cxl update > > (Jonathan) > > - Test for == CXL_CONFIG_COMMIT rather than >= CXL_CONFIG_COMMIT > > (Jonathan) > > - Add comment to clarify device_attach() return code expectation in > > cxl_add_to_region() (Jonathan) > > - Add a patch to split cxl_port_probe() into switch and endpoint port > > probe calls (Jonathan) > > - Collect reviewed-by and tested-by tags > > > > [1]: http://lore.kernel.org/r/167564534874.847146.5222419648551436750.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > > > > --- > > Cover letter same as v1 > > Thanks for all the review so far! The outstanding backlog is still too > high to definitively say this will make v6.3: > > http://lore.kernel.org/r/167601992789.1924368.8083994227892600608.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > http://lore.kernel.org/r/167601996980.1924368.390423634911157277.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > http://lore.kernel.org/r/167601999378.1924368.15071142145866277623.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > http://lore.kernel.org/r/167601999958.1924368.9366954455835735048.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > http://lore.kernel.org/r/167602000547.1924368.11613151863880268868.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > http://lore.kernel.org/r/167602001107.1924368.11562316181038595611.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > http://lore.kernel.org/r/167602002771.1924368.5653558226424530127.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > http://lore.kernel.org/r/167602003896.1924368.10335442077318970468.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > > ...what I plan to do is provisionally include it in -next and then make > a judgement call next Friday. > > I am encouraged by Fan's test results: > > http://lore.kernel.org/r/20230208173720.GA709329@bgt-140510-bm03 > > ...and am reminded that there are some non-trivial TODOs pent up behind > region enumeration: > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=9ea4dcf49878 I have also been testing with generally positive results, I've just been traveling so internet has been spotty. I'll post a full breakdown of what i've been doing on monday, including poking the new kernel boot parameter and such. ~Gregory
