>>> On 11/24/2010 at 9:56 AM, in message <20101124145617.ga11...@ioremap.net>,
Evgeniy Polyakov <z...@ioremap.net> wrote:
> Hi.
>
> I will ack connector part of course, but this hunk is actually quite
Thank you.
> bad.
>
>
>> +static void shutdown_onchannelcallback(void *context)
>> +{
>> + struct vmbus_channel *channel = context;
>> + u8 *buf;
>> + u32 buflen, recvlen;
>> + u64 requestid;
>> + u8 execute_shutdown = false;
>> +
>> + struct shutdown_msg_data *shutdown_msg;
>> +
>> + struct icmsg_hdr *icmsghdrp;
>> + struct icmsg_negotiate *negop = NULL;
>> +
>> + buflen = PAGE_SIZE;
>> + buf = kmalloc(buflen, GFP_ATOMIC);
>> +
>> + vmbus_recvpacket(channel, buf, buflen, &recvlen, &requestid);
>
> Boom. I did not read further, since this function returns void and thus
> can not propagate error, which is likely not a good idea.
Hank and Haiyang (both copied here) are the authors of this code. My
contribution to this code (in this patch) has been to change the name of the
file! I will let Hank and Haiyang comment on your feedback.
Regards,
K. Y
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/virtualization
