On Tue, 16 Oct 2018 09:46:07 +0300 Ceco <tz.stoyanov@xxxxxxxxx> wrote: > ping > Hi Steven, is there a problem with this patch ? Yordan needs these > changes, he wants to use the new API in KernelShark. Just woke up, so I haven't had a chance to look at the patch. But the one thing that is wrong with it, is that a v2 shouldn't be a reply to a v1. That is, it should start a new thread. With thousands of emails in my Inbox, I look at email threads (as do many kernel developers), and if there's a patch within the thread, it is commonly missed. I will say that I did mark it to look at it, so I did see it. But other patches came in during that time, and I sorted my inbox by thread, which did hide the patch again. I'll take a look at it later today. Also, don't be afraid to send a ping if there's no response for a week, as that usually means it was lost in the flood. Thanks! -- Steve > > On Fri, Aug 31, 2018 at 9:27 AM Tzvetomir Stoyanov (VMware) > <tz.stoyanov@xxxxxxxxx> wrote: > >
![]() |