Thread (9 messages) 9 messages, 2 authors, 2018-02-07

RE: [PATCH 2/4] MIPS: generic: Fix ranchu_of_match[] termination

From: Aleksandar Markovic <Aleksandar.Markovic@mips.com>
Date: 2018-02-07 16:44:46

________________________________________
From: James Hogan [jhogan@kernel.org]
Sent: Friday, February 2, 2018 11:14 PM
To: Ralf Baechle; linux-mips@linux-mips.org
Cc: Aleksandar Markovic; Goran Ferenc; Miodrag Dinic; James Hogan
Subject: [PATCH 2/4] MIPS: generic: Fix ranchu_of_match[] termination

ranchu_of_match[] has no terminating element to end the search for a
matching compatible string when the first and only element does not
match, so add one now.

Fixes: f2d0b0d5c171 ("MIPS: ranchu: Add Ranchu as a new generic-based board")
Signed-off-by: James Hogan <jhogan@kernel.org>
Reviewed-by: Paul Burton <redacted>
Reviewed-by: Matt Redfearn <redacted>
Cc: Ralf Baechle <redacted>
Cc: Miodrag Dinic <miodrag.dinic@mips.com>
Cc: Goran Ferenc <redacted>
Cc: Aleksandar Markovic <aleksandar.markovic@mips.com>
Cc: linux-mips@linux-mips.org
---
 arch/mips/generic/board-ranchu.c | 1 +
 1 file changed, 1 insertion(+)
Acked-by: Aleksandar Markovic <aleksandar.markovic@mips.com>
Acked-by: Miodrag Dinic <miodrag.dinic@mips.com>
From Aleksandar.Markovic@mips.com Wed Feb  7 17:45:00 2018
Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 07 Feb 2018 17:45:08 +0100 (CET)
Received: from 9pmail.ess.barracuda.com ([64.235.154.210]:57338 "EHLO
        9pmail.ess.barracuda.com" rhost-flags-OK-OK-OK-OK)
        by eddie.linux-mips.org with ESMTP id S23992692AbeBGQoVTfFKV convert rfc822-to-8bit
        (ORCPT <rfc822;linux-mips@linux-mips.org>);
        Wed, 7 Feb 2018 17:44:21 +0100
Received: from MIPSMAIL01.mipstec.com (mailrelay.mips.com [12.201.5.28]) by mx1402.ess.rzc.cudaops.com (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NO); Wed, 07 Feb 2018 16:43:35 +0000
Received: from MIPSMAIL01.mipstec.com ([fe80::5c93:1f20:524d:a563]) by
 MIPSMAIL01.mipstec.com ([fe80::5c93:1f20:524d:a563%13]) with mapi id
 14.03.0361.001; Wed, 7 Feb 2018 08:27:08 -0800
From:   Aleksandar Markovic <Aleksandar.Markovic@mips.com>
To:     James Hogan <jhogan@kernel.org>,
        Ralf Baechle [off-list ref],
        "linux-mips@linux-mips.org" [off-list ref]
CC:     Goran Ferenc <redacted>,
        Miodrag Dinic [off-list ref]
Subject: RE: [PATCH 3/4] MIPS: generic: Fix Makefile alignment
Thread-Topic: [PATCH 3/4] MIPS: generic: Fix Makefile alignment
Thread-Index: AQHTnHNL3+vM+XZkm0iQBtm1ToHHq6OZJp+i
Date:   Wed, 7 Feb 2018 16:27:08 +0000
Message-ID: [ref]
References: [ref],[ref]
In-Reply-To: [ref]
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-originating-ip: [82.117.201.26]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
X-BESS-ID: 1518021606-321458-21852-1632-11
X-BESS-VER: 2018.1-r1801291959
X-BESS-Apparent-Source-IP: 12.201.5.28
X-BESS-Outbound-Spam-Score: 1.10
X-BESS-Outbound-Spam-Report: Code version 3.2, rules version 3.2.2.189770
        Rule breakdown below
         pts rule name              description
        ---- ---------------------- --------------------------------
        0.50 BSF_RULE7568M          META: Custom Rule 7568M 
        0.00 BSF_BESS_OUTBOUND      META: BESS Outbound 
        0.60 MARKETING_SUBJECT      HEADER: Subject contains popular marketing words 
X-BESS-Outbound-Spam-Status: SCORE=1.10 using account:ESS59374 scores of KILL_LEVEL=7.0 tests=BSF_RULE7568M, BSF_BESS_OUTBOUND, MARKETING_SUBJECT
X-BESS-BRTS-Status: 1
Return-Path: [off-list ref]
X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0)
X-Orcpt: rfc822;linux-mips@linux-mips.org
Original-Recipient: rfc822;linux-mips@linux-mips.org
X-archive-position: 62458
X-ecartis-version: Ecartis v1.0.0
Sender: linux-mips-bounce@linux-mips.org
Errors-to: linux-mips-bounce@linux-mips.org
X-original-sender: Aleksandar.Markovic@mips.com
Precedence: bulk
List-help: <mailto:ecartis@linux-mips.org?Subject=help>
List-unsubscribe: <mailto:ecartis@linux-mips.org?subject=unsubscribe%20linux-mips>
List-software: Ecartis version 1.0.0
List-Id: linux-mips <linux-mips.eddie.linux-mips.org>
X-List-ID: linux-mips <linux-mips.eddie.linux-mips.org>
List-subscribe: <mailto:ecartis@linux-mips.org?subject=subscribe%20linux-mips>
List-owner: <mailto:ralf@linux-mips.org>
List-post: <mailto:linux-mips@linux-mips.org>
List-archive: <http://www.linux-mips.org/archives/linux-mips/>
X-list: linux-mips
________________________________________
From: James Hogan [jhogan@kernel.org]
Sent: Friday, February 2, 2018 11:14 PM
To: Ralf Baechle; linux-mips@linux-mips.org
Cc: Aleksandar Markovic; Goran Ferenc; Miodrag Dinic; James Hogan
Subject: [PATCH 3/4] MIPS: generic: Fix Makefile alignment

Fix whitespace of generic platform Makefile so that obj-y values align.

Fixes: f2d0b0d5c171 ("MIPS: ranchu: Add Ranchu as a new generic-based board")
Signed-off-by: James Hogan <jhogan@kernel.org>
Reviewed-by: Paul Burton <redacted>
Reviewed-by: Matt Redfearn <redacted>
Cc: Ralf Baechle <redacted>
Cc: Miodrag Dinic <miodrag.dinic@mips.com>
Cc: Goran Ferenc <redacted>
Cc: Aleksandar Markovic <aleksandar.markovic@mips.com>
Cc: linux-mips@linux-mips.org
---
 arch/mips/generic/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Aleksandar Markovic <aleksandar.markovic@mips.com>
Acked-by: Miodrag Dinic <miodrag.dinic@mips.com>
From shannon.nelson@oracle.com Wed Feb  7 18:02:37 2018
Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 07 Feb 2018 18:02:44 +0100 (CET)
Received: from userp2120.oracle.com ([156.151.31.85]:52984 "EHLO
        userp2120.oracle.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org
        with ESMTP id S23990434AbeBGRCg6aE-V (ORCPT
        <rfc822;linux-mips@linux-mips.org>); Wed, 7 Feb 2018 18:02:36 +0100
Received: from pps.filterd (userp2120.oracle.com [127.0.0.1])
        by userp2120.oracle.com (8.16.0.22/8.16.0.22) with SMTP id w17Gq6ZQ085370;
        Wed, 7 Feb 2018 17:02:24 GMT
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=subject : to : cc :
 references : from : message-id : date : mime-version : in-reply-to :
 content-type : content-transfer-encoding; s=corp-2017-10-26;
 bh=t8qkj05xE5jMGyqs1OsOZL7GRtK1W9ho20c5JJygHrY=;
 b=RWv2w5fV1XRN1aZrMYT/5yUTkcEmnqeHCtBwrBfLJThBqxe3AN9399dXgdKGBjDW6FC6
 lCgL15B/NosiHA2x3FJ+sA8OnypeeUwoZD+7ST5mJ98rIJ9Olf8agFxl9OtKd2N1O6FU
 NABG5ujgHzQksmFo10/H4gpl4I6vkyj2Q18/KUHmxV2f5ueUyMI6O1rzm0yBazn21dPK
 Yjt9a7JxP/QYb96FLsTWKb9uIolmBWHSZooGj+nPjQUCSDOr1IRi81JSqm1DGVGWS1pW
 JG2ZK7zHKvyjDoVtdjmlCL3xotPdymaG4IAoHnrAYWH1738vGidNZrPJUfDBKlkCdMef XQ== 
Received: from aserv0022.oracle.com (aserv0022.oracle.com [141.146.126.234])
        by userp2120.oracle.com with ESMTP id 2g058p0454-1
        (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK);
        Wed, 07 Feb 2018 17:02:23 +0000
Received: from aserv0121.oracle.com (aserv0121.oracle.com [141.146.126.235])
        by aserv0022.oracle.com (8.14.4/8.14.4) with ESMTP id w17H2MMs014469
        (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL);
        Wed, 7 Feb 2018 17:02:22 GMT
Received: from abhmp0008.oracle.com (abhmp0008.oracle.com [141.146.116.14])
        by aserv0121.oracle.com (8.14.4/8.13.8) with ESMTP id w17H2Kqg012829;
        Wed, 7 Feb 2018 17:02:22 GMT
Received: from [10.159.158.96] (/10.159.158.96)
        by default (Oracle Beehive Gateway v4.0)
        with ESMTP ; Wed, 07 Feb 2018 09:02:20 -0800
Subject: Re: [net-next,06/15] i40e: change flags to use 64 bits
To:     James Hogan <jhogan@kernel.org>,
        Jeff Kirsher [off-list ref]
Cc:     davem@davemloft.net, Alice Michael <redacted>,
        netdev@vger.kernel.org, nhorman@redhat.com, sassmann@redhat.com,
        jogreene@redhat.com, Ralf Baechle [off-list ref],
        linux-mips@linux-mips.org
References: [off-list ref]
 <20180207150907.GB5092@saruman>
From:   Shannon Nelson <redacted>
Organization: Oracle Corporation
Message-ID: [ref]
Date:   Wed, 7 Feb 2018 09:02:17 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.6.0
MIME-Version: 1.0
In-Reply-To: <20180207150907.GB5092@saruman>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
X-Proofpoint-Virus-Version: vendor=nai engine=5900 definitions=8798 signatures=668663
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 malwarescore=0
 phishscore=0 bulkscore=0 spamscore=0 mlxscore=0 mlxlogscore=779
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.0.1-1711220000 definitions=main-1802070213
Return-Path: [off-list ref]
X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0)
X-Orcpt: rfc822;linux-mips@linux-mips.org
Original-Recipient: rfc822;linux-mips@linux-mips.org
X-archive-position: 62459
X-ecartis-version: Ecartis v1.0.0
Sender: linux-mips-bounce@linux-mips.org
Errors-to: linux-mips-bounce@linux-mips.org
X-original-sender: shannon.nelson@oracle.com
Precedence: bulk
List-help: <mailto:ecartis@linux-mips.org?Subject=help>
List-unsubscribe: <mailto:ecartis@linux-mips.org?subject=unsubscribe%20linux-mips>
List-software: Ecartis version 1.0.0
List-Id: linux-mips <linux-mips.eddie.linux-mips.org>
X-List-ID: linux-mips <linux-mips.eddie.linux-mips.org>
List-subscribe: <mailto:ecartis@linux-mips.org?subject=subscribe%20linux-mips>
List-owner: <mailto:ralf@linux-mips.org>
List-post: <mailto:linux-mips@linux-mips.org>
List-archive: <http://www.linux-mips.org/archives/linux-mips/>
X-list: linux-mips

On 2/7/2018 7:09 AM, James Hogan wrote:
On Fri, Jan 26, 2018 at 01:24:50PM -0800, Jeff Kirsher wrote:
quoted
From: Alice Michael <redacted>

As we have added more flags, we need to now use more
bits and have over flooded the 32 bit size.  So
make it 64.

Also change all the existing bits to unsigned long long
bits.

Signed-off-by: Alice Michael <redacted>
Tested-by: Andrew Bowers <redacted>
Signed-off-by: Jeff Kirsher <redacted>
---
  drivers/net/ethernet/intel/i40e/i40e.h         | 67 +++++++++++++-------------
  drivers/net/ethernet/intel/i40e/i40e_ethtool.c |  4 +-
  2 files changed, 36 insertions(+), 35 deletions(-)
...
quoted
@@ -4323,7 +4323,7 @@ static int i40e_set_priv_flags(struct net_device *dev, u32 flags)
  	 * originally. We'll just punt with an error and log something in the
  	 * message buffer.
  	 */
-	if (cmpxchg(&pf->flags, orig_flags, new_flags) != orig_flags) {
+	if (cmpxchg64(&pf->flags, orig_flags, new_flags) != orig_flags) {
This breaks allyesconfig builds on certain architectures, for example
MIPS 32-bit with SMP enabled, which doesn't support cmpxchg64:

   CC      drivers/net/ethernet/intel/i40e/i40e_ethtool.o
drivers/net/ethernet/intel/i40e/i40e_ethtool.c: In function ���i40e_set_priv_flags���:
drivers/net/ethernet/intel/i40e/i40e_ethtool.c:4326:6: error: implicit declaration of function ���cmpxchg64���; did you mean ���__cmpxchg���? [-Werror=implicit-function-declaration]
   if (cmpxchg64(&pf->flags, orig_flags, new_flags) != orig_flags) {
       ^~~~~~~~~
       __cmpxchg

Should the driver now depend on 64BIT or something?
A long time ago this was the original expectation for this driver, but 
it was strongly suggested that in order to play well in the kernel it 
needed to be usable in 32-bit builds as well.  I suspect this sentiment 
remains, and besides we don't want to break existing support, so this 
probably needs to be addressed for 32-bit.

sln
Cheers
James
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help