Message ID | 20160121052928.8568.3019.stgit@localhost.localdomain |
---|---|
State | Superseded |
Headers | show |
Hi Thomas, When NVME patch will get into "buildroot" project ? On 01/21/2016 11:06 AM, Mamatha Inamdar wrote: > Add support for building NVME utility - a utility for interacting with > standard NVM Express (optimized PCI Express SSD interface) devices. > > Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > --- > package/Config.in | 1 + > package/nvme/0001-NVME-Makefile-cleanup.patch | 25 +++++++++++++++++++++++++ > package/nvme/Config.in | 6 ++++++ > package/nvme/nvme.hash | 2 ++ > package/nvme/nvme.mk | 22 ++++++++++++++++++++++ > 5 files changed, 56 insertions(+) > create mode 100644 package/nvme/0001-NVME-Makefile-cleanup.patch > create mode 100644 package/nvme/Config.in > create mode 100644 package/nvme/nvme.hash > create mode 100755 package/nvme/nvme.mk > > diff --git a/package/Config.in b/package/Config.in > index e0c2e2a..4d173d4 100644 > --- a/package/Config.in > +++ b/package/Config.in > @@ -380,6 +380,7 @@ endif > source "package/neard/Config.in" > source "package/nvidia-driver/Config.in" > source "package/nvidia-tegra23/Config.in" > + source "package/nvme/Config.in" > source "package/ofono/Config.in" > source "package/ola/Config.in" > source "package/on2-8170-modules/Config.in" > diff --git a/package/nvme/0001-NVME-Makefile-cleanup.patch b/package/nvme/0001-NVME-Makefile-cleanup.patch > new file mode 100644 > index 0000000..74beac9 > --- /dev/null > +++ b/package/nvme/0001-NVME-Makefile-cleanup.patch > @@ -0,0 +1,25 @@ > +From 985a18605f8decd766370a7d9873c77c16b9253c Mon Sep 17 00:00:00 2001 > +From: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > +Date: Mon, 18 Jan 2016 12:40:00 +0530 > +Subject: [PATCH V2] NVME Makefile update > + > +Makefile cleanup: remove -Werror flag in CFLAGS > + > +Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > +--- > + Makefile | 2 +- > + 1 file changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/Makefile b/Makefile > +index d0c0ce4..003928b 100644 > +--- a/Makefile > ++++ b/Makefile > +@@ -1,4 +1,4 @@ > +-CFLAGS += -std=gnu99 -O2 -g -Wall -Werror > ++CFLAGS += -std=gnu99 -O2 -g -Wall > + CPPFLAGS += -I $(SRC) -D_GNU_SOURCE > + NVME = nvme > + INSTALL ?= install > +-- > +1.9.3 > + > diff --git a/package/nvme/Config.in b/package/nvme/Config.in > new file mode 100644 > index 0000000..8b5d37d > --- /dev/null > +++ b/package/nvme/Config.in > @@ -0,0 +1,6 @@ > +config BR2_PACKAGE_NVME > + bool "nvme" > + help > + System utilities for IBM Power NVME devices > + > + https://github.com/linux-nvme/nvme-cli > diff --git a/package/nvme/nvme.hash b/package/nvme/nvme.hash > new file mode 100644 > index 0000000..539a2b6 > --- /dev/null > +++ b/package/nvme/nvme.hash > @@ -0,0 +1,2 @@ > +# Locally computed: > +sha256 914319930a51372293c981562b165ae6af0de70f7da37e42da23940fe23aa387 nvme-v0.2.tar.gz > diff --git a/package/nvme/nvme.mk b/package/nvme/nvme.mk > new file mode 100755 > index 0000000..f06ed47 > --- /dev/null > +++ b/package/nvme/nvme.mk > @@ -0,0 +1,22 @@ > +################################################################################ > +# > +# nvme > +# > +################################################################################ > + > +NVME_VERSION = v0.2 > +NVME_SITE = $(call github,linux-nvme,nvme-cli,$(NVME_VERSION)) > +NVME_LICENSE = GPLv2+ > +NVME_LICENSE_FILES = COPYING > + > +define NVME_BUILD_CMDS > + $(TARGET_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) \ > + LIBUDEV=1 -C $(@D) > +endef > + > +define NVME_INSTALL_TARGET_CMDS > + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) DESTDIR=$(TARGET_DIR) \ > + PREFIX=/usr install-bin > +endef > + > +$(eval $(generic-package)) > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot >
Hi Thomas, Please let us know when NVME patches will get into the build? On 02/04/2016 11:38 AM, Mamatha Inamdar wrote: > Hi Thomas, > > When NVME patch will get into "buildroot" project ? > > > On 01/21/2016 11:06 AM, Mamatha Inamdar wrote: >> Add support for building NVME utility - a utility for interacting with >> standard NVM Express (optimized PCI Express SSD interface) devices. >> >> Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> >> --- >> package/Config.in | 1 + >> package/nvme/0001-NVME-Makefile-cleanup.patch | 25 >> +++++++++++++++++++++++++ >> package/nvme/Config.in | 6 ++++++ >> package/nvme/nvme.hash | 2 ++ >> package/nvme/nvme.mk | 22 >> ++++++++++++++++++++++ >> 5 files changed, 56 insertions(+) >> create mode 100644 package/nvme/0001-NVME-Makefile-cleanup.patch >> create mode 100644 package/nvme/Config.in >> create mode 100644 package/nvme/nvme.hash >> create mode 100755 package/nvme/nvme.mk >> neric-package)) >> >> _______________________________________________ >> buildroot mailing list >> buildroot@busybox.net >> http://lists.busybox.net/mailman/listinfo/buildroot >> > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot >
On 21-01-16 06:36, Mamatha Inamdar wrote: > Add support for building NVME utility - a utility for interacting with > standard NVM Express (optimized PCI Express SSD interface) devices. This text should be part of the Config.in help text. > > Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > --- > package/Config.in | 1 + > package/nvme/0001-NVME-Makefile-cleanup.patch | 25 +++++++++++++++++++++++++ > package/nvme/Config.in | 6 ++++++ > package/nvme/nvme.hash | 2 ++ > package/nvme/nvme.mk | 22 ++++++++++++++++++++++ > 5 files changed, 56 insertions(+) > create mode 100644 package/nvme/0001-NVME-Makefile-cleanup.patch > create mode 100644 package/nvme/Config.in > create mode 100644 package/nvme/nvme.hash > create mode 100755 package/nvme/nvme.mk > > diff --git a/package/Config.in b/package/Config.in > index e0c2e2a..4d173d4 100644 > --- a/package/Config.in > +++ b/package/Config.in > @@ -380,6 +380,7 @@ endif > source "package/neard/Config.in" > source "package/nvidia-driver/Config.in" > source "package/nvidia-tegra23/Config.in" > + source "package/nvme/Config.in" > source "package/ofono/Config.in" > source "package/ola/Config.in" > source "package/on2-8170-modules/Config.in" > diff --git a/package/nvme/0001-NVME-Makefile-cleanup.patch b/package/nvme/0001-NVME-Makefile-cleanup.patch > new file mode 100644 > index 0000000..74beac9 > --- /dev/null > +++ b/package/nvme/0001-NVME-Makefile-cleanup.patch > @@ -0,0 +1,25 @@ > +From 985a18605f8decd766370a7d9873c77c16b9253c Mon Sep 17 00:00:00 2001 > +From: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > +Date: Mon, 18 Jan 2016 12:40:00 +0530 > +Subject: [PATCH V2] NVME Makefile update > + > +Makefile cleanup: remove -Werror flag in CFLAGS > + > +Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > +--- > + Makefile | 2 +- > + 1 file changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/Makefile b/Makefile > +index d0c0ce4..003928b 100644 > +--- a/Makefile > ++++ b/Makefile > +@@ -1,4 +1,4 @@ > +-CFLAGS += -std=gnu99 -O2 -g -Wall -Werror > ++CFLAGS += -std=gnu99 -O2 -g -Wall This patch doesn't apply, the CFLAGS line looks completely different. Have you tested your patch before submitting? > + CPPFLAGS += -I $(SRC) -D_GNU_SOURCE > + NVME = nvme > + INSTALL ?= install > +-- > +1.9.3 > + > diff --git a/package/nvme/Config.in b/package/nvme/Config.in > new file mode 100644 > index 0000000..8b5d37d > --- /dev/null > +++ b/package/nvme/Config.in > @@ -0,0 +1,6 @@ > +config BR2_PACKAGE_NVME > + bool "nvme" The CFLAGS explicitly pass -m64, so I guess it should depend on a 64-bit arch. And since this is hardware support for IBM Power devices, I guess it's best to depend on BR2_powerpc64 || BR2_powerpc64le > + help > + System utilities for IBM Power NVME devices > + > + https://github.com/linux-nvme/nvme-cli > diff --git a/package/nvme/nvme.hash b/package/nvme/nvme.hash > new file mode 100644 > index 0000000..539a2b6 > --- /dev/null > +++ b/package/nvme/nvme.hash > @@ -0,0 +1,2 @@ > +# Locally computed: > +sha256 914319930a51372293c981562b165ae6af0de70f7da37e42da23940fe23aa387 nvme-v0.2.tar.gz > diff --git a/package/nvme/nvme.mk b/package/nvme/nvme.mk > new file mode 100755 > index 0000000..f06ed47 > --- /dev/null > +++ b/package/nvme/nvme.mk > @@ -0,0 +1,22 @@ > +################################################################################ > +# > +# nvme > +# > +################################################################################ > + > +NVME_VERSION = v0.2 > +NVME_SITE = $(call github,linux-nvme,nvme-cli,$(NVME_VERSION)) > +NVME_LICENSE = GPLv2+ > +NVME_LICENSE_FILES = COPYING > + > +define NVME_BUILD_CMDS > + $(TARGET_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) \ > + LIBUDEV=1 -C $(@D) > +endef This LIBUDEV is a bit counterintuitive, so add a comment saying that LIBUDEV=1 means that libudev is _disabled_. It would also be good to mention in the commit log that udev handling is for a later patch. Regards, Arnout > + > +define NVME_INSTALL_TARGET_CMDS > + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) DESTDIR=$(TARGET_DIR) \ > + PREFIX=/usr install-bin > +endef > + > +$(eval $(generic-package)) > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot >
Hi Arnout, Thanks for the review comments updated all your comments and sent new version of Patch. On 02/11/2016 02:29 AM, Arnout Vandecappelle wrote: > diff --git a/package/nvme/0001-NVME-Makefile-cleanup.patch b/package/nvme/0001-NVME-Makefile-cleanup.patch > new file mode 100644 > index 0000000..74beac9 > --- /dev/null > +++ b/package/nvme/0001-NVME-Makefile-cleanup.patch > @@ -0,0 +1,25 @@ > +From 985a18605f8decd766370a7d9873c77c16b9253c Mon Sep 17 00:00:00 2001 > +From: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > +Date: Mon, 18 Jan 2016 12:40:00 +0530 > +Subject: [PATCH V2] NVME Makefile update > + > +Makefile cleanup: remove -Werror flag in CFLAGS > + > +Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> > +--- > + Makefile | 2 +- > + 1 file changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/Makefile b/Makefile > +index d0c0ce4..003928b 100644 > +--- a/Makefile > ++++ b/Makefile > +@@ -1,4 +1,4 @@ > +-CFLAGS += -std=gnu99 -O2 -g -Wall -Werror > ++CFLAGS += -std=gnu99 -O2 -g -Wall > This patch doesn't apply, the CFLAGS line looks completely different. Have you > tested your patch before submitting? yes I have tested my patch and it will apply fine. > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot > >
Hi Arnout, please take a look at my comments below On 02/11/2016 04:48 PM, Mamatha Inamdar wrote: > > > Thanks for the review comments > updated all your comments and sent new version of Patch. > > On 02/11/2016 02:29 AM, Arnout Vandecappelle wrote: >> diff --git a/package/nvme/0001-NVME-Makefile-cleanup.patch b/package/nvme/0001-NVME-Makefile-cleanup.patch >> new file mode 100644 >> index 0000000..74beac9 >> --- /dev/null >> +++ b/package/nvme/0001-NVME-Makefile-cleanup.patch >> @@ -0,0 +1,25 @@ >> +From 985a18605f8decd766370a7d9873c77c16b9253c Mon Sep 17 00:00:00 2001 >> +From: Mamatha Inamdar<mamatha4@linux.vnet.ibm.com> >> +Date: Mon, 18 Jan 2016 12:40:00 +0530 >> +Subject: [PATCH V2] NVME Makefile update >> + >> +Makefile cleanup: remove -Werror flag in CFLAGS >> + >> +Signed-off-by: Mamatha Inamdar<mamatha4@linux.vnet.ibm.com> >> +--- >> + Makefile | 2 +- >> + 1 file changed, 1 insertion(+), 1 deletion(-) >> + >> +diff --git a/Makefile b/Makefile >> +index d0c0ce4..003928b 100644 >> +--- a/Makefile >> ++++ b/Makefile >> +@@ -1,4 +1,4 @@ >> +-CFLAGS += -std=gnu99 -O2 -g -Wall -Werror >> ++CFLAGS += -std=gnu99 -O2 -g -Wall >> This patch doesn't apply, the CFLAGS line looks completely different. Have you >> tested your patch before submitting? > > yes I have tested my patch and it will apply fine. With regard to the above comment, upon checking the patch again, I see some issues in applying the patch due to version mismatch. While compiling op-build I see nvme-cli version 0.1 code is downloading which is the reason my patch fails to apply. I have created the patch on version 0.2 (using git clone https://github.com/linux-nvme/nvme-cli.git) I am looking into the reason why older version of nvme-cli is getting downloaded during compilation of op-build. > > >> _______________________________________________ >> buildroot mailing list >> buildroot@busybox.net >> http://lists.busybox.net/mailman/listinfo/buildroot >> > > > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
Hi Arnout, updated all your comments and sending new version of Patch. On 02/11/2016 10:05 PM, Mamatha Inamdar wrote: > Hi Arnout, > > please take a look at my comments below > > On 02/11/2016 04:48 PM, Mamatha Inamdar wrote: >> >> >> Thanks for the review comments >> updated all your comments and sent new version of Patch. >> >> On 02/11/2016 02:29 AM, Arnout Vandecappelle wrote: >>> diff --git a/package/nvme/0001-NVME-Makefile-cleanup.patch b/package/nvme/0001-NVME-Makefile-cleanup.patch >>> new file mode 100644 >>> index 0000000..74beac9 >>> --- /dev/null >>> +++ b/package/nvme/0001-NVME-Makefile-cleanup.patch >>> @@ -0,0 +1,25 @@ >>> +From 985a18605f8decd766370a7d9873c77c16b9253c Mon Sep 17 00:00:00 2001 >>> +From: Mamatha Inamdar<mamatha4@linux.vnet.ibm.com> >>> +Date: Mon, 18 Jan 2016 12:40:00 +0530 >>> +Subject: [PATCH V2] NVME Makefile update >>> + >>> +Makefile cleanup: remove -Werror flag in CFLAGS >>> + >>> +Signed-off-by: Mamatha Inamdar<mamatha4@linux.vnet.ibm.com> >>> +--- >>> + Makefile | 2 +- >>> + 1 file changed, 1 insertion(+), 1 deletion(-) >>> + >>> +diff --git a/Makefile b/Makefile >>> +index d0c0ce4..003928b 100644 >>> +--- a/Makefile >>> ++++ b/Makefile >>> +@@ -1,4 +1,4 @@ >>> +-CFLAGS += -std=gnu99 -O2 -g -Wall -Werror >>> ++CFLAGS += -std=gnu99 -O2 -g -Wall >>> This patch doesn't apply, the CFLAGS line looks completely different. Have you >>> tested your patch before submitting? >> >> yes I have tested my patch and it will apply fine. > > With regard to the above comment, upon checking the patch again, > I see some issues in applying the patch due to version mismatch. > > While compiling op-build I see nvme-cli version 0.1 code is > downloading which is the reason my patch fails to apply. > I have created the patch on version 0.2 (using git clone > https://github.com/linux-nvme/nvme-cli.git) > > I am looking into the reason why older version of nvme-cli is getting > downloaded during compilation of op-build. > I worked on the above issue and tested the patch. I am able to compile buildroot successfully with the patch. Sending new version of the patch with all your comments updated. Thanks Mamatha >> >> >>> _______________________________________________ >>> buildroot mailing list >>> buildroot@busybox.net >>> http://lists.busybox.net/mailman/listinfo/buildroot >>> >> >> >> >> _______________________________________________ >> buildroot mailing list >> buildroot@busybox.net >> http://lists.busybox.net/mailman/listinfo/buildroot > > > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
diff --git a/package/Config.in b/package/Config.in index e0c2e2a..4d173d4 100644 --- a/package/Config.in +++ b/package/Config.in @@ -380,6 +380,7 @@ endif source "package/neard/Config.in" source "package/nvidia-driver/Config.in" source "package/nvidia-tegra23/Config.in" + source "package/nvme/Config.in" source "package/ofono/Config.in" source "package/ola/Config.in" source "package/on2-8170-modules/Config.in" diff --git a/package/nvme/0001-NVME-Makefile-cleanup.patch b/package/nvme/0001-NVME-Makefile-cleanup.patch new file mode 100644 index 0000000..74beac9 --- /dev/null +++ b/package/nvme/0001-NVME-Makefile-cleanup.patch @@ -0,0 +1,25 @@ +From 985a18605f8decd766370a7d9873c77c16b9253c Mon Sep 17 00:00:00 2001 +From: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> +Date: Mon, 18 Jan 2016 12:40:00 +0530 +Subject: [PATCH V2] NVME Makefile update + +Makefile cleanup: remove -Werror flag in CFLAGS + +Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> +--- + Makefile | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/Makefile b/Makefile +index d0c0ce4..003928b 100644 +--- a/Makefile ++++ b/Makefile +@@ -1,4 +1,4 @@ +-CFLAGS += -std=gnu99 -O2 -g -Wall -Werror ++CFLAGS += -std=gnu99 -O2 -g -Wall + CPPFLAGS += -I $(SRC) -D_GNU_SOURCE + NVME = nvme + INSTALL ?= install +-- +1.9.3 + diff --git a/package/nvme/Config.in b/package/nvme/Config.in new file mode 100644 index 0000000..8b5d37d --- /dev/null +++ b/package/nvme/Config.in @@ -0,0 +1,6 @@ +config BR2_PACKAGE_NVME + bool "nvme" + help + System utilities for IBM Power NVME devices + + https://github.com/linux-nvme/nvme-cli diff --git a/package/nvme/nvme.hash b/package/nvme/nvme.hash new file mode 100644 index 0000000..539a2b6 --- /dev/null +++ b/package/nvme/nvme.hash @@ -0,0 +1,2 @@ +# Locally computed: +sha256 914319930a51372293c981562b165ae6af0de70f7da37e42da23940fe23aa387 nvme-v0.2.tar.gz diff --git a/package/nvme/nvme.mk b/package/nvme/nvme.mk new file mode 100755 index 0000000..f06ed47 --- /dev/null +++ b/package/nvme/nvme.mk @@ -0,0 +1,22 @@ +################################################################################ +# +# nvme +# +################################################################################ + +NVME_VERSION = v0.2 +NVME_SITE = $(call github,linux-nvme,nvme-cli,$(NVME_VERSION)) +NVME_LICENSE = GPLv2+ +NVME_LICENSE_FILES = COPYING + +define NVME_BUILD_CMDS + $(TARGET_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) \ + LIBUDEV=1 -C $(@D) +endef + +define NVME_INSTALL_TARGET_CMDS + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) DESTDIR=$(TARGET_DIR) \ + PREFIX=/usr install-bin +endef + +$(eval $(generic-package))
Add support for building NVME utility - a utility for interacting with standard NVM Express (optimized PCI Express SSD interface) devices. Signed-off-by: Mamatha Inamdar <mamatha4@linux.vnet.ibm.com> --- package/Config.in | 1 + package/nvme/0001-NVME-Makefile-cleanup.patch | 25 +++++++++++++++++++++++++ package/nvme/Config.in | 6 ++++++ package/nvme/nvme.hash | 2 ++ package/nvme/nvme.mk | 22 ++++++++++++++++++++++ 5 files changed, 56 insertions(+) create mode 100644 package/nvme/0001-NVME-Makefile-cleanup.patch create mode 100644 package/nvme/Config.in create mode 100644 package/nvme/nvme.hash create mode 100755 package/nvme/nvme.mk