lkml.org 
[lkml]   [2014]   [Jun]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCHv2 5/5] tools: ffs-test: add compatibility code for old kernels
    Date
    If ffs-test is used with a kernel prior to 3.14, which do not
    support the new descriptors format, it will fail when trying to
    write the descriptors. Add a function that converts the new
    descriptors to the legacy ones and use it to retry writing the
    descriptors using the legacy format.

    Also add “-l” flag to ffs-test which will cause the tool to
    never try the new format and instead immediatelly try the
    legacy one. This should be useful to test whether parsing
    of the old format still works on given 3.14+ kernel.

    Signed-off-by: Michal Nazarewicz <mina86@mina86.com>
    ---
    tools/usb/ffs-test.c | 84 ++++++++++++++++++++++++++++++++++++++++++++++++----
    1 file changed, 79 insertions(+), 5 deletions(-)

    diff --git a/tools/usb/ffs-test.c b/tools/usb/ffs-test.c
    index 708d317..407c828 100644
    --- a/tools/usb/ffs-test.c
    +++ b/tools/usb/ffs-test.c
    @@ -29,6 +29,7 @@
    #include <fcntl.h>
    #include <pthread.h>
    #include <stdarg.h>
    +#include <stdbool.h>
    #include <stdio.h>
    #include <stdlib.h>
    #include <string.h>
    @@ -172,6 +173,61 @@ static const struct {
    },
    };

    +static size_t descs_to_legacy(void **legacy, const void *descriptors) {
    + __u32 length, flags, fs_count = 0, hs_count = 0, count;
    + const unsigned char *descs = descriptors, *usb_descs;
    + unsigned char *out;
    +
    + if (get_unaligned_le32(descs) != FUNCTIONFS_DESCRIPTORS_MAGIC_V2)
    + return 0;
    +
    + length = get_unaligned_le32(descs + 4);
    + if (length < 8)
    + return 0;
    + descs += 8;
    + length -= 8;
    +
    +#define GET_LE32(ret) do { \
    + if (length < 4) \
    + return 0; \
    + ret = get_unaligned_le32(descs); \
    + descs += 4; \
    + length -= 4; \
    + } while (0)
    +
    + GET_LE32(flags);
    + if (flags & FUNCTIONFS_HAS_FS_DESC)
    + GET_LE32(fs_count);
    + if (flags & FUNCTIONFS_HAS_HS_DESC)
    + GET_LE32(hs_count);
    + if (!fs_count && !hs_count)
    + return 0;
    + if (flags & FUNCTIONFS_HAS_SS_DESC)
    + GET_LE32(count); /* The value is ignored later on anyway. */
    + if (flags)
    + return 0;
    +
    +#undef GET_LE32
    +
    + usb_descs = descs;
    + for (count = fs_count + hs_count; count; --count) {
    + if (length < *descs)
    + return 0;
    + length -= *descs;
    + descs += *descs;
    + }
    +
    + length = 16 + (descs - usb_descs);
    + out = malloc(length);
    + put_unaligned_le32(FUNCTIONFS_DESCRIPTORS_MAGIC, out);
    + put_unaligned_le32(length, out + 4);
    + put_unaligned_le32(fs_count, out + 8);
    + put_unaligned_le32(hs_count, out + 12);
    + memcpy(out + 16, usb_descs, length - 16);
    + *legacy = out;
    + return length;
    +}
    +

    #define STR_INTERFACE_ "Source/Sink"

    @@ -491,12 +547,29 @@ ep0_consume(struct thread *ignore, const void *buf, size_t nbytes)
    return nbytes;
    }

    -static void ep0_init(struct thread *t)
    +static void ep0_init(struct thread *t, bool legacy_descriptors)
    {
    + void *legacy;
    ssize_t ret;
    + size_t len;
    +
    + if (legacy_descriptors) {
    + info("%s: writing descriptors\n", t->filename);
    + goto legacy;
    + }

    - info("%s: writing descriptors\n", t->filename);
    + info("%s: writing descriptors (in v2 format)\n", t->filename);
    ret = write(t->fd, &descriptors, sizeof descriptors);
    +
    + if (ret < 0 && errno == EINVAL) {
    + warn("%s: new format rejected, trying legacy\n", t->filename);
    +legacy:
    + len = descs_to_legacy(&legacy, &descriptors);
    + if (len) {
    + ret = write(t->fd, legacy, len);
    + free(legacy);
    + }
    + }
    die_on(ret < 0, "%s: write: descriptors", t->filename);

    info("%s: writing strings\n", t->filename);
    @@ -507,14 +580,15 @@ static void ep0_init(struct thread *t)

    /******************** Main **************************************************/

    -int main(void)
    +int main(int argc, char **argv)
    {
    + bool legacy_descriptors;
    unsigned i;

    - /* XXX TODO: Argument parsing missing */
    + legacy_descriptors = argc > 2 && !strcmp(argv[1], "-l");

    init_thread(threads);
    - ep0_init(threads);
    + ep0_init(threads, legacy_descriptors);

    for (i = 1; i < sizeof threads / sizeof *threads; ++i)
    init_thread(threads + i);
    --
    2.0.0.526.g5318336
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2014-06-06 11:41    [W:4.721 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site