From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934074AbcCIT4j (ORCPT ); Wed, 9 Mar 2016 14:56:39 -0500 Received: from p3plsmtps2ded02.prod.phx3.secureserver.net ([208.109.80.59]:43913 "EHLO p3plsmtps2ded02.prod.phx3.secureserver.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934021AbcCIT4Y (ORCPT ); Wed, 9 Mar 2016 14:56:24 -0500 x-originating-ip: 72.167.245.219 From: "K. Y. Srinivasan" To: gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, devel@linuxdriverproject.org, olaf@aepfle.de, apw@canonical.com, vkuznets@redhat.com, jasowang@redhat.com Cc: Jake Oshins , "K. Y. Srinivasan" Subject: [PATCH 3/6] hv: Lock access to hyperv_mmio resource tree Date: Wed, 9 Mar 2016 13:35:22 -0800 Message-Id: <1457559325-10406-3-git-send-email-kys@microsoft.com> X-Mailer: git-send-email 1.7.4.1 In-Reply-To: <1457559325-10406-1-git-send-email-kys@microsoft.com> References: <1457559302-10365-1-git-send-email-kys@microsoft.com> <1457559325-10406-1-git-send-email-kys@microsoft.com> X-CMAE-Envelope: MS4wfJ5R4yIjldPlPRhbIOYuODmlxU8R3/kbZWThNnupi2vN9q2veVxYI1XptpsQY1ttbnNlNiRu4V1xkifUjSoabamZHHzHazCjgti7sjh9izL7T4wTyDJf XnheZHhq0oiVS5xsNYfJ3PJ+FQ7GzTUYSc9dUf7fC1+ZBUz02pAujJVQVT2TzlwIj9eWKYvKp+79ppVG1n3vENAQUgmaLhcfp/EJ7dc5j50GtD3xNMl5bkmW rvc3X9a3+iM+fFnHqr8zoX8pNPOwg98C6tKoxBikspfg6PhCFT06srEDkzUt4nPOLKMY1T3YR7Y0S4ad+oij8lN3uWg/xCGVbF/tILUFz3dG66sxIWHOZffC cgTRx+dCTWAOg/Q5mufjtaid8y84hppXfoJihkXoaRssc2Db3DX4rtuMXdYW427YnfT2KYD0/Wbx0GLfUz7zk1IxJT8QRw== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Jake Oshins In existing code, this tree of resources is created in single-threaded code and never modified after it is created, and thus needs no locking. This patch introduces a semaphore for tree access, as other patches in this series introduce run-time modifications of this resource tree which can happen on multiple threads. Signed-off-by: Jake Oshins Signed-off-by: K. Y. Srinivasan --- drivers/hv/vmbus_drv.c | 16 ++++++++++++---- 1 files changed, 12 insertions(+), 4 deletions(-) diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c index 44e95a4..60553c1 100644 --- a/drivers/hv/vmbus_drv.c +++ b/drivers/hv/vmbus_drv.c @@ -102,6 +102,7 @@ static struct notifier_block hyperv_panic_block = { }; struct resource *hyperv_mmio; +DEFINE_SEMAPHORE(hyperv_mmio_lock); static int vmbus_exists(void) { @@ -1132,7 +1133,10 @@ int vmbus_allocate_mmio(struct resource **new, struct hv_device *device_obj, resource_size_t range_min, range_max, start, local_min, local_max; const char *dev_n = dev_name(&device_obj->device); u32 fb_end = screen_info.lfb_base + (screen_info.lfb_size << 1); - int i; + int i, retval; + + retval = -ENXIO; + down(&hyperv_mmio_lock); for (iter = hyperv_mmio; iter; iter = iter->sibling) { if ((iter->start >= max) || (iter->end <= min)) @@ -1169,13 +1173,17 @@ int vmbus_allocate_mmio(struct resource **new, struct hv_device *device_obj, for (; start + size - 1 <= local_max; start += align) { *new = request_mem_region_exclusive(start, size, dev_n); - if (*new) - return 0; + if (*new) { + retval = 0; + goto exit; + } } } } - return -ENXIO; +exit: + up(&hyperv_mmio_lock); + return retval; } EXPORT_SYMBOL_GPL(vmbus_allocate_mmio); -- 1.7.4.1