From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966433AbdJRFpJ (ORCPT ); Wed, 18 Oct 2017 01:45:09 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:39326 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965804AbdJRFpC (ORCPT ); Wed, 18 Oct 2017 01:45:02 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 7E5E56086B Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=kvalo@codeaurora.org From: Kalle Valo To: Kees Cook Cc: "David S. Miller" , Network Development , Thomas Gleixner , LKML , linux-wireless Subject: Re: [PATCH 00/58] networking: Convert timers to use timer_setup() References: <1508200182-104605-1-git-send-email-keescook@chromium.org> <87mv4pvowa.fsf@purkki.adurom.net> Date: Wed, 18 Oct 2017 08:44:57 +0300 In-Reply-To: (Kees Cook's message of "Tue, 17 Oct 2017 12:47:26 -0700") Message-ID: <87h8ux2ena.fsf@codeaurora.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Kees Cook writes: > On Tue, Oct 17, 2017 at 7:18 AM, Kalle Valo wrote: >> + linux-wireless >> >> Hi Kees, >> >> Kees Cook writes: >> >>> This is the current set of outstanding networking patches to perform >>> conversions to the new timer interface (rebased to -next). This is not >>> all expected conversions, but it contains everything needed in networking >>> to eliminate init_timer(), and all the non-standard setup_*_timer() uses. >> >> So this also includes patches which I had queued for >> wireless-drivers-next: >> >> https://patchwork.kernel.org/patch/9986253/ >> https://patchwork.kernel.org/patch/9986245/ >> >> And looking at patchwork[1] I have even more timer_setup() related >> patches from you. It would be really helpful if you could clearly >> document to which tree you want the patches to be applied. I don't care > > Hi! Sorry about that. It's been a bit tricky to juggle everything. Yeah, I understand. >> if it's net-next or wireless-drivers-next as long as it's not the both >> (meaning that both Dave and me apply the same patch, which would be >> bad). The thing is that I really do not have time to figure out for >> every patch via which tree it's supposed to go. > > Which split is preferred? I had been trying to separate wireless from > the rest of net (but missed some cases). So what we try to follow is that I apply all patches for drivers/net/wireless to my wireless-drivers trees, with exception of Johannes taking mac80211_hwsim.c patches to his mac80211 trees. And Johannes of course takes all patches for net/wireless and net/mac80211. So in general I prefer that I take all drivers/net/wireless patches and make it obvious for Dave that he can ignore those patches (not mix wireless-drivers and net patches into same set etc). But like I said, it's ok to push API changes like these via Dave's net trees as well if you want (and if Dave is ok with that). The chances of conflicts is low, and if there are be any those would be easy to fix either by me or Dave. >> For now I'll just drop all your timer_setup() related patches from my >> queue and I'll assume Dave will take those. Ok? >> >> [1] https://patchwork.kernel.org/project/linux-wireless/list/ > > I guess I'll wait to see what Dave says. Ok, I don't drop the patches from my queue quite yet then. -- Kalle Valo