From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) (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 F2A063FC3 for ; Wed, 1 Sep 2021 11:36:43 +0000 (UTC) X-IronPort-AV: E=McAfee;i="6200,9189,10093"; a="279729839" X-IronPort-AV: E=Sophos;i="5.84,369,1620716400"; d="scan'208";a="279729839" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Sep 2021 04:36:43 -0700 X-IronPort-AV: E=Sophos;i="5.84,369,1620716400"; d="scan'208";a="466840713" Received: from silpixa00400314.ir.intel.com (HELO silpixa00400314) ([10.237.222.51]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Sep 2021 04:36:41 -0700 Date: Wed, 1 Sep 2021 12:36:33 +0100 From: Giovanni Cabiddu To: Andy Shevchenko Cc: kernel test robot , llvm@lists.linux.dev, kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, Herbert Xu , Fiona Trahe , Wojciech Ziemba Subject: Re: drivers/crypto/qat/qat_c62xvf/adf_c62xvf_hw_data.c:102:28: warning: taking address of packed member 'csr_ops' of class or structure 'adf_hw_device_data' may result in an unaligned pointer value Message-ID: References: <202109010001.5r11IRSX-lkp@intel.com> Precedence: bulk X-Mailing-List: llvm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Research and Development Ireland Ltd - Co. Reg. #308263 - Collinstown Industrial Park, Leixlip, County Kildare - Ireland On Wed, Sep 01, 2021 at 02:18:17PM +0300, Andy Shevchenko wrote: > On Wed, Sep 01, 2021 at 12:19:08AM +0800, kernel test robot wrote: > > >> drivers/crypto/qat/qat_c62xvf/adf_c62xvf_hw_data.c:102:28: warning: taking address of packed member 'csr_ops' of class or structure 'adf_hw_device_data' may result in an unaligned pointer value [-Waddress-of-packed-member] > > adf_gen2_init_hw_csr_ops(&hw_data->csr_ops); > > ^~~~~~~~~~~~~~~~ > > 3 warnings generated. > > > Why on Earth the struct adf_hw_device_data, that contains a lot of pointers, > marked as __packed?! That __packed has to be dropped. That struct doesn't need to be marked as __packed. I'm going to send a patch that drops that attribute for all the structures that do not need to be packed in the QAT driver. -- Giovanni