Hi Jeff, On Sat, 2013-03-23 at 00:32 -0700, Jeff Leung wrote: > Sure, not a problem. > > > Thanks for the detailed bug description. > > I'll provide the tcpdump's of the entire conversation between the > target and the initiators on a separate off-list message. For > reference I'll provide a dump that successfully validated and a dump > that didn't pass validation. > These are exactly what I wanted to see, thanks. > > I'll try reproducing with PR-OUT PREEMPT on my side, but also > posting a wire- > > shark capture to verify this output against would be very helpful. > > It never hurts to verify with your suspicions. > > > Btw, thanks for testing with v3.2 to verify that things are working > as > > expected. > > I'll see if I can revert those 2 changes to see if the code is the > problem. This will be easier said that done, given the number of locations that the sense_reason_t conversion patch ended up changing.. ;) > > > So starting with v3.8-rc1 code, there where two PR related changes > that > > went into mainline: > > > > target: pass sense_reason as a return value > > http://git.kernel.org/cgit/linux/kernel/git/nab/target- > > > pending.git/commit/drivers/target/target_core_pr.c?id=de103c93aff0bed0a > > e984274e5dc8b95899badab > > > > target: simplify reservations code > > http://git.kernel.org/cgit/linux/kernel/git/nab/target- > > > pending.git/commit/drivers/target/target_core_pr.c?id=d977f4377fbc396b8 > > 88e12fdb3b13118b09ca7db > > > > I've compiled the 3.7.10 kernel and that validates just fine. > Great, thanks for verifying this is indeed a >= v3.8 regression. Thank you, --nab -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
