From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753644AbcLGT6r (ORCPT ); Wed, 7 Dec 2016 14:58:47 -0500 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:25514 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753122AbcLGT6q (ORCPT ); Wed, 7 Dec 2016 14:58:46 -0500 X-IronPort-AV: E=Sophos;i="5.33,315,1477954800"; d="scan'208";a="203244725" Date: Wed, 7 Dec 2016 20:58:43 +0100 (CET) From: Julia Lawall X-X-Sender: jll@hadrien To: Sasikumar Chandrasekaran cc: jejb@kernel.org, hch@infradead.org, linux-scsi@vger.kernel.org, Sathya.Prakash@broadcom.com, linux-kernel@vger.kernel.org, christopher.owens@broadcom.com, kiran-kumar.kasturi@broadcom.com, thenzl@redhat.com, sasikumar.pc@broadcom.com, kbuild-all@01.org Subject: Re: [PATCH V4 04/11] megaraid_sas: SAS3.5 Generic Megaraid Controllers Stream Detection and IO Coalescing (fwd) Message-ID: User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The code on lines 1744 and 1749 seems to need to be indented more. 1744 should be lined up with the inside of the first ( and the comment and the continue should be indented. julia ---------- Forwarded message ---------- Date: Wed, 7 Dec 2016 12:14:15 +0800 From: kbuild test robot To: kbuild@01.org Cc: Julia Lawall Subject: Re: [PATCH V4 04/11] megaraid_sas: SAS3.5 Generic Megaraid Controllers Stream Detection and IO Coalescing In-Reply-To: <1481065220-18431-5-git-send-email-sasikumar.pc@broadcom.com> Hi Sasikumar, [auto build test WARNING on scsi/for-next] [also build test WARNING on next-20161206] [cannot apply to v4.9-rc8] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Sasikumar-Chandrasekaran/megaraid_sas-Updates-for-scsi-next/20161207-102153 base: https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next :::::: branch date: 2 hours ago :::::: commit date: 2 hours ago >> drivers/scsi/megaraid/megaraid_sas_fusion.c:1749:3-12: code aligned with following code on line 1750 git remote add linux-review https://github.com/0day-ci/linux git remote update linux-review git checkout c919127dc789eec06ff5e34e06ebb05ef30dbb5e vim +1749 drivers/scsi/megaraid/megaraid_sas_fusion.c c919127d Sasikumar Chandrasekaran 2016-12-06 1743 if ((io_info->ldStartBlock != current_sd->next_seq_lba) && c919127d Sasikumar Chandrasekaran 2016-12-06 1744 ((!io_info->isRead) || (!is_read_ahead))) c919127d Sasikumar Chandrasekaran 2016-12-06 1745 /* c919127d Sasikumar Chandrasekaran 2016-12-06 1746 * Once the API availible we need to change this. c919127d Sasikumar Chandrasekaran 2016-12-06 1747 * At this point we are not allowing any gap c919127d Sasikumar Chandrasekaran 2016-12-06 1748 */ c919127d Sasikumar Chandrasekaran 2016-12-06 @1749 continue; c919127d Sasikumar Chandrasekaran 2016-12-06 @1750 cmd->io_request->RaidContext.raid_context_g35.stream_detected c919127d Sasikumar Chandrasekaran 2016-12-06 1751 = true; c919127d Sasikumar Chandrasekaran 2016-12-06 1752 current_sd->next_seq_lba = c919127d Sasikumar Chandrasekaran 2016-12-06 1753 io_info->ldStartBlock + io_info->numBlocks; --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation