RE: [PATCH 1/2] hyperv: make mmio resource local

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

 



> From: devel [mailto:driverdev-devel-bounces@xxxxxxxxxxxxxxxxxxxxxx] On Behalf
> Of sthemmin@xxxxxxxxxxxxxxxxxxxxxx
> Sent: Tuesday, August 23, 2016 1:33
> To: KY Srinivasan <kys@xxxxxxxxxxxxx>; Haiyang Zhang
> <haiyangz@xxxxxxxxxxxxx>
> Cc: devel@xxxxxxxxxxxxxxxxxxxxxx; Stephen Hemminger
> <sthemmin@xxxxxxxxxxxxx>
> Subject: [PATCH 1/2] hyperv: make mmio resource local
> 
> This sender failed our fraud detection checks and may not be who they appear
> to be. Learn about spoofing at http://aka.ms/LearnAboutSpoofing
> 
> From: Stephen Hemminger <sthemmin@xxxxxxxxxxxxx>
> 
> This fixes a sparse warning because hyperv_mmio resources
> are only used in this one file and should be static.
> 
> Signed-off-by: Stephen Hemminger <sthemmin@xxxxxxxxxxxxx>
> ---
>  drivers/hv/vmbus_drv.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c
> index e82f7e1..954d16a 100644
> --- a/drivers/hv/vmbus_drv.c
> +++ b/drivers/hv/vmbus_drv.c
> @@ -105,8 +105,8 @@ static struct notifier_block hyperv_panic_block = {
> 
>  static const char *fb_mmio_name = "fb_range";
>  static struct resource *fb_mmio;
> -struct resource *hyperv_mmio;
> -DEFINE_SEMAPHORE(hyperv_mmio_lock);
> +static struct resource *hyperv_mmio;
> +static DEFINE_SEMAPHORE(hyperv_mmio_lock);
> 
>  static int vmbus_exists(void)
>  {
> --

I also found this and was trying to fix this. :-)

Reviewed-by: Dexuan Cui <decui@xxxxxxxxxxxxx>
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux