From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933388Ab2C2Qjo (ORCPT ); Thu, 29 Mar 2012 12:39:44 -0400 Received: from mail-iy0-f174.google.com ([209.85.210.174]:52682 "EHLO mail-iy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932727Ab2C2Qjj (ORCPT ); Thu, 29 Mar 2012 12:39:39 -0400 From: Jiang Liu To: Greg KH Cc: Jiang Liu , Jiang Liu , Keping Chen , linux-kernel@vger.kernel.org Subject: [PATCH v2 0/3] normalize return value of chip->irq_set_affinity() method Date: Fri, 30 Mar 2012 00:36:55 +0800 Message-Id: <1333039018-3845-1-git-send-email-jiang.liu@huawei.com> X-Mailer: git-send-email 1.7.5.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On x86 platforms, interrupt controller chip's irq_set_affinity() method always copies affinity mask to irq_data->affinity field but still returns 0(IRQ_SET_MASK_OK). That return value causes the interrupt core logic unnecessarily copies the mask to irq_data->affinity field again. So return IRQ_SET_MASK_OK_NOCOPY instead of IRQ_SET_MASK_OK to get rid of the duplicated copy operation. v2: split it into three patches for core, x86 and IA64 according to Greg KH's suggestion Jiang Liu (3): IRQ: introduce irq_do_set_affinity() to reduce duplicated code IRQ,x86: normalize return value of chip->irq_set_affinity() method IRQ,IA64: normalize return value of chip->irq_set_affinity() method arch/ia64/kernel/iosapic.c | 4 +++- arch/ia64/kernel/msi_ia64.c | 4 ++-- arch/ia64/sn/kernel/irq.c | 2 +- arch/ia64/sn/kernel/msi_sn.c | 2 +- arch/x86/kernel/apic/io_apic.c | 11 ++++++----- arch/x86/platform/uv/uv_irq.c | 2 +- kernel/irq/internals.h | 3 +++ kernel/irq/manage.c | 39 ++++++++++++++++++++++----------------- kernel/irq/migration.c | 6 +----- 9 files changed, 40 insertions(+), 33 deletions(-) -- 1.7.5.4