2022-08-23 17:19:36

by Ammar Faizi

[permalink] [raw]
Subject: [PATCH 0/2] Maintainer and uapi header update

From: Ammar Faizi <[email protected]>

Hi Jens,

There are two patches in this series.

1) MAINTAINERS: Add `include/linux/io_uring_types.h`.

File include/linux/io_uring_types.h doesn't have a maintainer, add it
to the io_uring section.

2) io_uring: uapi: Add `extern "C"` in io_uring.h for liburing.

On Tue, 28 Jun 2022 10:12:27 -0600, Jens Axboe wrote:
> On 6/28/22 10:10 AM, Ammar Faizi wrote:
>> Or better add that to the kernel tree as well, it won't break
>> the kernel because we have a __cplusplus guard here.
>>
>> Jens what do you think?
>
> It'd be nice to keep them fully in sync. If I recall correctly, the only
> differences right now is that clause, and the change to not using a zero
> sized array at the end of a struct (which is slated for the kernel too).

^ Do that.

Ref: https://lore.kernel.org/io-uring/[email protected]

Make it easy for liburing to integrate uapi header with the kernel.
Previously, when this header changes, the liburing side can't directly
copy this header file due to some small differences. Sync them.

Cc: Bart Van Assche <[email protected]>
Cc: Dylan Yudaken <[email protected]>
Cc: Facebook Kernel Team <[email protected]>
Signed-off-by: Ammar Faizi <[email protected]>
---

Ammar Faizi (2):
MAINTAINERS: Add `include/linux/io_uring_types.h`
io_uring: uapi: Add `extern "C"` in io_uring.h for liburing

MAINTAINERS | 1 +
include/uapi/linux/io_uring.h | 8 ++++++++
2 files changed, 9 insertions(+)


base-commit: 3f743e9bbb8fe20f4c477e4bf6341c4187a4a264
--
Ammar Faizi


2022-08-23 18:03:24

by Jens Axboe

[permalink] [raw]
Subject: Re: [PATCH 0/2] Maintainer and uapi header update

On 8/23/22 5:45 AM, Ammar Faizi wrote:
> From: Ammar Faizi <[email protected]>
>
> Hi Jens,
>
> There are two patches in this series.
>
> 1) MAINTAINERS: Add `include/linux/io_uring_types.h`.
>
> File include/linux/io_uring_types.h doesn't have a maintainer, add it
> to the io_uring section.
>
> 2) io_uring: uapi: Add `extern "C"` in io_uring.h for liburing.

Applied, thanks.

--
Jens Axboe