On Aug 8 2016 16:50, Takashi Iwai wrote:
On Mon, 08 Aug 2016 09:04:55 +0200, Takashi Iwai wrote:
On Sun, 07 Aug 2016 11:48:47 +0200, Takashi Sakamoto wrote:
In former commit, actual operations of each ioctl command get argument in kernel space. Copying from/to user space is performed outside of the function.
This commit optimizes to the new design.
Signed-off-by: Takashi Sakamoto o-takashi@sakamocchi.jp
While it's OK to split to small patches if you prefer, you don't have to do so. Basically all the rest are doing the same thing (strip copy_*_user() and replace to the pointer accesses), and it's rather boring to read repeated mails.
BTW, I'm afraid that the patch series breaks bisection. We need to consider rearranging the changes if we want to keep bisectionability.
As long as it's possible. But in this case, it's difficult. The relation between ioctl table and each functions is one to N. If we change them in one patch, the size is quite large (and alsa-project.org will skip it to deliver.). It's unavoidable.
Regards
Takashi Sakamoto