As part of the attempt to go upstream, this "hubcap" guy you see in the comments worked on a thing that changes 64bit userspace handles back and forth into 128bit kernel handles... we did this because one day, when we have orangefs3, we will be using 128bit uuid-derived handles, and we believe it is our responsibility to not break the upstream kernel module. Anywho, I bet you are right Al, he messed up this part of it... I'll look and see if that is really so, and get it fixed. -Mike "hubcap" On Thu, Feb 18, 2016 at 3:52 PM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote: > On Thu, Feb 18, 2016 at 03:38:26PM -0500, Mike Marshall wrote: >> WARNING: CPU: 0 PID: 1216 at fs/orangefs/devorangefs-req.c:423 >> set_op_state_serviced: op:ffff880011078000: process:pvfs2-client-co state -> 4 >> service_operation: wait_for_matching_downcall returned 0 for ffff880011078000 >> service_operation orangefs_create returning: 0 for ffff880011078000 >> orangefs_create: BENCHS.LWP: >> handle:00000000-0000-0000-0000-000000000000: fsid:0: >> new_op:ffff880011078000: ret:0: > > Smells like retry hitting EEXIST and package_downcall_members() treatment of > that case doesn't set create.refn at all - used to, but that code is commented > out. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
