On 09/17/2012 11:36 PM, Arnaldo Carvalho de Melo wrote: > Em Tue, Sep 11, 2012 at 04:27:17PM +0200, Jiri Olsa escreveu: >> On Mon, Sep 10, 2012 at 03:53:51PM +0800, Yan, Zheng wrote: >> >> Would it be possible to have all '*add_event' more obvious for usage. >> Also following code is duplicated after each call of __add_event: >> >> evsel = __add_event(idx, &attr, pmu_event_name(head_config), >> pmu->cpus); >> if (!evsel) { >> *idx = orig_idx; >> free_event_list(list); >> return -ENOMEM; >> } >> list_add_tail(&evsel->node, list); > > Any update here? Yan, did you look at folding Jiri's patch? I'm merging > the first bit, the kernel part, that was Acked by Ingo and Peter, > waiting on this part, looking at 2/3 now >
Hi, This patch still need some improvement, I don't expect it to get merged. Please merged the patch 1 and patch 2 first. I will send a new one soon. Regards Yan, Zheng -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/