d-e-s-o added a comment.

In D126839#3554570 <https://reviews.llvm.org/D126839#3554570>, @yonghong-song 
wrote:

> In D126839#3554474 <https://reviews.llvm.org/D126839#3554474>, @d-e-s-o wrote:
>
>> In D126839#3554125 <https://reviews.llvm.org/D126839#3554125>, 
>> @yonghong-song wrote:
>>
>>> Also let us merge it unti kernel side change is in reasonable shape.
>>
>> From what I understand it may take several days or some other not-precisely 
>> specified time until "nightly" builds, which we use in the BPF CI, pick up 
>> the change. Hence, I was under the impression that we should attempt to 
>> merge it sooner rather than later once it's ready to be merged. The kernel 
>> changes are in good enough shape to show feasibility, so I don't really see 
>> why not, but it's your call.
>
> We can merge after the new mechanism is acked (not patch getting merged) in 
> kernel review.

Sounds good.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126839/new/

https://reviews.llvm.org/D126839

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to