Claude review: dma-buf: Include ioctl.h in UAPI header

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Correctness:** The patch is correct. The file uses `_IOW` and `_IOWR` macros:

```c
#define DMA_BUF_IOCTL_SYNC	_IOW(DMA_BUF_BASE, 0, struct dma_buf_sync)
#define DMA_BUF_SET_NAME	_IOW(DMA_BUF_BASE, 1, const char *)
#define DMA_BUF_SET_NAME_A	_IOW(DMA_BUF_BASE, 1, __u32)
#define DMA_BUF_SET_NAME_B	_IOW(DMA_BUF_BASE, 1, __u64)
#define DMA_BUF_IOCTL_EXPORT_SYNC_FILE	_IOWR(DMA_BUF_BASE, 2, struct dma_buf_export_sync_file)
#define DMA_BUF_IOCTL_IMPORT_SYNC_FILE	_IOW(DMA_BUF_BASE, 3, struct dma_buf_import_sync_file)
```

These macros are provided by `<linux/ioctl.h>`, which is not currently included.

**Placement:** The new include is added before the existing `#include <linux/types.h>`, maintaining alphabetical order. Good.

```c
+#include <linux/ioctl.h>
 #include <linux/types.h>
```

**Commit message:** Clear and concise. Accurately describes the problem and the fix.

**Risk:** Essentially zero. Adding a missing include to a UAPI header cannot break existing users (they must have been including `<linux/ioctl.h>` themselves, either directly or transitively). It only fixes the case where they weren't.

**Reviewed-by worthy:** Yes. No concerns.

---
Generated by Claude Code Patch Reviewer