Hi Hans,
On 04/11/2018 12:08 PM, Hans Leidekker wrote:
}
+static int pipe_end_get_connection_attribute( struct pipe_end *pipe_end ) +{
- unsigned reply_size = get_reply_max_size();
This should be data_size_t instead. You use it in just one place, so you could consider just removing reply_size variable.
Also you may consider using set_reply_data() instead of set_reply_data_size(). Other than that the patch looks good to me.
Thanks,
Jacek