From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 70BF33C0C; Sat, 29 Jun 2024 01:59:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719626389; cv=none; b=VuIFb0ULdpDlC+jNWEOoHxcrXmD12fm/hl45zQ4eP9PiTA6sbHr6RbewBT0hGA3JjD3kJ0PYp3j7qA/I6DCdeGDZVmYCW0V6Fmvh0WGJVVEsFBzd+w/ut2W/UpMHQzq05CrqHjXZJyUAgCMHvoxfyNTudkaklXX/jOK1yh2TKqQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719626389; c=relaxed/simple; bh=dxNiEUCiVeI5V98ExqslHQUXuZLzcJJesPgtYyYGyjQ=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=rabLsXQbWCn0T69wW9yrPDOtkWXUNR64CPeVow5viJDE2osG6PkPpnGUaXm0dmaJl2a+Z7IKZhlwxa7OP22xx8ptiYV24RHRbm5DBJbF5/bxPysEEUKOGqWq69NKkIFW1l2+L/2VyzzRMpULc0LgGUYmbxg7rKxGxm18VTxaM7c= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=jzkLHxDp; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="jzkLHxDp" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 76BABC116B1; Sat, 29 Jun 2024 01:59:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1719626389; bh=dxNiEUCiVeI5V98ExqslHQUXuZLzcJJesPgtYyYGyjQ=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=jzkLHxDpFHNnEr4T8Z2KXPJ7qAnVR3MP6YjXJyCCl6JWsqkGbmAiYAlyJF2uChmfi +U8Mx7vu8dyiaN39Lgpros1rSrxgy+jTxlnOdEwBzrKPy9rWcmDsxRaPXKtUdfNEbh AiI3Zp0eiDikY49K/sPVZ8s5XZtcRBQTs8vrfgehvkalUF/+Hpc3wXMEXmY9kAmyYM 99KrSQxWW6gxQ2VQ3cCMs3FiUNBKNQioYw5hOZudYlzlmWSmbluZ0fJDa/Epho/jSc IslaajSqdjjzdWkpkO494Q0L5ew1gUzNdXFRPq3+TD9WgLVB9V/9A+aAtYpFCinIWq WwDsBURovvidw== Date: Fri, 28 Jun 2024 18:59:47 -0700 From: Jakub Kicinski To: Edward Cree Cc: Alexander Lobakin , "David S. Miller" , Eric Dumazet , Paolo Abeni , David Ahern , Xuan Zhuo , Andrew Lunn , nex.sw.ncis.osdt.itp.upstreaming@intel.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next 1/5] netdevice: convert private flags > BIT(31) to bitfields Message-ID: <20240628185947.4e8cec02@kernel.org> In-Reply-To: References: <20240625114432.1398320-1-aleksander.lobakin@intel.com> <20240625114432.1398320-2-aleksander.lobakin@intel.com> <20240626075117.6a250653@kernel.org> <20240627125541.3de68e1f@kernel.org> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Fri, 28 Jun 2024 17:03:10 +0100 Edward Cree wrote: > >> It generates sizeof(bitfield) which the compilers don't like and don't > >> want to compile ._. > > > > Mm. Okay, I have no better ideas then. > > > > Do consider moving the cold flags next to wol_enabled, tho? > > My RSS series moves wol_enabled out to struct ethtool_netdev_state [1] so > this may not be worthwhile? Speaking of which a new bit just appeared there, for the SFP module flashing. I'm gonna merge your series because it technically doesn't impact it, but could you follow up and move that bit to ethtool state?