Re: [PATCH v3 0/4] ALSA: seq: obsolete change of address limit

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

 



On Thu, 11 Aug 2016 10:58:30 +0200,
Takashi Sakamoto wrote:
> 
> Hi,
> 
> ALSA sequencer core has two types of clients; application and kernel.
> For kernel clients, the core should handle data in kernel space, meanwhile
> this is achieved with a hack of change of address limit. This should be
> purged.
> 
> This patchset is a revised version of my previous one:
>  obsolete change of address limit
> http://mailman.alsa-project.org/pipermail/alsa-devel/2016-August/110937.html
> 
> Changes in v2:
>  - Use '_IOC_SIZE' macro to calculate the size of argument of ioctl command.
>  - Keep redundant longer name.
>  - Improve comments.
>  - Merge some patches relevant to the same features.
> 
> Changes in v3:
>  - Improve series of patch
>  - Use 'dir' field of ioctl command to detect the length of argument

The patchset looks much better now.  The only issue I see is what
Clemens suggested.  So I'll merge v4 patchset once when you resubmit
with the fix.

But for v4 set, please put "v4" in the patch subjects, not only in the
cover letter.  You can generate it via a proper prefix option with
git send-email or git format-patch.


thanks,

Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux