RE: [PATCH 3/5] Drivers: scsi: storvsc: Implement multi-channelsupport

From: KY Srinivasan
Date: Thu Jun 27 2013 - 07:50:37 EST




> -----Original Message-----
> From: James Bottomley [mailto:jbottomley@xxxxxxxxxxxxx]
> Sent: Wednesday, June 26, 2013 11:22 PM
> To: KY Srinivasan
> Cc: gregkh@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> devel@xxxxxxxxxxxxxxxxxxxxxx; ohering@xxxxxxxx; hch@xxxxxxxxxxxxx; linux-
> scsi@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 3/5] Drivers: scsi: storvsc: Implement multi-channel support
>
> On Tue, 2013-06-04 at 12:05 -0700, K. Y. Srinivasan wrote:
> > Implement multi-channel support for the storage devices.
>
> This doesn't compile:
>
> CC [M] drivers/scsi/storvsc_drv.o
> drivers/scsi/storvsc_drv.c: In function âhandle_sc_creationâ:
> drivers/scsi/storvsc_drv.c:763:35: error: âstruct vmbus_channelâ has no
> member named âprimary_channelâ
> drivers/scsi/storvsc_drv.c: In function âhandle_multichannel_storageâ:
> drivers/scsi/storvsc_drv.c:805:2: error: implicit declaration of
> function
> âvmbus_set_sc_create_callbackâ [-Werror=implicit-function-declaration]
> drivers/scsi/storvsc_drv.c:812:2: error: implicit declaration of
> function
> âvmbus_are_subchannels_presentâ [-Werror=implicit-function-declaration]
> drivers/scsi/storvsc_drv.c: In function âstorvsc_on_channel_callbackâ:
> drivers/scsi/storvsc_drv.c:1223:13: error: âstruct vmbus_channelâ has no
> member named âprimary_channelâ
> drivers/scsi/storvsc_drv.c:1224:19: error: âstruct vmbus_channelâ has no
> member named âprimary_channelâ
> drivers/scsi/storvsc_drv.c: In function âstorvsc_do_ioâ:
> drivers/scsi/storvsc_drv.c:1341:2: error: implicit declaration of
> function
> âvmbus_get_outgoing_channelâ [-Werror=implicit-function-declaration]
> drivers/scsi/storvsc_drv.c:1341:19: warning: assignment makes pointer
> from integer without a cast [enabled by default]
> cc1: some warnings being treated as errors
> make[2]: *** [drivers/scsi/storvsc_drv.o] Error 1
>
> I assume this is a cross tree dependency? What's the relevant branch I
> need?

You are right; Greg checked in the relevant patch sometime back:

git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
in the char-misc-next branch.


Regards,

K. Y

>
> James

N‹§²æìr¸›yúèšØb²X¬¶ÇvØ^–)Þ{.nÇ+‰·¥Š{±‘êçzX§¶›¡Ü}©ž²ÆzÚ&j:+v‰¨¾«‘êçzZ+€Ê+zf£¢·hšˆ§~†­†Ûiÿûàz¹®w¥¢¸?™¨è­Ú&¢)ßf”ù^jÇy§m…á@A«a¶Úÿ 0¶ìh®å’i