> On 28 Apr 2017, at 17:14, Alexander Graf <agraf@xxxxxxx> wrote:
>
>
>
> On 28.04.17 09:57, Yan, Zheng wrote:
>>
>>> On 28 Apr 2017, at 00:34, Alexander Graf <agraf@xxxxxxx> wrote:
>>>
>>> The file open flags (O_foo) are platform specific and should never go
>>> out to an interface that is not local to the system.
>>>
>>> Unfortunately these flags have leaked out onto the wire in the cephfs
>>> implementation. That lead to bogus flags getting transmitted on ppc64.
>>>
>>> This patch converts the kernel view of flags to the ceph view of file
>>> open flags.
>>>
>>> Fixes: 124e68e74 ("ceph: file operations")
>>> Signed-off-by: Alexander Graf <agraf@xxxxxxx>
>>>
>>> —
>>
>> I removed the "unused open flags” dout and applied the patch. Thank you for tracking down and fixing the bug.
>
> I actually put that in on purpose, in case anyone in 2 years tries to find out why a particular flag doesn't get populated :).
>
Ok, I will put it back.
Regards
Yan, Zheng
>
> Alex
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
