[Openipmi-developer] [PATCH v7 1/3] ipmi: ssif_bmc: Add SSIF BMC driver
quan at os.amperecomputing.com
Wed Jun 1 18:23:11 AEST 2022
On 04/05/2022 19:06, Corey Minyard wrote:
> On Wed, May 04, 2022 at 01:45:03PM +0700, Quan Nguyen via Openipmi-developer wrote:
>>> I seem to remember mentioning this before, but there is no reason to
>>> pack the structures below.
>> The packed structure is because we want to pick the len directly from user
>> space without worry about the padding byte.
>> As we plan not to use the .h file in next version, I still would like to use
>> packed structure internally inside ssif_bmc.c file.
> Packed doesn't matter for the userspace API. If you look at other
> structures in the userspace API, they are not packed, either. The
> compiler will do the right thing on both ends.
>>> And second, the following is a userspace API structures, so it needs to
>>> be in its own file in include/uapi/linux, along with any supporting
>>> things that users will need to use. And your userspace code should be
>>> using that file.
>> Meantime, I'd like not to use .h as I see there is no demand for sharing the
>> data structure between kernel and user space yet. But we may do it in the
> If you have a userspace API, it needs to be in include/uapi/linux.
> You may not be the only user of this code. In fact, you probably won't
> be. You need to have a .h with the structures in it, you don't want the
> same structure in two places if you can help it.
Is it OK to push the structure definition into the
Or should it need to be in separate new header file in uapi/linux ?
More information about the Linux-aspeed