From patchwork Mon Oct 12 22:20:39 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Latypov X-Patchwork-Id: 11834351 X-Patchwork-Delegate: brendanhiggins@google.com Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 47B2A109B for ; Mon, 12 Oct 2020 22:21:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 1CBB320797 for ; Mon, 12 Oct 2020 22:21:20 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="NaQP/YeJ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388832AbgJLWVT (ORCPT ); Mon, 12 Oct 2020 18:21:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55730 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388682AbgJLWVT (ORCPT ); Mon, 12 Oct 2020 18:21:19 -0400 Received: from mail-yb1-xb4a.google.com (mail-yb1-xb4a.google.com [IPv6:2607:f8b0:4864:20::b4a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EBE5CC0613D0 for ; Mon, 12 Oct 2020 15:21:18 -0700 (PDT) Received: by mail-yb1-xb4a.google.com with SMTP id m62so15236193ybb.6 for ; Mon, 12 Oct 2020 15:21:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=sender:date:in-reply-to:message-id:mime-version:references:subject :from:to:cc; bh=m35WjvIMdsTWxP1ssovUOtDU6uHY4X6slQ9fwWQxET4=; b=NaQP/YeJkoi5eV1MeWEAwbMUYB5TURM2+Y6Z+GZflVYGZdW6GZFAyFOFWdURquNvaC s4eBulnfXFK6ABrnQ3sNdHObLnFdfR3+0nEi6aRJjNTKrMAvM/SJiRdO8PGgIsYKlsp5 Rdp7s87VhU9ZZ8im6ZN0X14ESo9Ms1KeYIyVEkVxDYxpGDM7hAxzvmORewzhaFzCxrHy FkK7nNddYd5m1+AICMJL8InDuvlF0nKtaL1XYcb8Yxe33gjOAbAsvsfdfnEoA9SVh+ZH b7SXzJ8TJUGvBRGdAV0pNabOMpboYcHN8JvKWZA699A8bt4BWwVlJzbiCzB0YmKn7iqU 6nOw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:in-reply-to:message-id:mime-version :references:subject:from:to:cc; bh=m35WjvIMdsTWxP1ssovUOtDU6uHY4X6slQ9fwWQxET4=; b=kmFaRZ1hQ8yOoDautMYbOmDNm4HVDKSVK7sj487yU5pkjVvX7pUvOAag4s/mKQfSSB VlPlogj+gnNOU7fHg0PfREBdvQPBwbcbKysVLyLwi6bT8hnqILiecwBLh/v1pWlraxQC ElgoKPMY8ZHmmMYL1dAmPMayt99xI/kl6cv835OqU6bpCv6llCPpVu1guwePjblCOqx7 mPRF7xcCyC7S5mrugVMQjgmltz99Xv8QCtkVdio0ZmtEaMJoEiGbcNnp2SkZNdFP7+Bd yn248xALldoIJdLU6TXNgalp4TUuhkg1rxasM03JUXISGvmQHNfnSiF4yAp0pyCPH24G WSMg== X-Gm-Message-State: AOAM530Z7ukSdtPwtdGPsI8dAT4crGs9KeYN3oH9PIjQnGJlDYyDHU/K HtTGrs56YtO2mobjUuePVdo9XaZrLh4Sqw== X-Google-Smtp-Source: ABdhPJxW/TlitWaP/RAp/cQMzpuTJpAfvjMr44SwofFBJS5F7J5XIDQObKaQdZInIkHIk1F0/yPBRKks+udEFA== Sender: "dlatypov via sendgmr" X-Received: from dlatypov.svl.corp.google.com ([2620:15c:2cd:202:a28c:fdff:fee3:28c6]) (user=dlatypov job=sendgmr) by 2002:a25:6585:: with SMTP id z127mr8041680ybb.33.1602541278166; Mon, 12 Oct 2020 15:21:18 -0700 (PDT) Date: Mon, 12 Oct 2020 15:20:39 -0700 In-Reply-To: <20201012222050.999431-1-dlatypov@google.com> Message-Id: <20201012222050.999431-2-dlatypov@google.com> Mime-Version: 1.0 References: <20201012222050.999431-1-dlatypov@google.com> X-Mailer: git-send-email 2.28.0.1011.ga647a8990f-goog Subject: [RFC v2 01/12] Revert "kunit: move string-stream.h to lib/kunit" From: Daniel Latypov To: dlatypov@google.com Cc: alan.maguire@oracle.com, brendanhiggins@google.com, davidgow@google.com, keescook@chromium.org, kunit-dev@googlegroups.com, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, mcgrof@kernel.org, sboyd@kernel.org, skhan@linuxfoundation.org Precedence: bulk List-ID: X-Mailing-List: linux-kselftest@vger.kernel.org This reverts commit 109fb06fdc6f6788df7dfbc235f7636a38e28fd4. string-stream will be used by kunit mocking code to print messages about mock expectations. It makes the code signifcantly simpler if string-stream objects can be part of structs declared in mocking headers. Signed-off-by: Daniel Latypov --- include/kunit/assert.h | 3 +-- {lib => include}/kunit/string-stream.h | 0 lib/kunit/assert.c | 2 -- lib/kunit/string-stream-test.c | 3 +-- lib/kunit/string-stream.c | 3 +-- lib/kunit/test.c | 2 +- 6 files changed, 4 insertions(+), 9 deletions(-) rename {lib => include}/kunit/string-stream.h (100%) diff --git a/include/kunit/assert.h b/include/kunit/assert.h index ad889b539ab3..db6a0fca09b4 100644 --- a/include/kunit/assert.h +++ b/include/kunit/assert.h @@ -9,11 +9,10 @@ #ifndef _KUNIT_ASSERT_H #define _KUNIT_ASSERT_H +#include #include -#include struct kunit; -struct string_stream; /** * enum kunit_assert_type - Type of expectation/assertion. diff --git a/lib/kunit/string-stream.h b/include/kunit/string-stream.h similarity index 100% rename from lib/kunit/string-stream.h rename to include/kunit/string-stream.h diff --git a/lib/kunit/assert.c b/lib/kunit/assert.c index 33acdaa28a7d..9c12e30792ba 100644 --- a/lib/kunit/assert.c +++ b/lib/kunit/assert.c @@ -8,8 +8,6 @@ #include #include -#include "string-stream.h" - void kunit_base_assert_format(const struct kunit_assert *assert, struct string_stream *stream) { diff --git a/lib/kunit/string-stream-test.c b/lib/kunit/string-stream-test.c index 110f3a993250..121f9ab11501 100644 --- a/lib/kunit/string-stream-test.c +++ b/lib/kunit/string-stream-test.c @@ -6,11 +6,10 @@ * Author: Brendan Higgins */ +#include #include #include -#include "string-stream.h" - static void string_stream_test_empty_on_creation(struct kunit *test) { struct string_stream *stream = alloc_string_stream(test, GFP_KERNEL); diff --git a/lib/kunit/string-stream.c b/lib/kunit/string-stream.c index 141789ca8949..151a0e7ac349 100644 --- a/lib/kunit/string-stream.c +++ b/lib/kunit/string-stream.c @@ -6,12 +6,11 @@ * Author: Brendan Higgins */ +#include #include #include #include -#include "string-stream.h" - struct string_stream_fragment_alloc_context { struct kunit *test; int len; diff --git a/lib/kunit/test.c b/lib/kunit/test.c index c36037200310..670d1cc9c105 100644 --- a/lib/kunit/test.c +++ b/lib/kunit/test.c @@ -7,12 +7,12 @@ */ #include +#include #include #include #include #include "debugfs.h" -#include "string-stream.h" #include "try-catch-impl.h" static void kunit_set_failure(struct kunit *test)