From patchwork Tue Nov 24 23:57:54 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Jakub Kicinski X-Patchwork-Id: 331885 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-19.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, INCLUDES_CR_TRAILER, INCLUDES_PATCH, MAILING_LIST_MULTI, SPF_HELO_NONE, SPF_PASS, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EDAB4C2D0E4 for ; Tue, 24 Nov 2020 23:58:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7E83E21527 for ; Tue, 24 Nov 2020 23:58:02 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="hNbyw4F6" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726660AbgKXX6B (ORCPT ); Tue, 24 Nov 2020 18:58:01 -0500 Received: from mail.kernel.org ([198.145.29.99]:46078 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726594AbgKXX6B (ORCPT ); Tue, 24 Nov 2020 18:58:01 -0500 Received: from kicinski-fedora-PC1C0HJN.thefacebook.com (unknown [163.114.132.1]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 048642137B; Tue, 24 Nov 2020 23:57:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1606262280; bh=RC2bgkm+xUG7RQ1zFnMO2b06e8dIAZeplG9t7RM0NYQ=; h=From:To:Cc:Subject:Date:From; b=hNbyw4F68JHPXDvMGquOJ5eWiCIVsUW+LnV1ByJdKaAmUGvBHMN/lSID9O4Gw51ny xVJlz7sx/a+LcMsmauBx4mnb+9zpMIivET8PHrCzP4E/VNUSlx4OFDdbTMtMr61HG2 h4vfmtX2iZViEN+yFs3RjvSDpqOYKyUUGNLKIQyU= From: Jakub Kicinski To: davem@davemloft.net Cc: netdev@vger.kernel.org, brouer@redhat.com, f.fainelli@gmail.com, andrea.mayer@uniroma2.it, dsahern@gmail.com, stephen@networkplumber.org, Jakub Kicinski Subject: [PATCH net] Documentation: netdev-FAQ: suggest how to post co-dependent series Date: Tue, 24 Nov 2020 15:57:54 -0800 Message-Id: <20201124235755.159903-1-kuba@kernel.org> X-Mailer: git-send-email 2.26.2 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Make an explicit suggestion how to post user space side of kernel patches to avoid reposts when patchwork groups the wrong patches. Signed-off-by: Jakub Kicinski Reviewed-by: Florian Fainelli --- Documentation/networking/netdev-FAQ.rst | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/Documentation/networking/netdev-FAQ.rst b/Documentation/networking/netdev-FAQ.rst index 21537766be4d..553eda8da9c7 100644 --- a/Documentation/networking/netdev-FAQ.rst +++ b/Documentation/networking/netdev-FAQ.rst @@ -254,6 +254,26 @@ you will have done run-time testing specific to your change, but at a minimum, your changes should survive an ``allyesconfig`` and an ``allmodconfig`` build without new warnings or failures. +Q: How do I post corresponding changes to user space components? +---------------------------------------------------------------- +A: Kernel patches often come with support in user space tooling +(e.g. `iproute2`). It's best to post both kernel and user space +code at the same time, so that reviewers have a chance to see how +user space side looks when reviewing kernel code. +If user space tooling lives in a separate repository kernel and user +space patches should form separate series (threads) when posted +to the mailing list, e.g.:: + + [PATCH net-next 0/3] net: some feature cover letter + └─ [PATCH net-next 1/3] net: some feature prep + └─ [PATCH net-next 2/3] net: some feature do it + └─ [PATCH net-next 3/3] selftest: net: some feature + + [PATCH iproute2-next] ip: add support for some feature + +Posting as one thread is discouraged because it confuses patchwork +(as of patchwork 2.2.2). + Q: Any other tips to help ensure my net/net-next patch gets OK'd? ----------------------------------------------------------------- A: Attention to detail. Re-read your own work as if you were the