Re: [PATCHv2 12/23] media: v4l2-subdev.h: increase struct v4l2_subdev name size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 25, 2023, at 08:26, Hans Verkuil wrote:
> On 23/09/2023 20:11, Laurent Pinchart wrote:
>>>  
>>> -#define V4L2_SUBDEV_NAME_SIZE 32
>>> +#define V4L2_SUBDEV_NAME_SIZE 52
>> 
>> Can we allocate it dynamically instead ?
>
> Anything is possible :-)
>
> But that's a separate issue and, I think, should be part of a larger
> uAPI discussion. Right now it is just to squash those annoying warnings.

Agreed, and I don't think that dynamic allocation here would
help either on saving memory (because of the slab overhead) or
for correctness (making it even harder for uapi to if there is no
upper bound on the name length).

   Arnd



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux