From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wg0-f50.google.com (mail-wg0-f50.google.com [74.125.82.50]) by dpdk.org (Postfix) with ESMTP id CB7877EB0 for ; Thu, 4 Dec 2014 16:33:02 +0100 (CET) Received: by mail-wg0-f50.google.com with SMTP id k14so23031432wgh.9 for ; Thu, 04 Dec 2014 07:33:02 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding:content-type; bh=WwnhlIHASZghKDiKREv92w26Br/SWxsJgzJjzTH+em8=; b=Gddem4QFEZ/cOTArM1LEEG2elqeEb6LHgP2jHjYq+DevQcgLiHiiY5g27ayvGi1iCx I+kLc8qBURi91WIerTSPYKvPq0NJfWTdf/f2jDGMVEGFAH3rrxSP7RKcJrAxZUtS7UN1 L5kxEXiuZnQ4LrsM///lQEZ+reb7vz1l94YuUjDC8RCw8dmZ/vf9+Gd6SX5TWgyWbmIm 8UqvaDl2yWKNif3coG7mF7OHWzX3T1RRbV/acc6LxMtxtfDVkza3z4dy1OMJ5feEhw7D my0vfSPT7GiWYf2H9tL3opEYMuvTEVI+R4srKL6DyTgqaoSlskzXVjiqqsvEgtpglIqq SIlA== X-Gm-Message-State: ALoCoQkBG2nsqJvvONlzrce2nsic8B4vdlOsPP820eSVtvdS9k27fSN5P2Tt8Y5TsH4NnnED6dui X-Received: by 10.180.109.45 with SMTP id hp13mr22053998wib.4.1417707182640; Thu, 04 Dec 2014 07:33:02 -0800 (PST) Received: from xps13.localnet (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by mx.google.com with ESMTPSA id nj9sm41608480wic.10.2014.12.04.07.33.01 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 04 Dec 2014 07:33:01 -0800 (PST) From: Thomas Monjalon To: "Chen, Jing D" Date: Thu, 04 Dec 2014 16:32:37 +0100 Message-ID: <1575841.on6VhD6YKG@xps13> Organization: 6WIND User-Agent: KMail/4.14.3 (Linux/3.17.4-1-ARCH; KDE/4.14.3; x86_64; ; ) In-Reply-To: <4341B239C0EFF9468EE453F9E9F4604D0162B469@shsmsx102.ccr.corp.intel.com> References: <1417686605-6778-1-git-send-email-jing.d.chen@intel.com> <2122900.sczHU7BWUg@xps13> <4341B239C0EFF9468EE453F9E9F4604D0162B469@shsmsx102.ccr.corp.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH] i40e: Fix a vlan bug X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Dec 2014 15:33:03 -0000 2014-12-04 14:29, Chen, Jing D: > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > 2014-12-04 10:30, Chen, Jing D: > > > As I don't know what commit he is based on, I'd like to generate a new > > patch with latest dpdk repo. > > > > There's something wrong here. You rework a patch and you don't know what > > was the current status but you expect that the reviewers can understand it > > better than you? > > You don't understand me. Please read my above words again. Yes there probably is a misunderstanding. > As I said, he is in vacation, I came to fix problem. I know exactly what's the problem. So, I used simple way. So Huawei was trying to fix the bug and you suggest another way to fix it. But you didn't explain why your fix is better than the previous one. And we don't know if it's the continuation of his work or not. If you are trying to fix exactly the same problem, incrementing the version number of the patch makes clear that previous version doesn't need to be reviewed, reworked or applied. In patchwork language, it supersedes the previous patch which won't appear anymore. > > You are breaking all the elementary rules of patch management. > > Please kindly list all the elementary rules of patch management, please. > If possible, can you post it somewhere so other new guys can find and follow? They are explained in http://dpdk.org/dev#send. That's the ones I've enumerated in my first email: - changelog - increment version number (v5 here) - use --in-reply-to > > We have currently 2 fixes pending for the same bug. To sum it up, we need: 1) a review 2) an agreement that the Huawei's fix is superseded by this one Thank you -- Thomas > > PS: please don't top post. > > I apologized for top post. > > > > > -- > > Thomas > > > > > > -----Original Message----- > > > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > > > Sent: Thursday, December 04, 2014 6:26 PM > > > > To: Chen, Jing D > > > > Cc: dev@dpdk.org; Qiu, Michael > > > > Subject: Re: [dpdk-dev] [PATCH] i40e: Fix a vlan bug > > > > > > > > 2014-12-04 10:18, Qiu, Michael: > > > > > Hi Mark, > > > > > > > > > > I think Huawei (huawei.xie@intel.com) has one patch set to fix this > > issue. > > > > > > > > > > If your patch is totally different with him: > > > > > > > > > > [dpdk-dev] [PATCH v4 0/2] lib/librte_pmd_i40e: set vlan filter fix > > > > > > > > > > please ignore my comments :) > > > > > > > > > > But you both calculation are different. > > > > > > > > Yes, please Jing (Mark), if you reworked the v4 patch, it would > > > > clearer to have a changelog, to name it v5 and to insert it in the > > > > previous thread with --in-reply-to. > > > > At the moment, both patches block each other. > > > > > > > > -- > > > > Thomas >