From mboxrd@z Thu Jan 1 00:00:00 1970 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=-5.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_MUTT 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 BE485C282CE for ; Wed, 22 May 2019 14:52:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9DB7620879 for ; Wed, 22 May 2019 14:52:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729710AbfEVOwi (ORCPT ); Wed, 22 May 2019 10:52:38 -0400 Received: from stargate.chelsio.com ([12.32.117.8]:43103 "EHLO stargate.chelsio.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728450AbfEVOwi (ORCPT ); Wed, 22 May 2019 10:52:38 -0400 Received: from localhost ([10.193.186.161]) by stargate.chelsio.com (8.13.8/8.13.8) with ESMTP id x4MEqYxG031431; Wed, 22 May 2019 07:52:35 -0700 Date: Wed, 22 May 2019 20:22:37 +0530 From: Vishal Kulkarni To: David Miller Cc: "netdev@vger.kernel.org" , Nirranjan Kirubaharan , Indranil Choudhury , dt Subject: Re: [PATCH net-next] cxgb4: Revert "cxgb4: Remove SGE_HOST_PAGE_SIZE dependency on page size" Message-ID: <20190522145235.GA8075@chelsio.com> References: <1558410122-29341-1-git-send-email-vishal@chelsio.com> <20190521.132331.1475679105999327536.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190521.132331.1475679105999327536.davem@davemloft.net> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Hi David, On Wednesday, May 05/22/19, 2019 at 01:53:31 +0530, David Miller wrote: > From: Vishal Kulkarni > Date: Tue, 21 May 2019 09:12:02 +0530 > > > This reverts commit 2391b0030e241386d710df10e53e2cfc3c5d4fc1 > > SGE's BAR2 Doorbell/GTS Page Size is now interpreted correctly in the > > firmware itself by using actual host page size. Hence previous commit > > needs to be reverted. > > > > Signed-off-by: Vishal Kulkarni > > Really confusing. > > First of all, I see a lot of cxgb4 patch submissions targetting net-next that > are really legitimate bug fixes. > > Are you only targetting net-next to be "on the safe side" because you are > unsure of the 'net' rules? Please don't do that. If it's a bug fix, send > it to 'net' as appropriate. My apologies. I will retarget to 'net' and resend. > > Second, so what happens to people running older firmware? Have you made a > completely incompatible change to the firmware behavior? If so, you have > to version check the firmware and use the correct interpretation based upon > how the firmware verion interprets things. The fix in the commit 2391b0030e241386d710df10e53e2cfc3c5d4fc1 is wrong and introduced regression. The correct fix is done in the firmware and hence it needs to be reverted. The changes done in firmware is compatible. > > Thanks.